In my file picker, paths to files have an extra "/" at the start of them. For eg. they show up as "//home/mikep" instead of "/home/mikep". Sorry if this isn't in the right component, I don't know where it should go.
Reporter: Please always specify which "Build ID" you're using, as found in the title bar of the main Mozilla window.
I'm seeing this with tip from this morning and every build as far back as I've been working onfilepicker (a month? more?). I'm almost certain this is an nsLocalFileUnix problem. I'll take this and investigate....
Pete, would you review?
Comment on attachment 68982 [details] [diff] [review] Patch Looks ok, but I wonder why we don't strip trailing slashes from fragment too. /be
Good question. Pete? Patch checked in, but leaving bug open for now for the trailing slash issue.
Agree, the trailing slash issue is rather annoying. I can do it, if you want to do it, that's cool too . . . --pete
*** Bug 84760 has been marked as a duplicate of this bug. ***
From my manual page of strlen: size_t strlen(const char *s); --> it returns size_t, not ssize_t.
ssize_t is the signed version, so size_t can just be cast over to it for not too big values of size_t. I just copied that part of the code from InitWithPath, but now that I look at it there seems to be no reason to use ssize_t here _or_ there. Or in Contains() for that matter....
> for not too big values Yes, that's it. If there's no reason to use the limited datatype, why use it? Maybe use size_t in all mentioned functions.
no more time to work on this...
I am no longer seeing this, is anybody else?
Christian, see comment #7, this has been checked in.