https://smaugmuds.afkmods.com/index.php?a=rssfeed&t=4901 The largest Smaug community resource site. - en SmaugMuds 60 https://smaugmuds.afkmods.com/topic/compiler-warning-for-decorateroom-4901/&p=23921#p23921 Thanks, that's got the compiler happy now. https://smaugmuds.afkmods.com/topic/compiler-warning-for-decorateroom-4901/&p=23921#p23921 Thu, 12 Mar 2020 15:03:00 CDT SmaugFUSS nobody@example.com (Samson) https://smaugmuds.afkmods.com/topic/compiler-warning-for-decorateroom-4901/&p=23920#p23920 It's still ugly, but I think it will do the same without the compiler complaining switch ( number_range( 1, ( ( 2 * ( iRand == nRand - 1 ) ) > 0 ) ? 1 : 2 ) ) https://smaugmuds.afkmods.com/topic/compiler-warning-for-decorateroom-4901/&p=23920#p23920 Wed, 11 Mar 2020 16:40:15 CDT SmaugFUSS nobody@example.com (Leia) https://smaugmuds.afkmods.com/topic/compiler-warning-for-decorateroom-4901/&p=23919#p23919 So I got bored and decided to do some cleanup on GCC 7.5 since it's been awhile since the last time that was done. There's one issue left to clear up but my C/C++ is too rusty these days. [code]act_move.c: In function ‘void decorate_room(ROOM_INDEX_DATA*)’: act_move.c:257:43: warning: ‘*’ in boolean context, suggest ‘&&’ instead [-Wint-in-bool-context] switch ( number_range( 1, ( 2 * ( iRand == nRand - 1 ) ) ? 1 : 2 ) ) https://smaugmuds.afkmods.com/topic/compiler-warning-for-decorateroom-4901/&p=23919#p23919 Tue, 10 Mar 2020 14:26:17 CDT SmaugFUSS nobody@example.com (Samson)