Re: [Gimp-developer] config - display dependency

2010-01-07 Thread Martin Nordholts
Sven Neumann wrote: When we split up the GIMP core into sub-directories, we also split the code into UI and core. You will find a few files in the UI sub-directories that hold nothing but config settings. Since the non-UI GIMP should be able to read the same config files that the UI-enabled

Re: [Gimp-developer] config - display dependency

2010-01-06 Thread Sven Neumann
On Tue, 2010-01-05 at 23:31 +0100, Martin Nordholts wrote: Hi again, Sven Neumann wrote: Since there is no GTK+ dependency in the two files, nor any dependency on code that has a GTK+ dependency, there is no dependency from core to the UI here. The files are just better located in

Re: [Gimp-developer] config - display dependency

2010-01-05 Thread Martin Nordholts
Sven Neumann wrote: Hi Martin, I've found your last commit message quite confusing and so is the FIXME that you added to gimpdisplayconfig.c. I'd very much welcome if such texts would not be added to the code without prior discussion here. Your comment states: /* FIXME: If we can get

Re: [Gimp-developer] config - display dependency

2010-01-05 Thread Sven Neumann
Hi, On Tue, 2010-01-05 at 22:46 +0100, Martin Nordholts wrote: If you find the comment confusing I should explain it in more detail. First, I should define the terms I use in the comment. When I say UI code I am referring to any code that is in modules with a GTK+ dependency, which

Re: [Gimp-developer] config - display dependency

2010-01-05 Thread Martin Nordholts
Hi again, Sven Neumann wrote: Since there is no GTK+ dependency in the two files, nor any dependency on code that has a GTK+ dependency, there is no dependency from core to the UI here. The files are just better located in app/display as that is where they are used. I don't understand why