разгребая чужие баги (программистское)

[avva]

Нет никаких сил.

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

Comments

[nice-beaver.livejournal.com]

Ужас. Сочувствую. Нет хуже мучения программисту, чем исправлять чужой код.

[dinozavrik.livejournal.com]

Супер! Такая идея не должна пропасть втуне - надо бы полную переделку :)

[breqwas.livejournal.com]

Класс :)
А кто кого шпандырем-то? Авва или Авву? :)

[dimrub.livejournal.com]

Tell me about that...

[avva.livejournal.com]

Не меня, конечно ;)

[nice-beaver.livejournal.com]

Минусы работы в крупной фирме.

[avva.livejournal.com]

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

[breqwas.livejournal.com]

(как же мне повезло, что мой предшественник был явно квалифицированнее меня...)

[pargentum.livejournal.com]

Тогда уж что-то типа: "Хозяин выволок меня за волосья на двор и отчесал шпандырем за то, что я создал несколько потоков и возвращал из служебной функции указатель на статический буфер. А на неделе ...." и далее по тексту.

[dimrub.livejournal.com]

царат рабим - это, конечно, нахамат типшим, но все же... Вчера бился с кодом, в котором в одной структуре некий UINT_32 разделен следующим образом: первые 16 LSB - битмаск, вторые - enum, являющийся указателем типа для другого поля этой структуры. Enum со значениями подряд (FLAG1 = 1, FLAG2 = 2, FLAG3 = 3), но некий гений проверял его значение следующим образом:

if ((flags >> 16) & FLAG3) {...}

(Вместо if ((flags >> 16) == FLAGS3)

При этом, разумеется, при значении flags >> 16 == FLAG{1,2} он в тело условия тоже заходит.

:-)

[bezukh.livejournal.com]

...И внизу подпись:
dedushka@derevnya.com

[flaass.livejournal.com]

Тогда уж bobbruisk.ru

[arpad.livejournal.com]

готично function пиши_исчо ( bool аффтар_жжот );

[desman.livejournal.com]

И
день программиста (http://ru.wikipedia.org/wiki/%D0%94%D0%B5%D0%BD%D1%8C_%D0%BF%D1%80%D0%BE%D0%B3%D1%80%D0%B0%D0%BC%D0%BC%D0%B8%D1%81%D1%82%D0%B0) как раз :)

С праздником, да!

тело-то забыли

[nine_k]

{
return РЖУ_НИМАГУ;
}

Re: :-)

[dadcaptain.livejournal.com]

http://www.derevnya.com/

Welcome.

Professional computer IT support for small businesses.

Наверное дойдет ...

[arpad.livejournal.com]

Это декларация

[elinai.livejournal.com]

Не надо недооценивать добротный, качественный повод для справедливого негодования ;-)

[ex-nat007983.livejournal.com]

разулыбило :))7 баллофф!!! до слез!

[avva.livejournal.com]

Да, хорошо так ;)

[flaass.livejournal.com]

Или вот этот, тоже подходит.

[alienor-sm.livejournal.com]

Вам хорошо, у Вас хоть чувства юмора на это хватает, а я тут скоро плакать стану. :)(

[avva.livejournal.com]

Что делать? Уж лучше смеяться, чем плакать. Держитесь!

[oxfv.livejournal.com]

Это что, я вот на новой работе обвыкаюсь. Вчера стал С++ код писать, включил в него сишный хедер. А там структура объявлена, а в ней мембер по имени private. Да-да, так и называется. Типа, кажется, char*. Ну что поделаешь, крякнул, написал дефайн перед включением хедера, рою траншею дальше.

Nice view from the window comes for a price :)

разгребая свои

[(Anonymous)]


void* runThread( void* arg )
{
if( arg )

}


разгребая свои

[(Anonymous)]


void* runThread( void* arg )
{
if( arg )
{
SomeObject* obj = (SomeObject*)obj;
obj->run();
}
}


Два часа, блин, два часа. [Гордо] Но ведь нашел!

[(Anonymous)]

А может он это написал чтоб не писать

if (((flags >> 16) == FLAGS1) || ((flags >> 16) == FLAGS2)) ?

[(Anonymous)]

Хотя все-равно ввалится по любому значению ;-) кроме == 0 и >3 ;-)

Re: разгребая свои

[lionofsion.livejournal.com]

Такое можно избежать вообще с помощью компилятора.
Например ставишь treat warning as errors. Получаешь:
"warning C4700: local variable 'obj' used without having been initialized"
Программа не собирается вообще. Зачем создавать себе проблемы? :)

P.S.: в C++ вместо C-style cast желательно использовать стандартные плюсовые, в твоем случаe:

SomeObject* obj = reinterpret_cast

[Error: Irreparable invalid markup ('<someobject *>') in entry. Owner must fix manually. Raw contents below.]

Такое можно избежать вообще с помощью компилятора.
Например ставишь treat warning as errors. Получаешь:
"warning C4700: local variable 'obj' used without having been initialized"
Программа не собирается вообще. Зачем создавать себе проблемы? :)

P.S.: в C++ вместо C-style cast желательно использовать стандартные плюсовые, в твоем случаe:

SomeObject* obj = reinterpret_cast<SomeObject *>(arg);

[father-hersgory.livejournal.com]

http://www.bobruisk.by

[ex-croco667.livejournal.com]

А я вот люблю по граблям ходить. Что в C++, что в plain C. Не могу себя отучить забывать, что default - это ключевое слово. И каждый раз на диагностику - как баран на новые ворота, секунд пятнадцать вспоминаю, что опять не то сделал.

Гадоль!!

[zyama-krendel.livejournal.com]

Замечательно!! :))

[(Anonymous)]

Представьте: все то же самое вот в таком (http://www.venge.net/graydon/talks/mkc/html/index.html)
контексте.

Подсветку синтаксиса отменили?

[buldozr.livejournal.com]

А код чем редактируете, классическим vi? ;)

Re: Подсветку синтаксиса отменили?

[ex-croco667.livejournal.com]

Подсветку я себе сам отменил. Глаза у меня отказываются работать с цветовыми схемами, отличными от "серым по черному", а в этой схеме синтаксхайлайтинг - это уже просто eye killer.

А редактирую в vim'е. ":synt off" стекает с пальцев при виде разноцветных букв чисто рефлекторно ;-)