Войти
ФлеймФорумПрограммирование

Летопись багов (11 стр)

Страницы: 110 11 12 13 14 Следующая »
#150
18:39, 20 авг. 2018

Все проще, вышел за пределы диапазона. А после массива там другие данные, которые затёрлись и я получил сегфолт в совершенно другом месте.

Добавил проверку, не достиг ли count максимально возможного значения.


#151
21:24, 26 сен. 2018
unsigned char* rs_mem_alloc(uint32_t bytes, int pool_index) {
    rs_mem_alloc_adv(bytes, "", 0, pool_index);
};

Варнинги, что ли, почитать...

#152
2:01, 27 сен. 2018

Роман Шувалов
> Варнинги, что ли, почитать...
обычно это считается ошибкой компиляции

#153
2:22, 27 сен. 2018

Нечтение варнингов?

#154
2:26, 27 сен. 2018

Роман Шувалов
-Wall -Wextra -Werror

#155
10:30, 27 сен. 2018

kipar
> -Werror
Там куча ошибок вроде "неиспользуемый параметр". По -Wall -Wextra уже сам прошелся.

#156
11:05, 27 сен. 2018

Роман Шувалов
как минимум надо:
-Werror=return-local-addr -Werror=return-type

#157
12:06, 27 сен. 2018

Роман Шувалов
Эти неиспользуемые параметры несколько раз мне баги ловили. Так что теперь для стороннего кода я использую -Wall-Werror, а в своём просто UNUSED(param1); UNUSED(param2); там где они и правда не используются (ну это макрос для "(void)param1", кажется мне нагляднее).

#158
14:08, 27 сен. 2018

https://en.cppreference.com/w/cpp/language/attributes/nodiscard

#159
14:55, 27 сен. 2018

kipar
> а в своём просто UNUSED(param1); UNUSED(param2);
Тоже так сделал, но таких штук оказалось много, надо потратить время чтобы везде исправить.

Delfigamer
> nodiscard
Си без плюсов, я даже получаю

||warning: command line option ‘-std=gnu99’ is valid for C/ObjC but not for C++|
#160
21:14, 26 ноя. 2018

Только что словил:

struct sidata
{   // Входные:
    float price_percent;
    wstring dirnamefile;
    wstring    namefile;
    wstring dirout;
    // Выходные:
    int amfileout;
}idata = { 0.0f, L"", L"", 0 };

сrash app,
целый час убил, можно сказать, на ровном месте.

#161
22:48, 26 ноя. 2018

Клапауций
> целый час убил
могло быть и больше))

#162
23:00, 26 ноя. 2018

Клапауций
> Только что словил:
-Werror=missing-field-initializers

#163
(Правка: 11:30) 11:23, 27 ноя. 2018

/A\
> -Werror=missing-field-initializers
vc++2105(upd3)

PVS-Studio тож не ловит.

ага PVS выкинуть,
поставить gcc со всеми ключами в качестве статанализатора))
(а проектик то vc++\clr)

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

и нашел я случайно, но надо быть весьма не ленивым, чтобы для этого сделать кучу телодвижений :/

#164
11:24, 27 ноя. 2018

Клапауций
> PVS-Studio тож не ловит.
Ну это прям не фейл, а фейлище PVS-Studio. Напиши разрабам.

Страницы: 110 11 12 13 14 Следующая »
ФлеймФорумПрограммирование