After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 41139 - Should return FALSE if errors during parsing config. file?
Should return FALSE if errors during parsing config. file?
Status: RESOLVED FIXED
Product: gnome-vfs
Classification: Deprecated
Component: Async operations
cvs (head)
Other All
: Normal normal
: ---
Assigned To: gnome-vfs maintainers
gnome-vfs maintainers
Depends on:
Blocks:
 
 
Reported: 2000-06-05 15:22 UTC by pavel
Modified: 2005-11-30 22:56 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
Proposed patch (against HEAD). (1.59 KB, patch)
2005-05-01 15:53 UTC, Christian Neumair
committed Details | Review

Description pavel 2001-09-10 00:33:38 UTC
FIXME: What should we do in parse_file if an error is hit -- some form of
notifying the user that the config file is bad but gracefully coninuing would be
in order.



------- Additional Comments From snickell@stanford.edu 2001-07-23 00:33:50 ----

Taking bugs previously assigned to Pavel, assigning them to myself. Will parse
them out at my leisure , but many are GnomeVFS bugs we should look at for 2.0



------- Bug moved to this database by unknown@bugzilla.gnome.org 2001-09-09 20:33 -------

The original reporter (pavel@eazel.com) of this bug does not have an account here.
Reassigning to the exporter, unknown@bugzilla.gnome.org.

Comment 1 Baptiste Mille-Mathias 2003-11-30 09:59:28 UTC
Hello, could we get news from this case ?
Is this bug still relevant ?

thanks
Comment 2 Seth Nickell 2004-10-27 03:49:49 UTC
MASS CHANGE VFS. Re-assigning bugs that I currently own in VFS to the proper
owner (probably Alex).
Comment 3 Christian Neumair 2005-05-01 15:44:21 UTC
Yes it is. The relevant code is in libgnomevfs/gnome-vfs-configuration.c.
Comment 4 Christian Neumair 2005-05-01 15:53:22 UTC
Created attachment 45907 [details] [review]
Proposed patch (against HEAD).

I've removed the parse_file retval. After applying the patch, we inform the
user that parsing on a particular file failed and continue parsing the
remaining files.
Comment 5 Christian Kellner 2005-11-30 22:56:49 UTC
Patch looked right to me, I committed it to HEAD. Thanks!