[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] jpeg image reader
From: |
Bean |
Subject: |
Re: [PATCH] jpeg image reader |
Date: |
Tue, 15 Jan 2008 22:36:14 +0800 |
On Jan 15, 2008 10:27 PM, Marco Gerards <address@hidden> wrote:
> Bean <address@hidden> 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 = grub_jpeg_get_byte (data); /* sampling factor */
> >> > if (!id)
> >> > {
> >> > data->vs = ss & 0xF; /* vertical sampling */
> >> > data->hs = ss >> 4; /* horizontal sampling */
> >>
> >> Please use correct interpunction.
> >
> > I'm sorry, what do you mean by correct interpunction ?
>
> That I prefer:
>
> /* vertical sampling */ -> /* Vertical sampling. */
>
> 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.
--
Bean
- Re: [PATCH] jpeg image reader, (continued)
- Re: [PATCH] jpeg image reader, Bean, 2008/01/14
- Re: [PATCH] jpeg image reader, Marco Gerards, 2008/01/15
- Re: [PATCH] jpeg image reader, Robert Millan, 2008/01/15
- Re: [PATCH] jpeg image reader, Marco Gerards, 2008/01/15
- Re: [PATCH] jpeg image reader, Robert Millan, 2008/01/15
- Re: [PATCH] jpeg image reader, Marco Gerards, 2008/01/15
- Re: [PATCH] jpeg image reader, Bean, 2008/01/15
- Re: [PATCH] jpeg image reader, Marco Gerards, 2008/01/15
- Re: [PATCH] jpeg image reader, Bean, 2008/01/15
- Re: [PATCH] jpeg image reader, Marco Gerards, 2008/01/15
- Re: [PATCH] jpeg image reader,
Bean <=
- Re: [PATCH] jpeg image reader, Marco Gerards, 2008/01/15
- Re: [PATCH] jpeg image reader, Bean, 2008/01/15
- Re: [PATCH] jpeg image reader, Bean, 2008/01/15
- Re: [PATCH] jpeg image reader, Marco Gerards, 2008/01/21
- Re: [PATCH] jpeg image reader, Bean, 2008/01/22