где подсветка? как это разбирать возможно?

ты бы уж тогда просто описал регистры, их назначение и формат а не копипастил индуский говнокод

это какойто дравер сетевухи чтоль?

Что это?

Бэкслэши лишние сами поставились, при постинге наверно.

Программисты из Майкрософт никогда ничего не комментируют - и так все понятно.

Сначала распарсил заголовок как «Кусок проприетарного говна», потом перечитал и осознал, что, по сути, именно это там и написано.

Хотеть!

Очень хороший код.

Самая мякотка ниже, там где начинаются ethatp_ifc_Open и ethatp_ifc_Open1 и т.д.

Эмотред

#if 1 || defined(CONFIG_IPPLAYER) 

  if ( mac[0]==0x00 && mac[1]==0x00 && mac[2]==0x00 && mac[3]==0x00 && mac[4]==0x00 && mac[5]==0x00 ) { ret = 1; } 
  if ( mac[0]==0xFF && mac[1]==0xFF && mac[2]==0xFF && mac[3]==0xFF && mac[4]==0xFF && mac[5]==0xFF ) { ret = 1; } 
  if ( mac[0]!=0x00 ) { ret = 1; } 

Рыдаю.

*закрыл лицо руками и отвернулся*

А как надо?

А для Ъ?

для Ъ - ТС не распарсил названия функций типа ethatpifc_GetMACAddress и переменные ethatp_ifc_Main_ptr->DevName_str. По-моему, нормальный код.

Нормальный такой вендокод. Чего не нравится?

> По-моему, нормальный код.

Ууу, чувак... Всё же просмотри код, организацию функций - чисто для эстетического удовольствия.

> Нормальный такой вендокод.

Работает под линухом, в продакшене.

Точнее, уже не работает. Я его переписал и закоммитил.

Мдя... Даже я такие ужасы не пишу.

Это C, детка.

Новые шедевры на руби скоро будут?

Обычный такой код. Только с комментами тяжко и без подсветки читается не фонтан.

> Это C, детка.

Спасибо, Капитан! Не узнал Вас в гриме.

Самый обычный код. Видали и страшнее

WTF?

Ты чё, поцанчик, на Си наезжаешь?

Я не вглядывался, но кроме foo[1]=0x01; foo[2]=0x02; foo[3]=0xf3; ничего не нашел. Код как код

> Точнее, уже не работает. Я его переписал и закоммитил.

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

Рекомендую бегло просмотреть каждую функцию.

> Новые шедевры на руби скоро будут?

Ты что, думаешь, он машина - каждый день шедевры клепать? :)

<сделал недоумевающий вид>

Ииии? if'ы не нравятся? А вообще, я часто пишу write-only код чтобы работало, а потом привожу всё в порядок. Можете ругать

> Вот за переписывание работающего кода только потому, что он некрасиво написан, руки надо в первую очередь отрывать.

Ты не знаешь контекста, так что не умничай.

Этот код не только «работал», он отнимал массу времени, и не только моего, при попытке в очередной раз разобраться, что не так, в каждом новом проекте, перед каждым вторым релизом и пререлизом.

Я только фрагмент показал, кстати.

А нет?

> А вообще, я часто пишу write-only код чтобы работало, а потом привожу всё в порядок. Можете ругать

А этот код использовался и «разрабатывался» года 4, разными людьми, в разных департаментах, в разных странах :).

У меня смутные подозрения, что если тебе сказать: «Напиши мне за вечер код такой-то, работы на пару часов», то ты напишешь не лучше. Даже если времени будет больше, можешь сказать: «работает - не кантуй».

Уже ответил :).

Если бы. Машина так не сможет.

каждый день

Уже 10 дней прошло после 'sudo rm -rf /var/cache/apt/archives/*.deb'. Пора уже. :)

> Ииии? if'ы не нравятся?

Причем тут ифы? Не нравится то, как «организована» поддержка multiple interfaces (eth0 и eth1).

> Ты не знаешь контекста

Да, не знаю

так что не умничай

Да бог с тобой. Предполагаю просто, зная сколько существует велосипедистов, готовых за здорово живёшь потратить время на пустое переписывание кода, только чтобы он был красивее. Над одним только Linux kernel тысячи таких идеалистов работают, которые переписывают даже очень и очень читабельный код.

Этот код не только «работал», он отнимал массу времени, и не только моего, при попытке в очередной раз разобраться

Всё верно, в этом случае и я бы переписал.

Ну то есть, блин, ну никак нельзя добавить аргумент в функцию, через 30 секунд релиз, копипаст - единственное решение, да?

if (handle == ethatpifc_drv_handle) strcpy(if_req.ifr_name, ETHATPIFC_DEV_NAME); else if (handle == ethatpifc_drv_handle_1) strcpy(if_req.ifr_name, ETHATPIFC_DEV_NAME_1);

Я про это, если чё. Если чё не так, извиняй, я всяких там кнутов с пряниками не читал - не образованный

> Точнее, уже не работает. Я его переписал и закоммитил.
Ты пишешь проприетарный код?

gpl кодом семью не прокормишь

ааа, мать моя женщина, каким дебилом надо быть чтоб писать такой код. По моему это форсед мем какой-то.

static int ethatpifc_Open(void) 
static int ethatpifc_Open_1(void) 

и код одинаковый)))

static int                ether_fd = -1;
static struct sockaddr_ll ether_haddr;

static int                ether_fd_1 = -1;
static struct sockaddr_ll ether_haddr_1; 

Какой то эмбеддед плейер судя по названию.

Код действительно г***о. Сразу видно писал программист, а не Системный программист.

> Какой то эмбеддед плейер судя по названию.

Не-а. Софт для тестирования аппаратных интерфейсов. ipplayer - это название совсем другого модуля, из которого когда-то кто-то копипастил сюда какой-то код.

> Сразу видно писал программист, а не Системный программист.

+1000

Я так понимаю, господин ТС хотел удивить обычным, среднестатистическим сишным кодом?

Нормальный C код, чо.

Дима, ну вот нахрена Вы выложели драйвер eth в общий доступ? Вы бы еще RTL модель всей микросхемы тарболом на рапиду выложили. Завтра ждем вас с заявлением у директора.

А если серьезно, то бывает код и хуже. намного хуже.