On windows 7
1. Copy/Paste works
2. Drag/Drop works
3. Copy as Path/Paste FAILS (shift right click to get a menu with copy as path).  The quoted value appears in the window, e.g. "C:\Users\tsteven4\Documents\Shared\ma.gpx"
gpsbabel -w -r -t -i gpx -f "C:\Users\tsteven4\Documents\Shared\ma.gpx" -x transform,trk=wpt -o gpx -F junk.gpx

Cannot open '"C:/Users/tsteven4/Documents/Shared/ma.gpx"' for read. Error was 'The filename, directory name, or volume label syntax is incorrect.'.


Error running gpsbabel: Process exited unsucessfully with code 1

One could argue that 3. isn't too important with 1 & 2 working.

On 6/8/2014 9:59 PM, Robert Lipe wrote:
1+2: Done.  Needs tested on Windows to confirm the original report is fixed.
3) Red herring.  We only quote what's in the display, not what's actually used.  I've verified that dropped filenames with spaces and quotes are handled correctly.
4) Relevant data structures are already QStringList and used properly.



On Fri, Jun 6, 2014 at 11:56 PM, Robert Lipe <robertlipe@gpsbabel.org> wrote:
I'm back in my home country, but about to retire for the evening.  I think we've identified the following issues here.

1) Windows stupidly makes local file drops into URLs because it's Windows.
Solution1) replace path() in MainWindow::dropEvent with toLocalFile()

2) Windows stupidly mixes forward slashes and backslashes in filenames because Windows.
Solution2) If http://qt-project.org/doc/qt-5/qurl.html#path can be believed, "...will use forward slashes"

3) We quote quoted data.
Solution3) In MainWindow::browseInputFile, test if first character is a quote and then skip.

4) We mishandle multiple files dropped.
PossibleSolution) Disallow that.
BetterSolution) Make relevant data structures handle QStringList instead of glomming them together (badly) at ahd end of browseInputFile()

5) Apparently a drag of an output file is actually useful.  (It also sounds dangerous for accidental overwrites...)
Solution5) don't subclass MainWindow::dragEnterEvent() & dropEvent()  Instead subclass the specific input widget for the current code and new code (bonus points for refactoring/reusing) to set outputFileNames_ for the new output drop events.

They're approximately ordered by development cost.  I think #1 and 2 combined are a one-liner plus testing.

I'll put this on my list tackle, but time's pretty scant for the next few weeks so if you're a budding Babel-head looking for a starter project, please go for it.

RJL


On Thu, Jun 5, 2014 at 1:07 PM, SRE <steve-babel@climber.org> wrote:
At 06:12 AM 6/5/2014, Jan Depner wrote:
>Not necessarily.  You may want to overwrite an existing file.

... or just change the extension without having to browse or
type or paste the full path.


------------------------------------------------------------------------------
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!
http://p.sf.net/sfu/NeoTech
_______________________________________________
Gpsbabel-code mailing list  http://www.gpsbabel.org
Gpsbabel-code@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/gpsbabel-code




------------------------------------------------------------------------------
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://www.hpccsystems.com


_______________________________________________
Gpsbabel-code mailing list  http://www.gpsbabel.org
Gpsbabel-code@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/gpsbabel-code