The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



Вариант для распечатки  
Пред. тема | След. тема 
Форум Разговоры, обсуждение новостей
Режим отображения отдельной подветви беседы [ Отслеживать ]

Оглавление

undertaker 1.0 - анализатор корректности блоков Си-препроцес..., opennews (ok), 30-Янв-11, (0) [смотреть все]

Сообщения [Сортировка по времени | RSS]


25. "очередной детсад от павлина"  +/
Сообщение от Вова (?), 01-Фев-11, 15:56 
>[оверквотинг удален]
> @@ -314,7 +314,7 @@
>      char *worklist = NULL;
>      char *whitelist = NULL;
> -    int threads = 1;
> +    long int threads = 1;
>      std::list<std::string> models;
>      std::string main_model = "x86";
>      /* Default is dead/undead analysis */
>
> И ЭТИ ЛЮДИ ЗАНИМАЮТСЯ АНАЛИЗОМ КАЧЕСТВА КОДА ЯДРА ..... :)

этот threads задаёт количество процессов, на которые надо распараллелить анализ ядра.
Если кто-то задаст значение, которое приведёт к переполнению, это уже само по себе некорректно.
Вполне нормальный кусок кода/

Ответить | Правка | К родителю #3 | Наверх | Cообщить модератору

26. "очередной детсад от вантуса"  +/
Сообщение от pavlinux (ok), 01-Фев-11, 17:13 
>[оверквотинг удален]
>>      char *worklist = NULL;
>>      char *whitelist = NULL;
>> -    int threads = 1;
>> +    long int threads = 1;
>>      std::list<std::string> models;
>>      std::string main_model = "x86";
>>      /* Default is dead/undead analysis */
>>
>> И ЭТИ ЛЮДИ ЗАНИМАЮТСЯ АНАЛИЗОМ КАЧЕСТВА КОДА ЯДРА ..... :)
> этот threads задаёт количество процессов, на которые надо распараллелить анализ ядра.

Пох...ю что он задаёт.

> Если кто-то задаст значение, которое приведёт к переполнению, это уже само по

Понятия "кто-то" в Си нету. Кода статистического анализа и расчёта вероятности не вижу.

> себе некорректно. Вполне нормальный кусок кода/

Функция возвращает long, значит присваиваться этот результат должен переменной
такого же типа, и нипёт, что "когда-то никогда что-то как-то" не случиться.

Надо было юзать atoi()

Ответить | Правка | Наверх | Cообщить модератору

27. "очередной детсад от вантуса"  +/
Сообщение от Вова (?), 01-Фев-11, 17:28 
Нет, это очередная ошибка, излишний пуризм. Данная переменная будет иметь  значения от единицы (умолчательное значение) до нескольких десятков, проверка на ошибочную работу strtol - имеется, проверка на значение переменной - тоже имеется. Вывод - если кто-то задал количество процессов в 24145123525523323242342343422  - тот сам себе злобный буратино.
Займись чем-нибудь полезным?
Ответить | Правка | Наверх | Cообщить модератору

30. "очередной детсад от вантуса"  +/
Сообщение от pavlinux (ok), 01-Фев-11, 17:42 
> - если кто-то задал количество процессов в

Программы делаются в помощь людям, а не для развития геморроя.

Ответить | Правка | Наверх | Cообщить модератору

31. "ещё раз"  +/
Сообщение от Вова (?), 01-Фев-11, 17:58 
Сценарий геморроя в данном конкретном случае?
Предусмотрен опциональный параметр: количество параллельно работающих потоков, аналог make -jN; параметр читается, значение параметра после этого проверяется, есть умолчательное значение - 1.
Какой геморрой?
Ответить | Правка | Наверх | Cообщить модератору

33. "ещё раз"  +/
Сообщение от pavlinux (ok), 01-Фев-11, 18:29 
Даже не знаю как вам объяснить, зачем у самолётов запас прочности превышает
все возможные перегрузки в 2.5 раза, за идиота примите.


Ответить | Правка | Наверх | Cообщить модератору

34. "ещё раз"  +/
Сообщение от Вова (?), 01-Фев-11, 18:55 
> Даже не знаю как вам объяснить, зачем у самолётов запас прочности превышает
> все возможные перегрузки в 2.5 раза, за идиота примите.

код, который вы пытаетесь "исправить", корректно поддерживает задание максимум 2147483647 процессов, реально же будут испольвано максимум 20 процессов в случае обычной пользовательской системы, и, видимо, 32 000 процессов в случае использования 32000-ядерной системы. Запас прочности составляет от 100 миллионов до 100 тысяч раз.

Принимаю.

Ответить | Правка | Наверх | Cообщить модератору

35. "ещё раз"  +/
Сообщение от pavlinux (ok), 01-Фев-11, 21:00 
Вы случаем в разработке PHP не участвуете?


Ответить | Правка | Наверх | Cообщить модератору

36. "уж лучше петросянить"  +/
Сообщение от Вова (?), 02-Фев-11, 09:46 
жги. Сразу два варианта остри - вариант на "да, я принимаю участие" и вариант "нет, не принимаю".

почитаю, может и посмеюсь.

Ответить | Правка | Наверх | Cообщить модератору

37. "очередной детсад от вантуса"  +/
Сообщение от nuclightemail (ok), 06-Фев-11, 18:06 
> Нет, это очередная ошибка, излишний пуризм. Данная переменная будет иметь  значения
> от единицы (умолчательное значение) до нескольких десятков, проверка на ошибочную работу

Соль в том, что этот проект как раз посвящен проверке качества и пуризма. А получается сапожник без сапог, хочется сказать авторам, "Врачу, исцелися сам".

Ответить | Правка | К родителю #27 | Наверх | Cообщить модератору

Архив | Удалить

Рекомендовать для помещения в FAQ | Индекс форумов | Темы | Пред. тема | След. тема




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру