1. Си / Говнокод #2128

    +139.3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (errno==EINTR) while (errno==EINTR)
     {
            fseek(fr,loffset,SEEK_SET);
            fgets(line_buffer,len,fr);
}

    sacc - система биллинга для squid

    Одной проверки на ошибку - видимо, недостаточно

    Запостил: fiss, 11 Ноября 2009

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

    • ava guest 11.11.2009 17:22 # −5

      показать все, что скрытоРазницу между if и while понимаем?
      Ответить

    • ava xaionaro 11.11.2009 17:47 # 0

      Школьный недочёт... неинтересно
      Ответить

      • ava xaionaro 12.11.2009 07:49 # 0

        Кстати говоря gcc4 с флагом -O2 вообще игнорирует данный if, т.ч. итоговый код всё-равно нормальный))
        Ответить

        • ava bildja 13.11.2009 02:16 # 0

          А, ну, тогда, просто, скорее всего именно gcc4 как раз с флагом -O2 и компилировали
          Ответить

          • ava xaionaro 13.11.2009 09:37 # 0

            Почему это вдруг? Вообще-то gcc иногда вполне эффективно убирает код не несущий никакой смысловой нагрузки.

            Вообще вы меня удивляете, у gcc достаточно серьёзные алгоритмы оптимизации и было бы странно, если бы gcc не замечал двойной проверки на "(errno==EINTR)".
            Ответить

            • ava bildja 13.11.2009 12:41 # 0

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

    • ava xaionaro 13.11.2009 22:52 # 0

      Кстати, а там после этого куска кода нет "else" случайно? :)
      Ответить

    • ava guest 16.11.2009 23:16 # 0

      Да, я постер.

      Вообще то я не понимаю разговоров по поводу gcc.. да естественно компилятор не идиот - и оптимизирует как надо, вопрос в другом - как можно было такое вообще написать.. И так же понятно, что можно просто обойтись while в данном случае - первая проверка инварианта цикла даст требуемое поведение.
      Ответить

      • ava Pencioner 13.01.2010 13:45 # 0

        Таварисч. Читайте маны!
        errno у нас статическая глобальная переменная устанавливающаяся во время ошибки системных вызовов, и вызов fgets() внутри цикла каждый раз ее устанавливает заново. Смысл кода в том чтобы пытаться выполнить пока не пройдет без прерывание эта операция. Так как fgets() использует системные вызовы, если возниктет прерывание, мы получим errno==EINTR опять, и вынуждены будем повторить чтение с данного места файла.

        Я бы даже сказал что это код с повышенной надежностью... но чего-то слегка не хватает, наверное проверки результата fgets() на EOF...
        Ответить

        • ava Pencioner 13.01.2010 13:53 # 0

          ааааа
          сорьк
          фсе понял... посыпаю голову пеплом...

          я чет подумал что наезды по поводу того шо while лишний...
          а там if ненадо, ой ненадо

          аааааа! был невнимателен....
          Ответить

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