Добро пожаловать, Гость. Пожалуйста авторизуйтесь здесь.
FGHIGate на GaNJa NeTWoRK ST@Ti0N - Просмотр сообщения в эхоконференции RU.HUSKY
Введите FGHI ссылку:


Присутствуют сообщения из эхоконференции RU.HUSKY с датами от 16 Jul 13 10:00:06 до 09 Aug 24 22:04:26, всего сообщений: 5336
Ответить на сообщение К списку сообщений Предыдущее сообщение Следующее сообщение
= Сообщение: 828 из 5336 ========================================== RU.HUSKY =
От   : Nickita A Startcev               2:5030/777.319     01 Dec 14 16:04:56
Кому : Alexey Vissarionov                                  01 Dec 14 16:04:56
Тема : hptzip
FGHI : area://RU.HUSKY?msgid=2:5030/777.319+547c6915
На   : area://RU.HUSKY?msgid=2:5020/545+547b18a3
= Кодировка сообщения определена как: CP866 ==================================
==============================================================================
Привет, Alexey !


 30 Nov 14 , 16:16  Alexey Vissarionov писал к Alexander N. Skovpen:

AS>> и на всякий случай warnings:
AS>> clang -c -O3 -march=native -fomit-frame-pointer -fPIC -Ihptzip
AS>> -I/usr/local/include -Wall -DUNIX -DUNAME=\"LNX\"
AS>> src/hptzip.c src/hptzip.c:251:11: warning: unused variable
AS>> 'ratio' [-Wunused-variable]
AS>>     uLong ratio=0;
AS>>           ^
AS>> src/hptzip.c:408:11: warning: unused variable 'fout'
AS>> [-Wunused-variable]
AS>>     FILE* fout=NULL;
AS>>           ^
AS>> src/hptzip.c:441:9: warning: unused variable 'err'
AS>> [-Wunused-variable]
AS>>     int err = UNZ_OK;
AS>>         ^
AS>> 3 warnings generated.

AV> Со временем выкинем, а пока я их закрыл в #if 0

AS>> warning: implicit declaration of function
AS>> warning: incompatible integer to pointer conversion

AV> С этим надо разбираться отдельно, так как велик риск поломать сборку
AV> под какие-нибудь форточки или всякую экзотику.

AS>> warning: equality comparison with extraneous parentheses
AS>> [-Wparentheses-equality]

AV> Это нормально.

AS>> warning: incompatible pointer types assigning to
AS>> 'const unsigned long *' from 'const z_crc_t *' (aka 'const
AS>> unsigned int *')

AV> Вот это мне очень не нравится (вообще за нелокальные int полагается
AV> голову отрывать ввиду ненадобности), но сходу не нашел, а времени
AV> копать глубже на данный момент нет.

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

например, почти любую понтовую стрёмную конструкцию можно аккуратно расписать так, чтоб она стала понятна и компилятору и любому мейнтейнеру.

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

ps: вот у меня, типично, разработка выглядит примерно так: 1 день читать чужие исходники, продираясь через косноязычие. 1 час думать. 20 минут писать код. 10 минут проверять собираемость. на этом фоне протратить еще 5 минут на аккуратное переприведение типов - это мелочи, порядка 10 минут, которые потом наэкономят чуть ли не половину от дня ковыряний в чужом.

в общем, имеем тут типичнейшую дилемму заключенного в комплекте с трагедией общин.

.                                            С уважением, Hикита.
icq:240059686, lj-user:nicka_startcev
... Диплодоки и диплококки
--- GoldED+/LNX 1.1.5-
* Origin: invalid origin or bed-encoding (2:5030/777.319)

К главной странице гейта
Powered by NoSFeRaTU`s FGHIGate
Открытие страницы: 0.070353 секунды