Re: [Nautilus-list] [PATCH] Use GConf for background settings



On Thu, 2002-02-07 at 17:48, Darin Adler wrote:
> On 1/25/02 3:40 PM, "Håvard Wigtil" <havardw stud ntnu no> wrote:
> 
> > The included patch reads desktop settings from GConf, and updates the
> > desktop when these settings change. This is (an attempt) at a straight
> > port, so it doesn't do any more error checking than the original code
> > did. It will crash on bad settings for colors or file name.
> 
> Please fix that, regardless of the status of the original code. I don't
> think you need "complete" error checking, just enough to avoid a crash if
> you get an unexpected value from gconf.

I'll send you a patch that survives gconftool --break-key by the end of
the week. ;)

Thank you for taking time to comment on the code, I'll make the changes
you suggested in the new patch.

-- 
   Håvard

mailto:havardw stud ntnu no||http://www.stud.ntnu.no/~havardw||73 52 55
76
All it takes to start an avalanche is a single snowflake||Or a
snowboarder
        Oh! Un Fraggle! Regarde, maman! J'ai attrapé un Fraggle!





[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]