Steve is, as usual, correct.

I'm on vacation and out of the country, but it looks like there are battling forms of cleverness here and they're defeating each other.

MainWindow::dropEvent() _should_ be putting the exact name into inputFileNameText via the setText().  The explicit quoting in MainWindow::browseInputFile() is quite suspicious. We're calling QProcess, not a shell, so we shouldn't be in the business of performing shell quoting to handle whitespace in pathnames. If we're going to support multiple names, we should probably make  inputFileNameText take a QStringList and quit trying to stuff everything into a single quoted string.  Once we do this, MW::applyActionX() probably needs some attention when getName() returns a string array.  

On the other issue, if the output filename isn't an absolute path, we should probably call to find the Desktop (DOcuments?) directory and then canonicalize that dir + path.

On Wed, Jun 4, 2014 at 12:23 AM, SRE <> wrote:
At 05:01 PM 6/3/2014, tsteven4 wrote:
>Thanks for pointing out the issues.  However, I am strongly opposed to
>patching up the performance of the windows GUI in the command line
>tool.  I think any necessary fiddling should be done in the GUI itself.

Strongly agree! Just don't know where to look...

Sort of surprised the routines underlying the GUI aren't automatically
making these sorts of corrections/edits. Maybe there's a mode switch?


Learn Graph Databases - Download FREE O'Reilly Book
"Graph Databases" is the definitive new guide to graph databases and their
applications. Written by three acclaimed leaders in the field,
this first edition is now available. Download your free book today!
Gpsbabel-code mailing list