- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
if ( cg_g2MarksAllModels == NULL )
{
cg_g2MarksAllModels = Cvar_Get( "cg_g2MarksAllModels", "0", 0 );
}
if (cg_g2MarksAllModels == NULL
|| !cg_g2MarksAllModels->integer )
{
firstModelOnly = qtrue;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+134
if ( cg_g2MarksAllModels == NULL )
{
cg_g2MarksAllModels = Cvar_Get( "cg_g2MarksAllModels", "0", 0 );
}
if (cg_g2MarksAllModels == NULL
|| !cg_g2MarksAllModels->integer )
{
firstModelOnly = qtrue;
}
Cvar_Get возвращает ненулевой указатель в любом случае.
wvxvw 31.08.2014 17:18 # +1
:*
gost 31.08.2014 17:19 # 0
Elvenfighter 31.08.2014 18:35 # +1
absolut 31.08.2014 21:26 # +4
tirinox 31.08.2014 19:06 # 0
gost 31.08.2014 20:46 # 0
absolut 31.08.2014 21:25 # +2
Вот если бы оно возвращало ссылку (то это был бы уже не си), то можно было бы и не проверять на нулл.
А так - уж лучше перепроверить, имхо, чтоб не свалиться с сигсегвом после очередного апгрейда.