LSF : & au lieu de && dans if (Bugzilla #449)

bdepardo commented 11 years ago

normal bug in component TMS Reported in version 2.0.0 beta2 on platform All

bdepardo commented 11 years ago

On 2012-12-08 17:30:18 +0100, 'Benjamin Depardon (benjamin.depardon@sysfera.com) wrote: En parcourant le code de LSFServer.cpp, je suis tombé sur les if suivants : if (queueInfo[i].qStatus & QUEUE_STAT_ACTIVE) { if (!queueInfo[i].qStatus & QUEUE_STAT_OPEN) { queue->setState(0); } if (queueInfo[i].qStatus & QUEUE_STAT_RUN) { queue->setState(2); } else { queue->setState(1); } } else {

Est-ce que ce sont bien des & qui doivent être utilisés ? ou c'est une erreur et ce sont des && ? Cppcheck m'a retourné cette erreur qui m'a permis de détecter ça : [TMS/src/server/LSFServer.cpp:665]: (style) Boolean result is used in bitwise operation. Clarify expression with parentheses

bdepardo commented 11 years ago

On 2013-01-17 13:29:13 +0100, 'Benjamin Depardon (benjamin.depardon@sysfera.com) wrote: C'est bien un & et non &&. Ça a été commenté et des parenthèses sont autour de l'expression. Commit 0752c20a084d8b265ea44d14eacd20c7783a0f7f

SysFera / vishnu

LSF : & au lieu de && dans if (Bugzilla #449) #306