взять и уебать

[mbr]

P4OUT &= ~R_B+~R_G; // сброс триггеров

Это из профессиональной железки, прошедшей сертификацию. Почему у нас падают спутники, я не удивляюсь.

Comments

[qehgt]

И что именно здесь такого страшного?

PS: здесь может быть несколько различных проблем, и даже не быть ни одной.

[mbr]

Понятно, в Таганрогском радиотехническом университете тоже не учат работать с битами.

[qehgt]

Ещё раз: можно написать корректный код, в котором данная строка будет иметь абсолютно defined behavior. И можно написать код, в котором данная строка будет иметь _разные_ проблемы (навскидку - неправильно разворачиваемый макрос, переопределённые операторы + и ~, неявные преобразования между signed/unsigned, неявные преобразования в int из более коротких целочисленных типов).

Поэтому спрошу ещё раз: что именно в этой строке какого WTF?

[display_none]

Ну Вы же не думаете правда что тут макрос и, например, R_B разворачивается огромное выражение, или например что это С++ и оператор + переопределен?

[sontar]

лол

[sontar]

а что за железка и сколько стоит?

[mbr]

Измерительный прибор, больше сказать не могу.

[display_none]

Верно-ли я понимаю что товарищ пытается обнулить в P4OUT биты, поднятые в R_B и R_G?
Ну тоесть P4OUT & (~R_B) & (~R_G) ?

[mbr]

Да.

Лулз в том, что биты этого порта управляют силовой периферией. К счастью, гасится лишь динамик.

[display_none]

А что, прёмочных тестов там не было?)

Все же мне странно предположить что человек не понимал двоинчную арифметику до степени 'совсем'.
Как же он остальное написал?

[mbr]

Там дальше был просто ад и израиль. Использовался soft float. При этом промежуточные константы были в интах, поэтому поплыли результаты. В итоге вполне годную в теории математику подгоняли мейджиками тупо по диапазону значений.

Дальше веселее - из-за soft float, вероятно, заканчивалась память. И автор перешел на ассемблер вообще без понятия о процедурах - одна точка входа и порядка 50 кб кода с метками исключительно вида "jjjj1a". Думаю, проще в бинарном коде разобраться, чем в этом высере. При этом с его талантами мне крайне сомнительно, что он смог родить код эффективнее компилятора. Зато читабельность упала до нуля.

И да, _все_ переменные - исключительно глобальные. Ну, ладно, допустим, по каким-то причинам автор не захотел использовать стек. Но можно ведь было хотя-бы их огородить между модулями статическим определением.

В общем, обычный продакшн код среднестатистической железки российского производства.

[vitamin_caig]

Отлить в бронзе табличку с теоремой Де Моргана и пиздить автора по башке до просветления:)