Messages in this thread | | | Date | Sun, 26 Sep 1999 01:09:27 +0200 | From | Martin Dalecki <> | Subject | Re: linux/drivers/char/defkeymap.c |
| |
Andries.Brouwer@cwi.nl wrote: > > In patch2.3.14 I find: > > diff -u --recursive --new-file v2.3.13/linux/drivers/char/defkeymap.c linux/dri\ > vers/char/defkeymap.c > --- v2.3.13/linux/drivers/char/defkeymap.c Fri Feb 24 11:38:27 1995 > +++ linux/drivers/char/defkeymap.c Thu Aug 12 10:17:17 1999 > @@ -233,6 +233,7 @@ > {'A', 'A', '\305'}, {'a', 'a', '\345'}, > {'A', 'E', '\306'}, {'a', 'e', '\346'}, > {',', 'C', '\307'}, {',', 'c', '\347'}, > + {'\'', 'C', '\307'}, {'\'', 'c', '\347'}, > {'`', 'E', '\310'}, {'`', 'e', '\350'}, > {'\'', 'E', '\311'}, {'\'', 'e', '\351'}, > {'^', 'E', '\312'}, {'^', 'e', '\352'}, > @@ -259,4 +260,4 @@ > {'s', 'z', '\337'}, {'i', 'j', '\377'}, > }; > > -unsigned int accent_table_size = 68; > +unsigned int accent_table_size = 70; > > Please undo this patch. > It is wrong for at least three reasons. > > First of all, it is superfluous. If someone needs different > key assignments he should use loadkeys, not change the kernel. > Secondly, this file should correspond to defkeymap.map, as the > first two lines say: > /* Do not edit this file! It was automatically generated by */ > /* loadkeys --mktable defkeymap.map > defkeymap.c */ > Thirdly, c-cedilla is not the same symbol as c-acute. > The former is used in French and has Unicode value U+00c7 and U+00e7 > (capital and lower case), the latter is used in Polish and has > Unicode value U+0106 and U+0107. It is very strange to have > acute accent combined with C produce a symbol different from C-acute. > > Andries
They even look different if one looks closely.
--Marcin
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/
| |