gqrx icon indicating copy to clipboard operation
gqrx copied to clipboard

IQ Tool: QFileSystemWatcher, tolerate directory changes

Open willcode opened this issue 2 years ago • 3 comments

  • Use QFileSystemWatcher instead of polling
  • Sort by time instead of name
  • Rec time is not dependent on dir contents
  • Gray out more widgets while recording

This is similar to #1164 and part of #1171 together, with some added tolerance for files changing during recording or being named as if they were recorded in the future.

willcode avatar May 07 '23 12:05 willcode

I suggest adding a context menu to the file list to make file sorting order/property configurable by the user. Sometimes it may be better to sort by creation/modification time, but as file name contains it's recording time, sorting by name effectively sorts by creation time.

vladisslav2011 avatar May 07 '23 21:05 vladisslav2011

Good idea. There's still a problem to deal with, which is why I made it by creation time. The tool specifically looks at the last line in the list to determine which file is being recorded. It does this because rx creates the file and the tool doesn't know the name otherwise. Since the tool looks at an "*.raw" file (we could make it more specific) it is possibly for other files to appear last in the list. It's also possibly to get a file from a computer that has a different clock setting, and has what looks like a future time. When I tested these cases, things didn't work right.

willcode avatar May 07 '23 22:05 willcode

Maybe we should just "do it right" and have the rx signal that it's recording/playing a certain file. Then the IQ tool doesn't need to infer the file name.

willcode avatar May 07 '23 22:05 willcode