fifonik
repeatable crash - distorted files click+drag
Steps to reproduce:
- Add a single reference file so that the reference area is valid
- Add a single distorted file to the project
- Click the file list item and drag to the column header
Expected behavior: Nothing happen
Actual behavior: App crashes
Version: 1.4.9b
Additional notes:
It's a weird edge case but my clumsy fingers have made it happen twice, so I thought I'd pass it along, especially since it doesn't happen in many cases. For instance, if multiple distorted files are in the list, or if no reference file is open. I hope this is helpful, it's pretty easily reproducible, though obviously a low priority.
I thought I wasn't able to, then got it to repeat again. I'm on Windows 10.
- Open FFMetrics
- Start a new project (ctrl+shift+N)
- Drag reference file to Reference 'path to file'
- Drag two files to 'distored' area, let media info load
- Delete one of the files (del)
- Drag distorted file to 'distorted' area, let media info load
- Drag distorted file to column header (sometimes I had to do this twice)
This 'feels' less perfectly repeatable today than it did last night, but I've also attached a file to help. I have such anxiety about uploading bad bug reports to projects 😅
https://github.com/fifonik/FFMetrics/assets/8808740/d5323e57-430b-4de6-9136-7a66507896a8
Thanks, I was able to re-produce it twice. Unfortunately, not really sure how to re-produce the issue reliably that is required to check if my code change will fix it. Still trying.
I've made some changes and hope it fixed the issue. Can send you executable for testing if you like.
Here you are. Please let me know when you download it. Thanks. {link deleted}
I can't promise it's fixed - it wasn't as perfectly reproducible as initially thought - but none of my attempts have broken it so far. I included some weird dumb stuff, including multi-select, weird ordering/deleting, etc. And all seemed to be fine. I'll keep working with this version and let you know if something comes up, otherwise this seems fixed. I won't close this in case you want to tie it to code changes or something, but you obviously can feel free to.