On 10/04/2018 13:52, Black Michael via wsjt-devel wrote:
When the save directory is messed up the error message isn't quite as useful as it could be.

Add the file path to the wav file error message

https://www.dropbox.com/s/u3x6xatwgpzapnu/error.patch?dl=1 <https://www.dropbox.com/s/u3x6xatwgpzapnu/error.patch?dl=0>


de Mike W9MDB

HI Mike,

off list - his time correctly.

Thanks for this patch. I will do a similar thing in a different way. The reason being that the ,WAV file write uses an asynchronous future and promise to do it's work so as to not block the main GUI thread. This means that using a variable in the main thread, set when initiating the future, later when the promise is delivered is unwise as it's value may well have changed. This future already returns a string as a promise which is any error message found in the deferred work so I will extend the promise to include the file name. That way the message is guaranteed to specify the actual file path that causes the error.

73
Bill
G4WJS.

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
wsjt-devel mailing list
wsjt-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wsjt-devel

Reply via email to