Re: [PATCH][2.5] UTF-8 support in console

Christoph Hellwig (hch@infradead.org)
Sat, 31 May 2003 15:21:33 +0100


+static void set_inverse_trans_unicode(struct vc_data *conp, struct uni_pagedir *p)

Please linewrap after 80 chars.

+{
+ int i, j, k, glyph;
+ u16 **p1, *p2;
+ u16 *q;
+
+ if (!p) return;

Please split this into two lines. Can p ever be null_

+ q = p->inverse_trans_unicode;
+
+ if (!q) {

Kill the blank line above.

+ q = p->inverse_trans_unicode = (u16 *)
+ kmalloc(MAX_GLYPH * sizeof(u16), GFP_KERNEL);

The cast is not needed. And btw, where is q freed?

+ if (!q) return;

Two lines again.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/