Re: [Gimp-developer] ACO and CSS palette importing (patch attached)

2007-08-08 Thread Sven Neumann
Hi,

On Tue, 2007-08-07 at 18:36 +0200, Michael Natterer wrote:

> Please fix these warnings and attach the updated patch to
> bugzilla. This way it won't get lost.
> 
> The patch can unfortunately not be included in 2.4 because
> we are too close to a release, and we don't depend on a glib
> version that has GRegex yet. I'll look into applying it after
> the 2.4 release so it will be in 2.6.

I already applied the ACO import part of the patch. Nicola has attached
a patch with the CSS import code to bug #464480. We will look at that
for GIMP 2.6.


Sven


___
Gimp-developer mailing list
Gimp-developer@lists.XCF.Berkeley.EDU
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer


Re: [Gimp-developer] ACO and CSS palette importing (patch attached)

2007-08-07 Thread Michael Natterer
Thanks, that patch looks generally fine, also coding style wise,
however:

gimppalette-load.c: In function ‘gimp_palette_load_aco’:
gimppalette-load.c:440: warning: unused variable ‘color_bytes’
gimppalette-load.c:598:28: warning: unknown escape sequence '\)'
gimppalette-load.c: In function ‘gimp_palette_load_css’:
gimppalette-load.c:593: warning: unused variable ‘status’
gimppalette-load.c:592: warning: unused variable ‘format2_preamble’
gimppalette-load.c:591: warning: unused variable ‘color_info’
gimppalette-load.c:590: warning: unused variable ‘header’
gimppalette-load.c:589: warning: unused variable ‘i’
gimppalette-load.c:588: warning: unused variable ‘number_of_colors’
gimppalette-load.c:587: warning: unused variable ‘format_version’
gimppalette-load.c:586: warning: unused variable ‘color_bytes’
gimppalette-load.c: In function ‘gimp_palette_load_detect_format’:
gimppalette-load.c:679: warning: ISO C90 forbids mixed declarations and
code

Please fix these warnings and attach the updated patch to
bugzilla. This way it won't get lost.

The patch can unfortunately not be included in 2.4 because
we are too close to a release, and we don't depend on a glib
version that has GRegex yet. I'll look into applying it after
the 2.4 release so it will be in 2.6.

thanks,
--mitch


On Sat, 2007-08-04 at 19:45 +0100, Nicola Archibald wrote:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA1
> 
> Hi All,
> 
> Attached is a patch, against svn, which adds ACO importing, and CSS
> color extraction, for the palette system.
> 
> The ACO code only reads the version 1 section at the moment, I plan on
> finishing the version 2 support to extract color names a little later.
> Color spaces supported are RGB, CMYK, HSV and greyscale, but not Lab.
> 
> The CSS import extracts colors defined in a .css file.
> 
> 
> If this patch meets the standards, perhaps someone would like to
> incorporate it into the repository.
> 
> This patch would, I believe, enable the closing of Bug #316618

___
Gimp-developer mailing list
Gimp-developer@lists.XCF.Berkeley.EDU
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer


Re: [Gimp-developer] ACO and CSS palette importing (patch attached)

2007-08-07 Thread Michael Natterer
[resending, apparently the firts try didn't make it...]

Thanks, that patch looks generally fine, also coding style wise,
however:

gimppalette-load.c: In function ‘gimp_palette_load_aco’:
gimppalette-load.c:440: warning: unused variable ‘color_bytes’
gimppalette-load.c:598:28: warning: unknown escape sequence '\)'
gimppalette-load.c: In function ‘gimp_palette_load_css’:
gimppalette-load.c:593: warning: unused variable ‘status’
gimppalette-load.c:592: warning: unused variable ‘format2_preamble’
gimppalette-load.c:591: warning: unused variable ‘color_info’
gimppalette-load.c:590: warning: unused variable ‘header’
gimppalette-load.c:589: warning: unused variable ‘i’
gimppalette-load.c:588: warning: unused variable ‘number_of_colors’
gimppalette-load.c:587: warning: unused variable ‘format_version’
gimppalette-load.c:586: warning: unused variable ‘color_bytes’
gimppalette-load.c: In function ‘gimp_palette_load_detect_format’:
gimppalette-load.c:679: warning: ISO C90 forbids mixed declarations and
code

Please fix these warnings and attach the updated patch to
bugzilla. This way it won't get lost.

The patch can unfortunately not be included in 2.4 because
we are too close to a release, and we don't depend on a glib
version that has GRegex yet. I'll look into applying it after
the 2.4 release so it will be in 2.6.

thanks,
--mitch


On Sat, 2007-08-04 at 19:45 +0100, Nicola Archibald wrote:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA1
> 
> Hi All,
> 
> Attached is a patch, against svn, which adds ACO importing, and CSS
> color extraction, for the palette system.
> 
> The ACO code only reads the version 1 section at the moment, I plan on
> finishing the version 2 support to extract color names a little later.
> Color spaces supported are RGB, CMYK, HSV and greyscale, but not Lab.
> 
> The CSS import extracts colors defined in a .css file.
> 
> 
> If this patch meets the standards, perhaps someone would like to
> incorporate it into the repository.
> 
> This patch would, I believe, enable the closing of Bug #316618

___
Gimp-developer mailing list
Gimp-developer@lists.XCF.Berkeley.EDU
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer


Re: [Gimp-developer] ACO and CSS palette importing (patch attached)

2007-08-05 Thread Sven Neumann
Hi,

On Sat, 2007-08-04 at 19:45 +0100, Nicola Archibald wrote:

> Attached is a patch, against svn, which adds ACO importing, and CSS
> color extraction, for the palette system.
> 
> The ACO code only reads the version 1 section at the moment, I plan on
> finishing the version 2 support to extract color names a little later.
> Color spaces supported are RGB, CMYK, HSV and greyscale, but not Lab.
> 
> The CSS import extracts colors defined in a .css file.
> 
> 
> If this patch meets the standards, perhaps someone would like to
> incorporate it into the repository.
> 
> This patch would, I believe, enable the closing of Bug #316618

Very nice, thank you. Could you please attach these patches to the bug
report? That will make sure that they aren't forgotten. Perhaps I can
even get to review them later tonight. But it would still be nice to
have them attached to the bug report.


Sven


___
Gimp-developer mailing list
Gimp-developer@lists.XCF.Berkeley.EDU
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gimp-developer