1. C++ / Говнокод #4507

    +190

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    ////// вот такое мы нашли после 3 месяцев отладки //////

long WINAPI ThreadFunc(long lParam);
{
	//Kill yourself!
	//good debug, Bitches! >=]
	while(1) free(rand());
	return 0;
}

///// и где-то в дебрях кода еще вот такое ////

CreateThread(NULL,0,(LPTHREAD_START_ROUTINE)ThreadFunc,NULL,0,&dwID[0]);

    В общем как то так =(

    Запостил: vii, 01 Ноября 2010

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

    • ava burdakovd 01.11.2010 19:40 # +11

      while(1) хавало бы весь CPU и тред был бы обнаружен чуть ранее чем сразу?

      Что вы делали три месяца?
      Ответить

      • ava rat4 01.11.2010 22:17 # +1

        "и где-то в дебрях кода"
        Иначе бы не "хавало весь CPU", а сразу падало
        Ответить

        • ava burdakovd 01.11.2010 22:21 # 0

          А если переданный в free адрес не был ранее выделен через malloc/calloc/realloc, то ничего не происходит или происходит ошибка?
          Ответить

      • ava absolut 01.11.2010 22:20 # 0

        Пиво они пили видимо все это время. В системе надо знать сколько тредов должно быть.
        Ответить

      • ava vii 02.11.2010 01:06 # −1

        В основном мы честно искали где освобождаются элементы одного большого связного списка... Оказалось что везде с ними все всегда нормально. Поток - один из пачки рабочих потоков короткоживущего компрессора изображений. По загрузке CPU на фоне еще 16 не особо заметно. Вообще, в остальном компрессор работает идеально и отношения к проблемному списку никакого не имеет.
        Ответить

        • ava absolut 02.11.2010 08:17 # +1

          бесконечный цикл без пауз и ожиданий чего-либо не особо заметен ? да что вы говорите
          Ответить

          • ava vii 02.11.2010 12:37 # −1

            рядом с 16 другими циклами - не заметно абсолютно.
            Ответить

            • ava absolut 02.11.2010 13:03 # +1

              ну, если они такие как этот - то не удивительно :)
              Ответить

            • ava bugmenot 02.11.2010 18:42 # +8 

              while(1) malloc( rand() );

              остальные треды такие?
              Ответить

          • ava Анонимус 02.11.2010 23:26 # 0

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

    • ava sera 01.11.2010 20:14 # +8

      даже инквизиция не позволяла себе такие муки)) но в самом деле, чем вы 3 месяца занимались? Оо
      Ответить

    • ava 3.14159265 01.11.2010 20:33 # +15

      > while(1) free(rand());
      fucking MASTERPIECE garbage collector i ever seen!!!
      Ответить

      • ava absolut 01.11.2010 22:45 # +5

        можно ещё более полный: 
        typedef unsigned long long ull;
for( ull i = 0; i<std::numeric_limits<ull>::max(); ++i )
{
    free( (void*)i );
}
        Ответить

        • ava Altravert 02.11.2010 08:54 # 0

          Ты жестокий.
          Ответить

        • ava nil 04.11.2010 16:10 # +2

          Собирает ВЕСЬ мусор.
          А что не мусор, то делает мусором и собирает.
          Ответить

          • ava bugmenot 04.11.2010 23:32 # +1

            Собранное сборщиком мусора по определению является мусором. Уловка 22 отдыхает. Слава роботам!
            Ответить

            • ava nil 04.11.2010 23:42 # 0

              Это если постулировать непогрешимость сборщика мусора.
              Гм, или попробовать апофатией?
              Приятно, что Уловку 22 помнят:) (Сдается мне, что лурк надо благодарить)
              Ответить

              • ava eth0 05.11.2010 11:29 # 0

                (Не обязательно /lm/. Полно и без него в сети источников того, что стоит почитать.)
                Ответить

              • ava bugmenot 05.11.2010 19:36 # 0

                в моем случае Геллер стоял на полке еще когда луркоморья в планах не было :)
                Ответить

                • ava nil 05.11.2010 19:49 # 0

                  Да что это я, чувствуется-чувствуется, коллега!
                  Ответить

                  • ava bugmenot 05.11.2010 21:23 # +2

                    в стране в всеобщей воинской повинностью - обязательное чтение, кстати =)
                    Ответить

    • ava rat4 02.11.2010 10:46 # +1

      Нормальный компилятор, кстати, не компилит это:
      error: invalid conversion from 'int' to 'void*'
      Ответить

      • ava absolut 02.11.2010 11:18 # +2

        а менее нормальный пишет :
        Warning! W113: Pointer type mismatch
        Ответить

        • ava Altravert 02.11.2010 11:22 # +6

          А еще менее нормальный программист отключит предупреждения.
          Ответить

        • ava istem 02.11.2010 11:51 # 0

          Вы случайно не этот компилятор используете?
          http://govnokod.ru/4502#comment50696
          Ответить

          • ava absolut 02.11.2010 11:54 # 0

            мы все его неявно используем, когда тут комментарии добавляем
            Ответить

      • ava Мистер Хэнки 02.11.2010 18:19 # +1

        а на
        <(long lParam);
        <{
        не ругается?
        Ответить

    • ava gegMOPO4 30.11.2010 16:44 # 0

      svn blame и паяльник в руки.
      Ответить

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