From: "Petr Vandrovec" <VANDROVE@vc.cvut.cz>
To: Denis Zaitsev <zzz@cd-club.ru>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH] matroxfb_base.c - a little fix
Date: Tue, 26 Feb 2002 20:43:45 +0100 [thread overview]
Message-ID: <12EE5C947552@vcnet.vc.cvut.cz> (raw)
On 27 Feb 02 at 0:26, Denis Zaitsev wrote:
> It was two identical if-else branches in matroxfb_decode_var - one for
> var->bits_per_pixel == 4
> and the other for
> var->bits_per_pixel <= 8
> . So, I've removed 4's one. It should be ok, if this branch was ok -
> i.e. was intentionally made the same as the 8bpp's. So, Petr, please,
> apply this fix, if all ok.
Somebody changed == 8 to <= 8 when I was not looking... But as
bits_per_pixel is already validated, it looks correct. Applied,
will go to Linus next round.
Best regards,
Petr Vandrovec
vandrove@vc.cvut.cz
next reply other threads:[~2002-02-26 19:45 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-02-26 19:43 Petr Vandrovec [this message]
2002-02-26 22:36 ` [PATCH] matroxfb_base.c - a little fix Denis Zaitsev
-- strict thread matches above, loose matches on Subject: below --
2002-02-27 0:01 Petr Vandrovec
2002-02-04 1:01 Denis Zaitsev
2002-02-04 1:08 ` Petr Vandrovec
2002-02-26 19:26 ` Denis Zaitsev
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=12EE5C947552@vcnet.vc.cvut.cz \
--to=vandrove@vc.cvut.cz \
--cc=linux-kernel@vger.kernel.org \
--cc=zzz@cd-club.ru \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.