[test-results] [Bamboo] Asterisk > Asterisk Build > #1800 has FAILED. Change made by Matt Jordan <mjordan at digium.com>.

Bamboo noreply at bamboo.asterisk.org
Sun Apr 26 15:55:49 CDT 2015


-----------------------------------------------------------------------
Asterisk > Asterisk Build > #1800 failed.
-----------------------------------------------------------------------
Code has been updated by Matt Jordan <mjordan at digium.com>.
1/2 jobs failed, no tests found.

https://bamboo.asterisk.org/bamboo/browse/AST-ATRUNKBUILD-1800/

---------------------
Currently Responsible
---------------------

foobar  (Automatically assigned)



--------------
Failing Jobs
--------------
  - Basic Build (Basic Build): No tests found.



--------------
Code Changes
--------------
Diederik de Groot <ddegroot at talon.nl> (f8e21a1adf91bfa141771415b19c31236c2f1a17):

>Clang: Fix some more tautological-compare warnings.
>clang can warn about a so called tautological-compare, when it finds
>comparisons which are logically always true, and are therefor deemed
>unnecessary.
>
>Exanple:
>unsigned int x = 4;
>if (x > 0)    // x is always going to be bigger than 0
>
>Enum Case:
>Each enumeration is its own type. Enums are an integer type but they
>do not have to be *signed*. C leaves it up to the compiler as an
>implementation option what to consider the integer type of a particu-
>lar enumeration is. Gcc treats an enum without negative values as
>an int while clang treats this enum as an unsigned int.
>
>rmudgett & mmichelson: cast the enum to (unsigned int) in assert.
>The cast does have an effect. For gcc, which seems to treat all enums
>as int, the cast to unsigned int will eliminate the possibility of
>negative values being allowed. For clang, which seems to treat enums
>without any negative members as unsigned int, the cast will have no
>effect. If for some reason in the future a negative value is ever
>added to the enum the assert will still catch the negative value.
>
>ASTERISK-24917
>Change-Id: Ief23ef68916192b9b72dabe702b543ecfeca0b62

Matt Jordan <mjordan at digium.com> (0c92a85aee281b4cd162db6125bcc1d63ed4d6af):

>Merge "Clang: Fix some more tautological-compare warnings."



--
This message is automatically generated by Atlassian Bamboo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/test-results/attachments/20150426/bbe2e6fe/attachment-0001.html>


More information about the Test-results mailing list