Hi,

On Sunday 18 February 2007 11:36, Christian Stimming wrote:
> Am Samstag, 17. Februar 2007 18:22 schrieb Martin Preuss:
[...]
> This particular separator mixup that I've reported could (IMHO) be fixed by
> the attached patch (the macro GWEN_DIR_SEPARATOR as a character was
> introduced by me in gwenhywfarapi.h a few revisions ago). Does that seem ok
> to you?
[...]

Hmm, we always use "/" for paths (e.g. for GWEN_DB paths), I don't think we 
should change that...

Since the path stuff is used widely we shouldn't introduce these changes 
there.

I think it would be better if we somewhat caught the case where a string 
starts with "x:". I still don't see where this conversion takes place for the 
string we found in AqHBCI, but I didn't look into it...

[...]
> As you've already said in the other thread, the path separator doesn't seem
> too much of an issue anyway. On the other hand, the actual error in aqhbci
> (in that yet other thread) with the extra escaping of "C:\" is indeed a
> problem - can you respond to my proposal there, or should I already suggest
> a path? Thanks for that.
[...]

See above, we should rather change the caller of that function to check for 
the "x:" in advance...


Regards
Martin


-- 
"Things are only impossible until they're not"

AqBanking - http://www.aqbanking.de/
LibChipcard - http://www.libchipcard.de/

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Aqbanking-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/aqbanking-devel

Reply via email to