Так может патч выслать? Опенсорс же.

Чем больше я вижу открытого кода, тем больше недоумеваю, что за криворукие индусы его пишут.

Это ты ещё коммерческого кода не видел.

Человеку было надо - он написал как умел. Опенсорс не эльфы пишут.

Работает? Вот и славно. Не фапать же на код, правда?

Лорчую товарища лоровца.

Сначала сообщество кричит «пиши код», а потом «кто так пишет?». Будьте добрее.

Сделай патч и стань героем.

++

++

Язык поощряет такой быдлокод.

Двусмысленно звучит.

Два термоса чая этому гуманисту.

Покажи свой код.

Чем больше я вижу открытого кода, тем больше недоумеваю, что за криворукие индусы его пишут.

Суть открытого кода в том, что если сообщество посчитает нужным - то перепишет как правильно. Вот например ты используешь этот код - значит уже часть сообщества, знаешь как сделать код лучше - так возьми и вышли патч с исправлениями

Будьте добрее.

За добротой тебе на швабр. Там как раз любят добряков-криворучек. Прямо-таки эдакий хоспис для тех, кому «не дано».

Что же с тобой будет, когда ты закрытый код под NDA увидишь...там боль, слезы, и поплакаться на лоре в жилетку анону лицензия не позволяет.

Суть открытого кода в том, что если сообщество посчитает нужным - то перепишет как правильно.

Ты не понимаешь сути опенсорса. Суть открытого кода в том, что если тебе чего-то не хватает, берешь и костылишь, не дожидаясь, когда там автор снизойдет. Если не лень, оформляешь и засылаешь патчик, который (если не лень), автор может и принять.

К слову, очень многие популярные продукты с открытым кодом состоят из леденящего душу ужаса. Чтобы далеко не ходить, назовем ffmeg и mplayer своими именами. И до тех пор, пока «все работает», сообщество не особо и чешется.

когда ты закрытый код под NDA

Детка, я себе на бутерброд с икрой как раз закрытым кодом уже не первый десяток лет зарабатываю. И смею тебя уверить, что никаких принципиальных отличие от опенсорса там нет: если продукт пишут под «хотелки эффективных медежеров», да так, чтоб «побыстрее результат получить», будет такой же лютый высер, как и 95% опенсорца.

Но иногда звезды встают на небе таким образом, что попадается адекватный сеньор/тимлид/архитектор, которые не допускают скатывания продукта в леденящий душу ужас. Случается это почти так же «часто», как в опенсорсе.

Доброта есть внутри каждого из нас, и каждый из нас может нести ее в окружающий мир. Я буду просвещать ЛОР. Зачем идти туда, где и так все хорошо?

PS: Просто не Хабре быдло долго не живет, вот и результат.

Перепиши так, как считаешь нужным.

Детка, я себе на бутерброд с икрой как раз закрытым кодом уже не первый десяток лет зарабатываю.

Тогда чего ноешь, как маленькая девочка? Раз заявляешь что не ребенок - будь мужиком, нарисуй патч.

если продукт пишут под «хотелки эффективных медежеров», да так, чтоб «побыстрее результат получить»

Только так сейчас и делают, к сожалению, ибо либо time2market, либо все полимеры уйдут конкурентам. А потом мы разгребаем потом SDK очередного чипа с матами.

Но иногда звезды встают на небе таким образом, что попадается адекватный сеньор/тимлид/архитектор, которые не допускают скатывания продукта в леденящий душу ужас. Случается это почти так же «часто», как в опенсорсе.

Адекватный подход и хороший код еще не значит, что все будет красиво и хорошо. Больше половины случаев адекватного подхода гробит оверинжениринг, в котором иногда сложнее разобраться, чем в спагетти студенческого качества.

Язык поощряет такой быдлокод.

Собака делает „гав”, кошка делает „мяу”, корова делает „му”, депутаты не делают нихрена, говно делают все.

Да. Мне тоже пришлось как-то заглянуть в xprop и еще какую-то утилиту... Больше никаких иксов, пишу с тостера.

Ну молодец, ты познал суть опенсорца. Патчик-то высылать будешь, или ты тут так, набросить? =)

А как это еще переписать? Условия не взаимоисключающие.

Что не так? Надо было гору фабрик, визиторов и прочих паттернов нагородить?

А как это еще переписать?

Банально «заменить копипасту на макрос»:

#define grab(ledmask) \
  XGrabButton (dpy, button, modifier | ledmask, (win ? win : DefaultRootWindow (dpy)),
	       False, ButtonPressMask | ButtonReleaseMask,
	       GrabModeAsync, GrabModeAsync, None, None)
#define grab_if(cond, ledmask) if (cond) do_grab(ledmask)
#define grab_if1(mod) grab_if(mod, mod)
#define grab_if2(mod1, mod2) grab_if(mod1 && mod2, mod1 | mod2)
#define grab_if3(mod1, mod2, mod3) grab_if(mod1 && mod2 && mod3, mod1 | mod2 | mod3)

grab(0);
grab_if1(numlock_mask); grab_if1(capslock_mask); grab_if1(scrolllock_mask);
grab_if2(numlock_mask, capslock_mask); grab_if2(numlock_mask, scrolllock_mask); grab_if2(capslock_mask, scrolllock_mask);
grab_if3(numlock_mask, capslock_mask, scrolllock_mask);

Портянка сразу становится намного короче и читабельней.

#define grab_if(cond, ledmask) if (cond) do_grab(ledmask)

do_grab это откуда?

grab_if1
grab_if2
grab_if3

Это всё как-то криво, даже для меня.
Лучше хотя-бы так:

if (numlock_mask || capslock_mask || scrolllock_mask) {
   XGrabButton (dpy, button,
		(modifier | 
	        (numlock_mask ? numlock_mask : 0) |
	        (capslock_mask ? capslock_mask : 0) |
	        (scrolllock_mask ? scrolllock_mask : 0)),
		(win ? win : DefaultRootWindow (dpy)), False,
		ButtonPressMask | ButtonReleaseMask, GrabModeAsync,
		GrabModeAsync, None, None);
   }

modifier | (numlock_mask ? numlock_mask : 0)

numlock_mask — это какой-нибудь int ? Тогда зачем тернарный оператор? «|» и так ничего не сделает, если numlock_mask == 0.

Тогда зачем тернарный оператор?

Для сохранения поведения кода.

«|» и так ничего не сделает, если numlock_mask

Ну, да, логично. Тогда вообще всё просто:

if (numlock_mask || capslock_mask || scrolllock_mask) {
   XGrabButton (dpy, button,
		modifier | numlock_mask | capslock_mask | scrolllock_mask,	        
		(win ? win : DefaultRootWindow (dpy)), False,
		ButtonPressMask | ButtonReleaseMask, GrabModeAsync,
		GrabModeAsync, None, None);
   }

Да ладно тебе. Зато с таким кодом сразу видно, где отвечающий за определённую лампочку код находится. И вообще у тебя есть возможность предложить свой вариант покороче.

Я даже больше скажу - большая часть софта в принципе состоит из леденящего душу кода. И немалая часть держится на леденящей душу архитектуре. Увы, но не всегда есть время, чтобы сделать «правильно».

О читабельности твоего варианта поспорить вполне можно.

А закрытый код ты просто не видишь.

Нынче модно быть озлобленным говном и хаять Хабр, на котором есть специалисты покруче тебя, но якобы атмосфера не понравилась?

Нынче модно быть озлобленным говном и хаять Хабр, на котором есть специалисты покруче тебя, но якобы атмосфера не понравилась?

Заходя на любой сайт в интернете, можно предполагать что среди его аудитории есть специалисты (в какой либо области) круче чем ты. Разве это запрещает составлять собственное мнение об этом сайте?

Никто ничего не запрещает.

Никто ничего не запрещает.

Почему же тогда

но якобы атмосфера не понравилась?

якобы

Выглядит так как будто ты не допускаешь, что дела на хабре идут именно так как описано.

Один парень рассказывал, что у них в конторе одного только GUI на яве 600МБ в исходниках. Я так и не понял, хвастался он, или жаловался.

Были товарищи, которые гордились запросом (я не шучу) в 15 тыс. строк. И те, которые хвастались тем, что пишут в день не меньше тыщи строк.

Карма, правила, счас даже вечеринки без модераторов не обходятся. Ну их.

Тваюмат! Это шо за леденящий душу пздц? И этот человек тут рассказывает за быдлокод?

Ниже мастер_костылей выкатил достаточно годный вариант, да. А у тебя тот же пздц что и был, но в другой обертке.

Чем больше я вижу открытого кода, тем больше недоумеваю, что за криворукие индусы его пишут.

Эти хотя бы сами пишут. А другие индусы потом копипастят этот код к себе в энтерпрайз.

Нынче модно быть озлобленным говном и хаять Хабр, на котором есть специалисты покруче тебя, но якобы атмосфера не понравилась?

Специалисты на швабре покруче меня? Ну это если только «по программированию на HTML5 и CSS3». В среднем уровень там весьма днищный. А «покруче меня» даже в России наберется не так уж много людей.

Кстати, совсем забыл

Перед этим леденящим душу кодом идет другой. Как говорится, «найди 5 отличий»:

static void
my_grab_key (Display * dpy, KeyCode keycode, unsigned int modifier,
	     Window win)
{
  modifier &= ~(numlock_mask | capslock_mask | scrolllock_mask);


  XGrabKey (dpy, keycode, modifier, (win ? win : DefaultRootWindow (dpy)),
	    False, GrabModeAsync, GrabModeAsync);

  if (modifier == AnyModifier)
    return;

  if (numlock_mask)
    XGrabKey (dpy, keycode, modifier | numlock_mask,
	      (win ? win : DefaultRootWindow (dpy)),
	      False, GrabModeAsync, GrabModeAsync);

  if (capslock_mask)
    XGrabKey (dpy, keycode, modifier | capslock_mask,
	      (win ? win : DefaultRootWindow (dpy)),
	      False, GrabModeAsync, GrabModeAsync);

  if (scrolllock_mask)
    XGrabKey (dpy, keycode, modifier | scrolllock_mask,
	      (win ? win : DefaultRootWindow (dpy)),
	      False, GrabModeAsync, GrabModeAsync);

  if (numlock_mask && capslock_mask)
    XGrabKey (dpy, keycode, modifier | numlock_mask | capslock_mask,
	      (win ? win : DefaultRootWindow (dpy)),
	      False, GrabModeAsync, GrabModeAsync);

  if (numlock_mask && scrolllock_mask)
    XGrabKey (dpy, keycode, modifier | numlock_mask | scrolllock_mask,
	      (win ? win : DefaultRootWindow (dpy)),
	      False, GrabModeAsync, GrabModeAsync);

  if (capslock_mask && scrolllock_mask)
    XGrabKey (dpy, keycode, modifier | capslock_mask | scrolllock_mask,
	      (win ? win : DefaultRootWindow (dpy)),
	      False, GrabModeAsync, GrabModeAsync);

  if (numlock_mask && capslock_mask && scrolllock_mask)
    XGrabKey (dpy, keycode,
	      modifier | numlock_mask | capslock_mask | scrolllock_mask,
	      (win ? win : DefaultRootWindow (dpy)), False, GrabModeAsync,
	      GrabModeAsync);

}

Ну, да, логично. Тогда вообще всё просто:

Да, все просто: тебе следует уйти прочь из профессии, как говорил Артемий Татьянович. Видимо ты не осилил понять, что приведенная простынка должна добавить захват сочетаний клавиш при всевозможных комбинациях индикаторов {num,caps,scroll}lock.

Банально «заменить копипасту на макрос»:

А ты специалист. Мне тот «говнокод» куда больше по душе чем твой чудокод.

Критикуй быдлокод, предлагай спагетти из макросов.

Ты диверсант?

Когда-то я был молод, горяч и не гнушался использовать yp-подход к разработке, ведь повторяющихся фрагментов в коде всегда дохрена. Но с тех пор прошло много лет, я увидел, какая жопа наступает, когда yp размножает баги, которые потом правят в некоторых местах, а в других банально не видят/забывают/не понимают. С тех пор активно использую препроцессор для безупречной и безбажной копипасты.

Отлично, но при чем тут пример из начального поста?