Re: UI Freeze Break for the Desktop Background capplet

[Luis Villa <luis villa gmail com>]

On 8/15/05, Rodney Dawes <dobey novell com> wrote:
> I've compiled it, and it doesn't crash. The code really isn't changing
> that much. It's basically moving a bunch of calls into a new method,
> and adding a check to make sure the key is writable. Though, now that
> I look at the code beyond things that would be obvious and not compile,
> I see a couple of issues, though nothing fatal. One line of code that
> does not need to be there (that I had already asked the patch author
> to remove), a small style issue, and a leak. All of these are simple
> to fix though, and I'll put up a patch that has those fixes tonight.

Thanks.

Luis