spatny priklad fbcon?

[Gratz Jaroslav]

On Wed, 28 Mar 2001, Michl Ladislav wrote:

> dobry vecer,
>
> dopisuju funkci con_font_op do newport_con.c (uz chodi, ale font je zatim
> spolecny pro vsechny konzole) a chtel jsem vzit jako priklad fbcon.c
> (jadro 2.4.2) vsimnete si podtrzeneho a pouziti promenne h.
>
> static inline int fbcon_copy_font(int unit, struct console_font_op *op)
> {
>     struct display *od, *p = &fb_display[unit];
>     int h = op->height;

Tohle se vola jen pri kopirovani fontu (z konzole na konzoli?) a docela
bych i rekl, ze tady se v op->height nenachazi vyska, ale cislo virtualni
konzoly, pro kterou se operace kopirovani provadi (ona tam ta vyska neni
pravdepodobne na nic potreba, takze si tak ten clovek co to psal
"usnadnil" praci a pouzil "nevyuzitou" promennou).

<snip>

>     if (w <= 8) {
>         for (i = 0; i < op->charcount; i++) {
>             memcpy(p, data, h);
>             data += 32;
>             p += h;
>             ^^^^^^^ - tady se s h pracuje jako s vyskou fontu
>         }
>     }

Tady je to myslim jasne, je to vyska.

J.G.