Говнокод #16251 — Си — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

Си / Говнокод #16251

+136

switch(event->touch_point) {
      case 5:
         input_report_abs(data->input_dev, ABS_MT_TOUCH_MAJOR, event->pressure);
         input_report_abs(data->input_dev, ABS_MT_POSITION_X, event->x5);
         input_report_abs(data->input_dev, ABS_MT_POSITION_Y, event->y5);
         input_report_abs(data->input_dev, ABS_MT_WIDTH_MAJOR, 1);
         input_mt_sync(data->input_dev);
//         printk("===x2 = %d,y2 = %d ====\n",event->x2,event->y2);
      case 4:
         input_report_abs(data->input_dev, ABS_MT_TOUCH_MAJOR, event->pressure);
         input_report_abs(data->input_dev, ABS_MT_POSITION_X, event->x4);
         input_report_abs(data->input_dev, ABS_MT_POSITION_Y, event->y4);
         input_report_abs(data->input_dev, ABS_MT_WIDTH_MAJOR, 1);
         input_mt_sync(data->input_dev);
//         printk("===x2 = %d,y2 = %d ====\n",event->x2,event->y2);
      case 3:
         input_report_abs(data->input_dev, ABS_MT_TOUCH_MAJOR, event->pressure);
         input_report_abs(data->input_dev, ABS_MT_POSITION_X, event->x3);
         input_report_abs(data->input_dev, ABS_MT_POSITION_Y, event->y3);
         input_report_abs(data->input_dev, ABS_MT_WIDTH_MAJOR, 1);
         input_mt_sync(data->input_dev);
//         printk("===x2 = %d,y2 = %d ====\n",event->x2,event->y2);
      case 2:
         input_report_abs(data->input_dev, ABS_MT_TOUCH_MAJOR, event->pressure);
         input_report_abs(data->input_dev, ABS_MT_POSITION_X, event->x2);
         input_report_abs(data->input_dev, ABS_MT_POSITION_Y, event->y2);
         input_report_abs(data->input_dev, ABS_MT_WIDTH_MAJOR, 1);
         input_mt_sync(data->input_dev);
//         printk("===x2 = %d,y2 = %d ====\n",event->x2,event->y2);
      case 1:
         input_report_abs(data->input_dev, ABS_MT_TOUCH_MAJOR, event->pressure);
         input_report_abs(data->input_dev, ABS_MT_POSITION_X, event->x1);
         input_report_abs(data->input_dev, ABS_MT_POSITION_Y, event->y1);
         input_report_abs(data->input_dev, ABS_MT_WIDTH_MAJOR, 1);
         input_mt_sync(data->input_dev);
         printk("===x1 = %d,y1 = %d ====\n",event->x1,event->y1);

      default:
//         printk("==touch_point default =\n");
         break;
   }

Автору платили за строки?
Из драйвера тачскрина ft5x0x

Запостил:

Pythoner, 28 Июня 2014

Комментарии (28) RSS

Qwertiy 28.06.2014 21:27 # 0

Хм.. Без break'ов и с каким-то printk. И что же оно делает?
Ответить
- guest 28.06.2014 21:47 # 0
  
  хуйню.
  Ответить
- bormand 28.06.2014 22:06 # +4
  
  > printk
  Ядрёный логгер в линухе.
  
  > что же оно делает
  Упихивает координаты касаний в какой-то массив ивентов, вестимо. Т.е. если 5 точек касания - то упихает туда 5 наборов координат, если 4 - то 4. Поэтому бряков нет. Ну это так, предположение.
  Ответить
- Xom94ok 28.06.2014 22:07 # +2
  
  > с каким-то printk
  printk аналогичен printf, только пишет в системный лог, а не на stdout log level проебали...
  > что же оно делает?
  делает вид, что умеет мультитач в пять точек
  
  алсо, копипаст детектед
  Ответить
  - bormand 28.06.2014 22:10 # +1
    
    > делает вид, что умеет мультитач
    Судя по ABS_MT_WIDTH_MAJOR = 1 именно что делает вид. Хуй руку от пальца не отличит ;(
    
    Хотя тачскрину это не особо и надо, в отличие от тачпада...
    Ответить
    - Pythoner 28.06.2014 22:33 # 0
      
      Тач используется на дешевых китайцах, искал сорцы драйвера в связи глюками тачскрина - сам нажимает, а откалибровать хуй. Ах, да, к чему я это... На самом деле тач поддерживает только 2 касания, так что хз, или драйвер делает вид, или в ядре где то режутся касания с ид больше 2.
      Ответить
      - bormand 28.06.2014 22:43 # +1
        
        > или в ядре где то режутся касания с ид больше 2
        Да поди сам контроллер ёмкостного тача больше джвух точек не распознает и не отправляет. А драйвер, возможно, рассчитан на всю линейку - и где 5 распознаются, и где всего 2. Поэтому и switch на 5 точек.
        Ответить
        
        Pythoner 28.06.2014 22:54 # 0
        
        А еще он нераспознает силы нажатия - поле pressure в структуре event одно на все касания, и захардкожено = 200. Так что хз что делать - думал сортировать касания по силе нажатия, а тут такая фигня:(
        Ответить
      - Xom94ok 28.06.2014 22:50 # 0
        
        > хз, или драйвер делает вид, или в ядре где то режутся касания с ид больше 2
        Судя по строкам 8, 15, 22 и 29, я бы в первую очередь грешил на драйвер :)
        Гуглеж дал ссылку, тут выглядит чуть поаккуратней:
        http://preview.tinyurl.com/ft5x0x-drv
        Ответить
brutushafens 28.06.2014 23:41 # −3

cleaned
Ответить
- bormand 29.06.2014 00:01 # +2
  
  > А разве драйвера не на асме штампуют?
  Боже упаси... Давным-давно уже не пишут ;) На асме нынче пишут только небольшие кусочки, которые компилятор либо не сможет сгенерить вообще, либо они получатся слишком медленными.
  Ответить
  - brutushafens 29.06.2014 00:02 # −3
    
    cleaned
    Ответить
    - bormand 29.06.2014 00:09 # +5
      
      > Мне нужны самые быстрые драйверы.
      А ты уверен, что ты сможешь писать на асме лучше нынешних компиляторов сишечки?
      Ответить
      - brutushafens 29.06.2014 00:14 # −2
        
        cleaned
        Ответить
        
        bormand 29.06.2014 00:15 # 0
        
        Не вели казнить, вели слово молвить!
        Ответить
        
        brutushafens 29.06.2014 00:16 # 0
        
        cleaned
        Ответить
        
        kegdan 29.06.2014 00:45 # −1
        
        Ты вызвал дух говнокода. Задавай 3 вопроса, смертный
        Ответить
        
        myaut 29.06.2014 10:06 # +1
        
        А чего не три желания?
        Ответить
        
        myaut 29.06.2014 10:06 # +3
        
        Чорт, я израсходовал вопрос, да?
        Ответить
        
        chtulhu 29.06.2014 11:05 # +2
        
        и второй тоже )
        Ответить
        
        kegdan 29.06.2014 12:35 # +2
        
        1. я что, долбанный джин по твоему?
        2. да
        остался один вопрос
        Ответить
    - chtulhu 29.06.2014 05:28 # +3
      
      Царь, ты? Сишка уже надоела?
      Ответить
      - brutushafens 29.06.2014 14:58 # −2
        
        cleaned
        Ответить
  - Webius 30.06.2014 18:00 # 0
    
    Я кстати зашол такое же написать) А вообще асм - тот же байт-код, только вместо цыфр в некоторых местах слова. И вообще, асм очень простой язык, по тому, что там нет абстракции разных, и такая тупая простость.
    Ответить
    - kegdan 30.06.2014 18:08 # 0
      
      >>А вообще асм - тот же байт-код, только вместо цыфр в некоторых местах слова.
      байткод может быть гораздо более высокоуровневым, чем асм.
      Ответить
TarasB 30.06.2014 18:16 # −1

Ручной разворот цикла с неизвестной длиной11111
Ответить
- bormand 30.06.2014 19:13 # 0
  
  Почему неизвестной? Длина до 5.
  Ответить
  - TarasB 30.06.2014 19:34 # +2
    
    Неизвестной, про которую известно лишь то, что она меньше 5.
    Ответить

Говнокод: по колено в коде.

Си / Говнокод #16251

Комментарии (28) RSS

Добавить комментарий