I think that would be a good idea.  This is the way it seems to always go
for me too--have a patch to add something to KiCad, and expose an issue we
didn't realize we had :)

Adam Wolf

On Mon, Dec 7, 2015 at 12:10 PM, Mark Roszko <[email protected]> wrote:

> wxFileName makes some "choices". i.e. its constructor/assignment takes
> the string and tries to separate file from path.
>
> in the case of wxFileName = "helloworld"; it assumes its a filename
> in the case of wxFileName = "/helloworld", it assumes its a file name
> in the case of wxFilename = "helloworld/"; it assumes its a folder
>
> and of course AppendDir adds a directory the path and because filename
> is separate from directory, the end behavior is what you say. (i.e.
> directory gets concatted before file)
>
>
> The proper solution to fix this would be to GetPathSeparator() and
> append it to the environment variable string after reading but before
> assigning it to the tmpFileName, thus creating
> double slashes in the worst case but wxFileName should clean that out.
> I'll add it to the patch later tonight if need be.
>
_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to