[Geeqie-devel] [PATCH] fix screen profile leak

2009-06-09 Thread Omari Stephens
Geeqie loads the screen profile for every image. If this is a file, there's no problem — the file itself is read in by lcms and the parsed profile object is dealt with properly in color_man.c. If the profile is in the X11 _ICC_PROFILE atom, though, the profile data is read and stored "in" a p

[Geeqie-devel] [PATCH] Re: Make "View in new window" a color-managed view

2009-06-09 Thread Omari Stephens
Vladimir Nadvornik wrote: On po 25. května 2009, Omari Stephens wrote: Nth time is the charm, apparently. Patch (against r1711) attached. Doesn't seem to affect panview. OK, if the fix is that simple we can add it now. But the change below looks wrong. The color processing should be change

Re: [Geeqie-devel] Doxygen and other improovements

2009-06-09 Thread Klaus Ethgen
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, As I am just adding the doxygen part to CODING I stumbled over the SVN commint message. I'd like to change it a bit (See also http://www.tpope.net/node/106, which applies to git but it is independent of the underlying rc system.): - - A short

Re: [Geeqie-devel] Doxygen and other improovements

2009-06-09 Thread Laurent Monin
Klaus Ethgen a écrit : > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA512 > > Hi, > > As I am just adding the doxygen part to CODING I stumbled over the SVN > commint message. > > I'd like to change it a bit (See also http://www.tpope.net/node/106, > which applies to git but it is independent of t

Re: [Geeqie-devel] [PATCH] fix screen profile leak

2009-06-09 Thread Vladimir Nadvornik
On Tuesday 09 June 2009 10:12:36 Omari Stephens wrote: > Geeqie loads the screen profile for every image. If this is a file, > there's no problem — the file itself is read in by lcms and the parsed > profile object is dealt with properly in color_man.c. > > If the profile is in the X11 _ICC_PROFIL

Re: [Geeqie-devel] [PATCH] Re: Make "View in new window" a color-managed view

2009-06-09 Thread Vladimir Nadvornik
On Tuesday 09 June 2009 10:50:25 Omari Stephens wrote: > Vladimir Nadvornik wrote: > > On po 25. května 2009, Omari Stephens wrote: > >> Nth time is the charm, apparently. Patch (against r1711) attached. > >> Doesn't seem to affect panview. > > > > OK, if the fix is that simple we can add it now.

Re: [Geeqie-devel] Doxygen and other improovements

2009-06-09 Thread Vladimir Nadvornik
On Tuesday 09 June 2009 11:22:54 Laurent Monin wrote: > Klaus Ethgen a écrit : > > -BEGIN PGP SIGNED MESSAGE- > > Hash: SHA512 > > > > Hi, > > > > As I am just adding the doxygen part to CODING I stumbled over the SVN > > commint message. > > > > I'd like to change it a bit (See also http:/

Re: [Geeqie-devel] Full screen changes in beta1

2009-06-09 Thread Vladimir Nadvornik
On Tuesday 09 June 2009 02:16:47 Reid Rivenburgh wrote: > Hi. The recent mail about possible bugs in beta 1 has reminded me to > write regarding a few other issues. It seems that in beta1, it's no > longer possible to zoom in/out, rotate, or switch images (via mouse > wheel) while in full screen

Re: [Geeqie-devel] [Patch] [Fullscreen image on pan view]

2009-06-09 Thread Vladimir Nadvornik
On Monday 08 June 2009 18:51:07 Laurent Monin wrote: > Oscar Miras a écrit : > > - I forgot to say that I know it's not 'docoxygenated' yet. I will do > > with all functions that I 'understand' and those wich are mine. > > > > - This is commented in pan_fullscreen_toggle_image function.If I do: > >

Re: [Geeqie-devel] Full screen changes in beta1

2009-06-09 Thread Reid Rivenburgh
On Tue, Jun 9, 2009 at 1:35 PM, Vladimir Nadvornik wrote: > On Tuesday 09 June 2009 02:16:47 Reid Rivenburgh wrote: >> Hi.  The recent mail about possible bugs in beta 1 has reminded me to >> write regarding a few other issues.  It seems that in beta1, it's no >> longer possible to zoom in/out, rot