#21 Fix warnings of incorrect abs()/fabsf() usage

[Alexey Dokuchaev]

In many places, int abs(int) is being called with float argument, while float fabsf(float) should be called instead. There's one case when double fabs(double) called with integer argument. Patch is attached.

[Kirk Barnes]

я сделал доступ к свн, можешь без патчей изменять

[Alexey Dokuchaev]

Вообще варнингов огромное количество. Даже на дефолтных настройках Clang их находит сейчас 245; есть включить -Wall -Wextra будет 13132 (sic!).

GCC 4.8 на дефолте находит 164, с -Wall -Wextra уже 3505.

Я попробую исправить самые очевидные и которые платформо-независимые, если быстро не надоест. ;-)

Странно, что студия их не показывает. В MSVC можно как-то поднять warning level?

[Kirk Barnes]

странно, я пишу на предпоследнем уровне варнингов, все чисто-чисто

07.08.2015, 19:11, "Alexey Dokuchaev" danfe@users.sf.net:

Вообще варнингов огромное количество. Даже на дефолтных настройках Clang их находит сейчас 245; есть включить -Wall -Wextra будет 13132 (sic!).

GCC 4.8 на дефолте находит 164, с -Wall -Wextra уже 3505.

Я попробую исправить самые очевидные и которые платформо-независимые, если быстро не надоест. ;-)

Странно, что студия их не показывает. В MSVC можно как-то поднять warning level?

---

[bugs:#21] Fix warnings of incorrect abs()/fabsf() usage

Status: open
Group: v1.0_(example)
Created: Fri Aug 07, 2015 04:02 PM UTC by Alexey Dokuchaev
Last Updated: Fri Aug 07, 2015 04:06 PM UTC
Owner: nobody
Attachments:

• abs-fabsf.patch (3.9 kB; text/x-diff)

In many places, int abs(int) is being called with float argument, while float fabsf(float) should be called instead. There's one case when double fabs(double) called with integer argument. Patch is attached.

---

Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/quake2xp/bugs/21/

To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

[Alexey Dokuchaev]

Ну, на самом деле Clang уж очень щепетилен в мелочах; например, ему не нравятся конструкции типа if (foo = bar) — он считает, что это потенциальная ошибка, и либо должно быть ==, либо заключайте выражение в скобки, и таких примеров можно много привести.

С одной стороны он прав, с другой стороны — работает, не трожь! (ц)

[Alexey Dokuchaev]

Спасибо, это действительно проще. Но нужно кое-что сперва обсудить: у тебя есть jabber? или прямой email хотя бы, чтобы тут не спамить в трекере? Напиши мне на email скрыт от спамеров.

[Alexey Dokuchaev]

Fixed in [r600].