Bug #44570 In DEBUG, Disable (un)likely branch hints as they hide precious compiler warning

Description:
Branch prediction hints likely/unlikely are implemented by usage of __builtin_expect(). This has the side effect that compiler warnings about missuse
of '=' instead of '==' in comparisons are lost - Ref bug#44567.

In DEBUG mode we should remove the branch hinting in order to make these warnings vissible

How to repeat:
Code feature - Not a bug.

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/73091

2879 Ole John Aske	2009-04-30 [merge]
      Fix bug#44570

Pushed into 5.1.32-ndb-7.0.6 (revid:ole.john.aske@sun.com-20090430084504-2c8iheeuc631njs5) (version source revid:ole.john.aske@sun.com-20090430084504-2c8iheeuc631njs5) (merge vers: 5.1.32-ndb-7.0.6) (pib:6)

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/73152

2880 Ole John Aske	2009-04-30
      bug#44570, In DEBUG, Disable (un)likely branch hints as they hide precious compiler warning
     @ storage/ndb/src/kernel/blocks/dbtup/DbtupExecQuery.cpp
        Remove warning created by redefined unlikely() macro

It looks like you can remove the "ret" variable in that scope and then just keep the original condition:

	unlikely(handle_size_change_after_update(req_struct,
						       base,
						       regOperPtr.p,
						       regFragPtr,
						       regTabPtr,
						       sizes)

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/73259

2881 Ole John Aske	2009-05-02
      bug#44570, fix changed according to review comment

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/73260

2879 Ole John Aske	2009-05-02 [merge]
      bug#44570, comlete fix including review comments

OK

Pushed into 5.1.32-ndb-7.0.6 (revid:ole.john.aske@sun.com-20090504100939-z2u4n8ac2xpkbrsn) (version source revid:ole.john.aske@sun.com-20090504100939-z2u4n8ac2xpkbrsn) (merge vers: 5.1.32-ndb-7.0.6) (pib:6)

A patch for this bug has been committed. After review, it may
be pushed to the relevant source trees for release in the next
version. You can access the patch from:

  http://lists.mysql.com/commits/73288

2880 Ole John Aske	2009-05-04
      Fix bug#44570, after merge to resolve conflicts
     @ storage/ndb/src/kernel/blocks/dbtup/DbtupExecQuery.cpp
        Had to change usage of unlikely() to avoide compiler error

Documented bugfix in the NDB-7.0.6 changelog as follows:

        Use of __builtin_expect() had the side effect that compiler
        warnings about misuse of '=' instead of '==' in comparisons were
        lost when building in debug mode. This is no longer used when
        configuring the build employing --with-debug.