Merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba

[Paul J. Lucas]

Added:

    bool ItemHandle::operator!() const;

so you can write code like:

    if ( !h )

rather than the annoying:

    if ( !h.getp() )

Note that is a hack fix. The correct way to do this is by adding conversion-to-bool. However, adding that causes ambiguities because whoever wrote ItemHandle added implicit conversion to T* which is wrong. Removing implicit conversion to T*, however, breaks too much code; hence the hack fix.

Added:

    bool ItemHandle::operator!() const;

so you can write code like:

    if ( !h )

rather than the annoying:

    if ( !h.getp() )

Note that is a hack fix. The correct way to do this is by adding conversion-to-bool. However, adding that causes ambiguities because whoever wrote ItemHandle added implicit conversion to T* which is wrong. Removing implicit conversion to T*, however, breaks too much code; hence the hack fix.

[Paul J. Lucas (paul-lucas)]

review: Approve

[Matthias Brantner (matthias-brantner)]

review: Approve

[Zorba Build Bot (zorba-buildbot)]

Validation queue starting for merge proposal.
Log at: http://zorbatest.lambda.nu:8080/remotequeue/pjl-misc-2013-04-04T03-44-47.432Z/log.html

[Zorba Build Bot (zorba-buildbot)]

Validation queue job pjl-misc-2013-04-04T03-44-47.432Z is finished. The final status was:

All tests succeeded!

=== modified file 'src/store/api/item_handle.h'
--- src/store/api/item_handle.h	2013-03-06 10:36:56 +0000
+++ src/store/api/item_handle.h	2013-04-04 02:55:28 +0000
@@ -65,6 +65,7 @@
   }
 
   bool isNull () const { return p == NULL; }
+  bool operator!() const { return !p; }
 
   void setNull() { p = NULL; }