@abh3 haha, well, it's GCC making sure people don't miss things unintentionally - still compiles the same. Maybe this is even nicer for readability?: ```c++ // Set all other static values // noids = Parms.oidsOK == 0; if (Parms.Dflttl > 0) dflTTL = Parms.Dflttl; if (Parms.Maxttl > 0) maxTTL = Parms.Maxttl; if (Parms.Logok >= 0) LogOK = Parms.Logok; if (Parms.Strm > 0) tcpSTRM= Parms.Strm; if (Parms.SMax > 0) tcpSMax= Parms.SMax; if (Parms.Xmax > 0) xfrMax = Parms.Xmax; if (Parms.Grab < 0) errMon = Parms.Grab; if (Parms.xEcho >= 0) doEcho = Parms.xEcho != 0; if (Parms.autoRM >= 0) autoRM = Parms.autoRM != 0; ``` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/xrootd/xrootd/issues/961#issuecomment-483228038 ######################################################################## Use REPLY-ALL to reply to list To unsubscribe from the XROOTD-DEV list, click the following link: https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1