Re: [Gimp-developer] refactor palette loading code

2013-09-08 Thread Warren Turkal
Hey, I will get the format-patch as soon as I can. However, if you want to do it the git way and get it sooner, you can always add my remote to your git with the following command: git remote add wt g...@github.com:wt/gimp.git Note that "wt" can be whatever alias you want, but my commands below a

Re: [Gimp-developer] refactor palette loading code

2013-09-08 Thread Jehan Pagès
Hi, I searched a little in this web UI, but couldn't find: is there a way to generate a proper patch from this? Otherwise, could you generate one with "git format-patch origin/master", run on your locale branch? Thanks. Jehan On Mon, Sep 9, 2013 at 12:20 AM, Warren Turkal wrote: > BTW, I teste

Re: [Gimp-developer] refactor palette loading code

2013-09-08 Thread Warren Turkal
BTW, I tested this with gpl, act, aco, pal, and css palette files. I couldn't find RIFF file. If anyone has one, would you mind emailing it to me so that I can try loading it? Thanks, wt On Sun, Sep 8, 2013 at 5:06 AM, Warren Turkal wrote: > Hi again, > > Here's a > link

[Gimp-developer] refactor palette loading code

2013-09-08 Thread Warren Turkal
Hi again, Here's a linkto a commit containing a refactor of the palette loading code. I have moved the file open/close logic to common code. This change actually removes more code than it adds. Here's a interesting diffst