Re: [PATCH] jpeg image reader

2008-01-22 Thread Bean
On Jan 21, 2008 6:21 PM, Marco Gerards <[EMAIL PROTECTED]> wrote: > Bean <[EMAIL PROTECTED]> writes: > > Hi, > > > i finish converting the idct and color code, now it should be ok. > > If you are 100% sure all code is yours and there are no legal issues, > feel free to commit it. Your code looks g

Re: [PATCH] jpeg image reader

2008-01-21 Thread Marco Gerards
Bean <[EMAIL PROTECTED]> writes: Hi, > i finish converting the idct and color code, now it should be ok. If you are 100% sure all code is yours and there are no legal issues, feel free to commit it. Your code looks good to me :-) -- Marco ___ Grub

Re: [PATCH] jpeg image reader

2008-01-15 Thread Bean
Hi, i finish converting the idct and color code, now it should be ok. /* * GRUB -- GRand Unified Bootloader * Copyright (C) 2008 Free Software Foundation, Inc. * * GRUB is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as pu

Re: [PATCH] jpeg image reader

2008-01-15 Thread Bean
On Jan 16, 2008 12:01 AM, Marco Gerards <[EMAIL PROTECTED]> wrote: > The license state that their README should be distributed together > with GRUB 2, if I understand it correctly. As you might guess, this > is not acceptable. Please be careful when using code from other > projects (or even bette

Re: [PATCH] jpeg image reader

2008-01-15 Thread Marco Gerards
Bean <[EMAIL PROTECTED]> writes: [...] >> Same for the other two lines. With this change, only the legal >> question remains. > > oh, i see. > > The IJG's source code: > > http://www.ijg.org/files/jpegsrc.v6b.tar.gz > > there is a legal issue section in the README file. The license state that t

Re: [PATCH] jpeg image reader

2008-01-15 Thread Bean
On Jan 15, 2008 10:27 PM, Marco Gerards <[EMAIL PROTECTED]> wrote: > Bean <[EMAIL PROTECTED]> writes: > > > [...] > > >> > for (i = 0; i < cc; i++) > >> > { > >> > int id, ss; > >> > > >> > id = grub_jpeg_get_byte (data) - 1; > >> > if ((id < 0) || (id >= 3)) > >> > re

Re: [PATCH] jpeg image reader

2008-01-15 Thread Marco Gerards
Bean <[EMAIL PROTECTED]> writes: [...] >> > for (i = 0; i < cc; i++) >> > { >> > int id, ss; >> > >> > id = grub_jpeg_get_byte (data) - 1; >> > if ((id < 0) || (id >= 3)) >> > return grub_error (GRUB_ERR_BAD_FILE_TYPE, "jpeg: invalid index"); >> > >> > ss = gr

Re: [PATCH] jpeg image reader

2008-01-15 Thread Marco Gerards
Bean <[EMAIL PROTECTED]> writes: > On Jan 15, 2008 8:46 PM, Marco Gerards <[EMAIL PROTECTED]> wrote: >> Robert Millan <[EMAIL PROTECTED]> writes: >> >> > On Tue, Jan 15, 2008 at 01:21:04PM +0100, Marco Gerards wrote: >> >> >> >> >> >> I do not think these algorithms can be copyrighted and are in t

Re: [PATCH] jpeg image reader

2008-01-15 Thread Bean
On Jan 15, 2008 7:01 PM, Marco Gerards <[EMAIL PROTECTED]> wrote: > > static grub_err_t > > grub_jpeg_decode_sof (struct grub_jpeg_data *data) > > { > > int i, cc; > > grub_uint32_t next_marker; > > > > next_marker = data->file->offset; > > next_marker += grub_jpeg_get_word (data); > > > >

Re: [PATCH] jpeg image reader

2008-01-15 Thread Bean
On Jan 15, 2008 8:46 PM, Marco Gerards <[EMAIL PROTECTED]> wrote: > Robert Millan <[EMAIL PROTECTED]> writes: > > > On Tue, Jan 15, 2008 at 01:21:04PM +0100, Marco Gerards wrote: > >> >> > >> >> I do not think these algorithms can be copyrighted and are in the > >> >> public domain? Can you check

Re: [PATCH] jpeg image reader

2008-01-15 Thread Marco Gerards
Robert Millan <[EMAIL PROTECTED]> writes: > On Tue, Jan 15, 2008 at 01:21:04PM +0100, Marco Gerards wrote: >> >> >> >> I do not think these algorithms can be copyrighted and are in the >> >> public domain? Can you check this? >> > >> > Only actual code can be copyrighted. Algorithms can only be

Re: [PATCH] jpeg image reader

2008-01-15 Thread Robert Millan
On Tue, Jan 15, 2008 at 01:21:04PM +0100, Marco Gerards wrote: > >> > >> I do not think these algorithms can be copyrighted and are in the > >> public domain? Can you check this? > > > > Only actual code can be copyrighted. Algorithms can only be patented. > > > > Copyright expires into public d

Re: [PATCH] jpeg image reader

2008-01-15 Thread Marco Gerards
Robert Millan <[EMAIL PROTECTED]> writes: > On Tue, Jan 15, 2008 at 12:01:47PM +0100, Marco Gerards wrote: >> Bean <[EMAIL PROTECTED]> writes: >> >> > On Jan 14, 2008 2:31 AM, Marco Gerards <[EMAIL PROTECTED]> wrote: >> >> > /* >> >> > * GRUB -- GRand Unified Bootloader >> >> > * Copyright

Re: [PATCH] jpeg image reader

2008-01-15 Thread Robert Millan
On Tue, Jan 15, 2008 at 12:01:47PM +0100, Marco Gerards wrote: > Bean <[EMAIL PROTECTED]> writes: > > > On Jan 14, 2008 2:31 AM, Marco Gerards <[EMAIL PROTECTED]> wrote: > >> > /* > >> > * GRUB -- GRand Unified Bootloader > >> > * Copyright (C) 2008 Free Software Foundation, Inc. > >> > >>

Re: [PATCH] jpeg image reader

2008-01-15 Thread Marco Gerards
Bean <[EMAIL PROTECTED]> writes: > On Jan 14, 2008 2:31 AM, Marco Gerards <[EMAIL PROTECTED]> wrote: >> > /* >> > * GRUB -- GRand Unified Bootloader >> > * Copyright (C) 2008 Free Software Foundation, Inc. >> >> Did you write all the code yourself? > > yes, but i take some code segment from

Re: [PATCH] jpeg image reader

2008-01-14 Thread Bean
On Jan 14, 2008 2:31 AM, Marco Gerards <[EMAIL PROTECTED]> wrote: > > /* > > * GRUB -- GRand Unified Bootloader > > * Copyright (C) 2008 Free Software Foundation, Inc. > > Did you write all the code yourself? yes, but i take some code segment from Independent JPEG Group's implementation, su

Re: [PATCH] jpeg image reader

2008-01-13 Thread Marco Gerards
Bean <[EMAIL PROTECTED]> writes: Hi, > change the handling of width/height that's not aligned, now it use the > exact size instead of padding. You forgot the changelog entry. I'll do a quick review. Especially the longjmp stuff should be changed, IMO. Otherwise the code is of good quality and

Re: [PATCH] jpeg image reader

2008-01-13 Thread Bean
Hi, change the handling of width/height that's not aligned, now it use the exact size instead of padding. /* * GRUB -- GRand Unified Bootloader * Copyright (C) 2008 Free Software Foundation, Inc. * * GRUB is free software: you can redistribute it and/or modify * it under the terms of

Re: [PATCH] jpeg image reader

2008-01-12 Thread Bean
Hi, changes in this new version: handle width/height that's not multiple of 8 add support for 1:1:1 sampling skip unrecognized marker instead of error. verify the size of sof, sos, quantization table and huffman table. /* * GRUB -- GRand Unified Bootloader * Copyright (C) 2008 Free Softwa

Re: [PATCH] jpeg image reader

2008-01-10 Thread Bean
On Jan 10, 2008 4:33 AM, Vesa Jääskeläinen <[EMAIL PROTECTED]> wrote: > Nice job for implementing JPEG. This is the major reason I made the > bitmap loader interface, to allow other people to write loaders... now > if we would just have PNG support ;) This is my next project. > I know lots of JPE

Re: [PATCH] jpeg image reader

2008-01-09 Thread Vesa Jääskeläinen
Bean wrote: Hi, I just fix a small bug that can cause color distortion, here is the new patch. Nice job for implementing JPEG. This is the major reason I made the bitmap loader interface, to allow other people to write loaders... now if we would just have PNG support ;) I will not comment

Re: [PATCH] jpeg image reader

2008-01-09 Thread Bean
Hi, I just fix a small bug that can cause color distortion, here is the new patch. diff --git a/conf/i386-pc.rmk b/conf/i386-pc.rmk index a46bc74..4beeeff 100644 --- a/conf/i386-pc.rmk +++ b/conf/i386-pc.rmk @@ -136,7 +136,7 @@ pkglib_MODULES = biosdisk.mod _chain.mod _linux.mod linux.mod normal.

[PATCH] jpeg image reader

2008-01-09 Thread Bean
Hi, this patch handle jpeg image files. diff --git a/conf/i386-pc.rmk b/conf/i386-pc.rmk index a46bc74..4beeeff 100644 --- a/conf/i386-pc.rmk +++ b/conf/i386-pc.rmk @@ -136,7 +136,7 @@ pkglib_MODULES = biosdisk.mod _chain.mod _linux.mod linux.mod normal.mod \ _multiboot.mod chain.mod mult