caxapa.ru :: Когда рука тянется к "define" напиши "inline" :) Замените макрос функцией (можно inline). Макросы очень небезопасный механизм. Эта ваша ";" в макросе может неслабо "икнуться".

Скрипач, пророк (10.09.2013 10:32, просмотров: 78) ответил Mazur на clr_dsp:

Когда рука тянется к "define" напиши "inline" :) Замените макрос функцией (можно inline). Макросы очень небезопасный механизм. Эта ваша ";" в макросе может неслабо "икнуться".

... но не любой ценой.

Ответить

- Сделал инлайн, получил что хотел и линейкой по рукам за это получать не буду. :) - Mazur(10.09.2013 11:46)
  - А с другой стороны. Почему за это ругаетесь, если скомпилировано и работает? Mazur(225 знак., 10.09.2013 11:49)
    - Повторяю в 100500 раз: надо не в дизассемблере ковыряться, а создавать работающую программу. Причём с вменяемым исходником. А эти ваши инлайны никакого отношения к созданию рабочей программы не имеют, то есть отвлекают от дела, причём самым SciFi(156 знак., 10.09.2013 12:37 - 12:40)
      - Дело-то не в inline, дело в самом программисте. Или нет? - Bill(10.09.2013 13:05)
        
        Ну так я пытаюсь быть вежливым, поэтому не перехожу на личности :-) - SciFi(10.09.2013 13:06)
- Да там этот макрос нафиг не нужен. Любой вменяемый человек знает, что такое "memset(buffer, ' ', sizeof(buffer));". SciFi(60 знак., 10.09.2013 10:51)
  - Указкой жестоко. Линейкой :) Скрипач(133 знак., 10.09.2013 10:58)
    - Если используется более чем в одном месте, то да, так как дублирование кода - это зло. Иначе - лишнее телодвижение. - SciFi(10.09.2013 11:03)
      - Ибо HAL. При проектировании "сверху - вниз" никто не опускается до подсчета вызовов :) - Скрипач(10.09.2013 11:12)
        
        Я за такое проектирование: придумать интерфейс модуля, оформить в виде красивого хедера. Но тут же совсем другая история: просто ловим блох, так что это ни о чём. - SciFi(10.09.2013 11:14)
      - Не всегда. Иногда код тот же самый, а объединять в функцию -- та самая преждевременная оптимизация, известно чем кончающаяся. Вообще такими штуками компилятор умеет заниматься. - fk0(10.09.2013 11:05)
        
        Я говорю про случаи, когда в одном экземпляре дублирующегося кода что-то исправил, а в другом - забыл. Поэтому и зло. - SciFi(10.09.2013 11:08)
        
        Бывает и обратное! В "оптимизированном" таким образом коде в общей для всех функции исправил... а оно не ко всем вариантам теперь подходит! - fk0(10.09.2013 11:19)
    - Да у него C вообще отобрать нужно. Слишком много вольности. В Паскале (с объектами) написал бы нормально, в ОО стиле, с методом clear и все дела. А C и C++ это два перегиба в разные стороны. В одном слишком мало нормальных средств и слишком много fk0(73 знак., 10.09.2013 11:03)
      - Я уже писал где-то, что его нужно научить использовать printf. Зачем вообще этот "драйвер"? - Vladimir Ljaschko(10.09.2013 11:08)
        
        Printf низзя, sprintf можно ;) - Гyдвин(10.09.2013 11:23)
        
        Это нам нельзя ;) - Vladimir Ljaschko(10.09.2013 11:33)
        
        Да без этого вашего жырного printf любой проект влезает в Atmega8. А с printf перестаёт влезать в AT91SAM7X256. - fk0(10.09.2013 11:20)
        
        "Parole, parole, parole..." ;) - Vladimir Ljaschko(10.09.2013 11:22)
        
        Вот-вот. Вроде бы уговорили его переходить на Си, а он упорно кодит на ассемблере, только сишными буковками. Контрреволюционер :-) - SciFi(10.09.2013 11:10)
  - Любой профессионал библиотечных функций не использует! - fk0(10.09.2013 10:53)

Средства и методы разработки