[PATCH keyboard] Removing unused code

Alexandr Shadchin alexandr.shadchin at gmail.com
Tue Jan 4 12:03:30 PST 2011


On Wed, Dec 01, 2010 at 08:46:03AM +1000, Peter Hutterer wrote:
> On Tue, Nov 30, 2010 at 08:12:24PM +0500, Alexandr Shadchin wrote:
> > In OpenBSD removed support PCCONS in 2002 year
> > http://marc.info/?l=openbsd-cvs&m=102435816424294&w=2
> > 
> > Signed-off-by: Alexandr Shadchin <Alexandr.Shadchin at gmail.com>
> > ---
> 
> Acked-by: Peter Hutterer <peter.hutterer at who-t.net>
> 
> but I'd like to get Matthieu's ack on this as well before pushing.
> 
> Cheers,
>   Peter
> 

Matthieu ok.
Somebody push? Thanks in advance.

> 
> >  src/bsd_KbdMap.c |  198 ------------------------------------------------------
> >  1 files changed, 0 insertions(+), 198 deletions(-)
> > 
> > diff --git a/src/bsd_KbdMap.c b/src/bsd_KbdMap.c
> > index 6cfdbe4..1b5cc3e 100644
> > --- a/src/bsd_KbdMap.c
> > +++ b/src/bsd_KbdMap.c
> > @@ -189,98 +189,6 @@ static KeySym eascii_to_x[512] = {
> >  	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol
> >  };
> >  
> > -#ifdef __OpenBSD__
> > -/* don't mark AltR and  CtrlR for remapping, since they 
> > - * cannot be remapped by pccons */
> > -static unsigned char pccons_remap[128] = {
> > -     0, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07,   /* 0x00 - 0x07 */
> > -  0x08, 0x09, 0x0a, 0x0b, 0x0c, 0x0d, 0x0e, 0x0f,   /* 0x08 - 0x0f */
> > -  0x10, 0x11, 0x12, 0x13, 0x14, 0x15, 0x16, 0x17,   /* 0x10 - 0x17 */
> > -  0x18, 0x19, 0x1a, 0x1b, 0x1c, 0x1d, 0x1e, 0x1f,   /* 0x18 - 0x1f */
> > -  0x20, 0x21, 0x22, 0x23, 0x24, 0x25, 0x26, 0x27,   /* 0x20 - 0x27 */
> > -  0x28, 0x29, 0x2a, 0x2b, 0x2c, 0x2d, 0x2e, 0x2f,   /* 0x28 - 0x2f */
> > -  0x30, 0x31, 0x32, 0x33, 0x34, 0x35, 0x36,    0,   /* 0x30 - 0x37 */
> > -  0x38, 0x39, 0x3a, 0x3b, 0x3c, 0x3d, 0x3e, 0x3f,   /* 0x38 - 0x3f */
> > -  0x40, 0x41, 0x42, 0x43, 0x44, 0x45, 0x46,    0,   /* 0x40 - 0x47 */
> > -     0,    0,    0,    0,    0,    0,    0,    0,   /* 0x48 - 0x4f */
> > -     0,    0,    0,    0,    0,    0, 0x56, 0x57,   /* 0x50 - 0x57 */
> > -  0x58,    0,    0,    0,    0,    0,    0,    0,   /* 0x58 - 0x5f */
> > -     0,    0,    0,    0,    0,    0,    0,    0,   /* 0x60 - 0x67 */
> > -     0,    0,    0,    0,    0,    0,    0,    0,   /* 0x68 - 0x6f */
> > -     0,    0,    0,    0,    0,    0,    0,    0,   /* 0x70 - 0x77 */
> > -     0,    0,    0,    0,    0,    0,    0,    0,   /* 0x78 - 0x7f */
> > -};
> > -
> > -/* This table assumes an iso8859_1 encoding for the characters 
> > - * > 80, as returned by pccons */
> > -static KeySym latin1_to_x[256] = {
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	XK_BackSpace,	XK_Tab,		XK_Linefeed,	NoSymbol,
> > -	NoSymbol,	XK_Return,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	XK_Escape,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	XK_space,	XK_exclam,	XK_quotedbl,	XK_numbersign,
> > -	XK_dollar,	XK_percent,	XK_ampersand,	XK_apostrophe,
> > -	XK_parenleft,	XK_parenright,	XK_asterisk,	XK_plus,
> > -	XK_comma,	XK_minus,	XK_period,	XK_slash,
> > -	XK_0,		XK_1,		XK_2,		XK_3,
> > -	XK_4,		XK_5,		XK_6,		XK_7,
> > -	XK_8,		XK_9,		XK_colon,	XK_semicolon,
> > -	XK_less,	XK_equal,	XK_greater,	XK_question,
> > -	XK_at,		XK_A,		XK_B,		XK_C,
> > -	XK_D,		XK_E,		XK_F,		XK_G,
> > -	XK_H,		XK_I,		XK_J,		XK_K,
> > -	XK_L,		XK_M,		XK_N,		XK_O,
> > -	XK_P,		XK_Q,		XK_R,		XK_S,
> > -	XK_T,		XK_U,		XK_V,		XK_W,
> > -	XK_X,		XK_Y,		XK_Z,		XK_bracketleft,
> > -	XK_backslash,	XK_bracketright,XK_asciicircum,	XK_underscore,
> > -	XK_grave,	XK_a,		XK_b,		XK_c,
> > -	XK_d,		XK_e,		XK_f,		XK_g,
> > -	XK_h,		XK_i,		XK_j,		XK_k,
> > -	XK_l,		XK_m,		XK_n,		XK_o,
> > -	XK_p,		XK_q,		XK_r,		XK_s,
> > -	XK_t,		XK_u,		XK_v,		XK_w,
> > -	XK_x,		XK_y,		XK_z,		XK_braceleft,
> > -	XK_bar,		XK_braceright,	XK_asciitilde,	XK_Delete,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	NoSymbol,	NoSymbol,	NoSymbol,	NoSymbol,
> > -	XK_nobreakspace,XK_exclamdown,	XK_cent,	XK_sterling,
> > -	XK_currency,	XK_yen,		XK_brokenbar,	XK_section,
> > -	XK_diaeresis,	XK_copyright,	XK_ordfeminine,	XK_guillemotleft,
> > -	XK_notsign,	XK_hyphen,	XK_registered,	XK_macron,
> > -	XK_degree,	XK_plusminus,	XK_twosuperior,	XK_threesuperior,
> > -	XK_acute,	XK_mu,		XK_paragraph,	XK_periodcentered,
> > -	XK_cedilla,	XK_onesuperior,	XK_masculine,	XK_guillemotright,
> > -	XK_onequarter,	XK_onehalf,	XK_threequarters,XK_questiondown,
> > -	XK_Agrave,	XK_Aacute,	XK_Acircumflex,	XK_Atilde,
> > -	XK_Adiaeresis,	XK_Aring,	XK_AE,		XK_Ccedilla,
> > -	XK_Egrave,	XK_Eacute,	XK_Ecircumflex,	XK_Ediaeresis,
> > -	XK_Igrave,	XK_Iacute,	XK_Icircumflex,	XK_Idiaeresis,
> > -	XK_ETH,		XK_Ntilde,	XK_Ograve,	XK_Oacute,
> > -	XK_Ocircumflex,	XK_Otilde,	XK_Odiaeresis,	XK_multiply,
> > -	XK_Ooblique,	XK_Ugrave,	XK_Uacute,	XK_Ucircumflex,
> > -	XK_Udiaeresis,	XK_Yacute,	XK_THORN,	XK_ssharp,
> > -	XK_agrave,	XK_aacute,	XK_acircumflex,	XK_atilde,
> > -	XK_adiaeresis,	XK_aring,	XK_ae,	        XK_ccedilla,
> > -	XK_egrave,	XK_eacute,	XK_ecircumflex,	XK_ediaeresis,
> > -	XK_igrave,	XK_iacute,	XK_icircumflex, XK_idiaeresis,
> > -	XK_eth,		XK_ntilde,	XK_ograve, 	XK_oacute,
> > -	XK_ocircumflex,	XK_otilde,	XK_odiaeresis,	XK_division,
> > -	XK_oslash,	XK_ugrave,	XK_uacute,	XK_ucircumflex,
> > -	XK_udiaeresis,	XK_yacute,	XK_thorn, 	XK_ydiaeresis
> > -      };
> > -#endif
> > -
> >  #ifdef SYSCONS_SUPPORT
> >  static
> >  unsigned char sysconsCODEMap[] = {
> > @@ -838,112 +746,6 @@ KbdGetMapping (InputInfoPtr pInfo, KeySymsPtr pKeySyms, CARD8 *pModMap)
> >  #ifndef __bsdi__
> >    switch (pKbd->consType) {
> >  
> > -#ifdef PCCONS_SUPPORT
> > -  case PCCONS:
> > -#if defined(__OpenBSD__)
> > -    /*
> > -     * on OpenBSD, the pccons keymap is programmable, too
> > -     */
> > -    {
> > -      pccons_keymap_t keymap[KB_NUM_KEYS];
> > -      if (ioctl(pInfo->fd, CONSOLE_GET_KEYMAP, &keymap) != -1) {
> > -	for (i = 0; i < KB_NUM_KEYS; i++) 
> > -	  if (pccons_remap[i]) {
> > -	    k = map + (pccons_remap[i] << 2);
> > -	    switch (keymap[i].type) {
> > -	    case KB_ASCII:
> > -	      /* For ASCII keys, there is only one char in the keymap */
> > -	      k[0] = latin1_to_x[(unsigned char)keymap[i].unshift[0]];
> > -	      k[1] = latin1_to_x[(unsigned char)keymap[i].shift[0]];
> > -	      k[2] = latin1_to_x[(unsigned char)keymap[i].altgr[0]];
> > -	      k[3] = latin1_to_x[(unsigned char)keymap[i].shift_altgr[0]];
> > -	      break;
> > -	    case KB_SCROLL:
> > -	      k[0] = XK_Scroll_Lock;
> > -	      goto special;
> > -	    case KB_NUM:
> > -	      k[0] = XK_Num_Lock;
> > -	      goto special;
> > -	    case KB_CAPS:
> > -	      k[0] = XK_Caps_Lock;
> > -	      goto special;
> > -	    case KB_SHIFT:
> > -	      switch (keymap[i].unshift[0]) {
> > -	      case 1:
> > -		/* left shift */
> > -		k[0] = XK_Shift_L;
> > -		break;
> > -	      case 2:
> > -		/* right shift */
> > -		k[0] = XK_Shift_R;
> > -		break;
> > -	      default:
> > -		k[0] = NoSymbol;
> > -	      }
> > -	      goto special;
> > -	    case KB_CTL:
> > -	      k[0] = XK_Control_L;
> > -	      goto special;
> > -	    case KB_ALT:
> > -	      k[0] = XK_Alt_L;
> > -	      goto special;
> > -	    case KB_FUNC:
> > -	      switch (keymap[i].unshift[2]) {
> > -	      case 'M':
> > -		k[0] = XK_F1;
> > -		break;
> > -	      case 'N':
> > -		k[0] = XK_F2;
> > -		break;
> > -	      case 'O':
> > -		k[0] = XK_F3;
> > -		break;
> > -	      case 'P':
> > -		k[0] = XK_F4;
> > -		break;
> > -	      case 'Q':
> > -		k[0] = XK_F5;
> > -		break;
> > -	      case 'R':
> > -		k[0] = XK_F6;
> > -		break;
> > -	      case 'S':
> > -		k[0] = XK_F7;
> > -		break;
> > -	      case 'T':
> > -		k[0] = XK_F8;
> > -		break;
> > -	      case 'U':
> > -		k[0] = XK_F9;
> > -		break;
> > -	      case 'V':
> > -		k[0] = XK_F10;
> > -		break;
> > -	      case 'W':
> > -		k[0] = XK_F11;
> > -		break;
> > -	      case 'X':
> > -		k[0] = XK_F12;
> > -		break;
> > -	      default:
> > -		k[0] = NoSymbol;
> > -		break;
> > -	      }
> > -	      goto special;
> > -	    default:
> > -	      k[0] = NoSymbol;
> > -	    special:
> > -	      k[1] = k[2] = k[3] = NoSymbol;
> > -	    }
> > -	  }
> > -      } else {
> > -	ErrorF("Can't read pccons keymap\n");
> > -      }
> > -    }
> > -#endif /* __OpenBSD__ */
> > -  break;
> > -#endif
> > -
> >  /*
> >   * XXX wscons has no GIO_KEYMAP
> >   */
> > -- 
> > 1.7.3.2

-- 
Alexandr Shadchin



More information about the xorg-devel mailing list