On Mon, 2005-10-17 at 19:26 +0200, Christian Neumair wrote:
> Am Montag, den 17.10.2005, 16:18 +0200 schrieb Alexander Larsson:
> > On Fri, 2005-10-14 at 22:51 +0200, Christian Neumair wrote:
> > > OK, the GnomeVFS ls -lga parser is seriously hosed by only handling one
> > > filename at a time, but that's a different topic.
> > > The attached patch fixes bug 163671 [1]. It makes GnomeVFS pass all
> > > "test-parse-ls-lga" tests, and even display files named like a time or
> > > like a year correctly in FTP directories, which was a bit tricky, since
> > > the FTP module didn't split the string at '\n' characters before passing
> > > it to the parser. However, since the API suggests that
> > > gnome_vfs_parse_ls_lga can parse multiple lines ([2]), I've preferred to
> > > add multiline detection semantics to the latter.
> > >
> > > [1] http://bugzilla.gnome.org/show_bug.cgi?id=163671
> > > [2] which is - I repeat - NOT true as of writing, it only sets the first
> > > member of the passed-in filename array
> >
> > I'm not sure why you think the filename argument is an array. Its just a
> > normal out-parameter.
>
> Ouch, sorry. Please ignore my stupid comment.
>
> > Furthermore, it looks like this is not a full patch. vfs_parse_filedate
> > adds more arguments, but the calls to it are not changed.
>
> That was removed by accident.
I'm not sure I understand the newline checks in is_last_column. Is that
really required?
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Alexander Larsson Red Hat, Inc
[EMAIL PROTECTED] [EMAIL PROTECTED]
He's a lonely flyboy househusband on his last day in the job. She's a blind
motormouth journalist with the soul of a mighty warrior. They fight crime!
_______________________________________________
gnome-vfs-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/gnome-vfs-list
