No need to test Solvable for being identical with itself.

diff --git a/zypp/PoolItem.h b/zypp/PoolItem.h
index d8854da..bf10d7d 100644 (file)
--- a/zypp/PoolItem.h
+++ b/zypp/PoolItem.h
@@ -187,7 +187,7 @@ namespace zypp
 
   /** \relates PoolItem Test for same content. */
   inline bool identical( const PoolItem & lhs, const PoolItem & rhs )
-  { return lhs.satSolvable().identical( rhs.satSolvable() ); }
+  { return lhs == rhs || lhs.satSolvable().identical( rhs.satSolvable() ); }
 
   /** \relates PoolItem Test for same content. */
   inline bool identical( const PoolItem & lhs, sat::Solvable rhs )

diff --git a/zypp/sat/Solvable.cc b/zypp/sat/Solvable.cc
index c15d169..453875f 100644 (file)
--- a/zypp/sat/Solvable.cc
+++ b/zypp/sat/Solvable.cc
@@ -503,7 +503,7 @@ namespace zypp
     {
       NO_SOLVABLE_RETURN( ! rhs.get() );
       ::_Solvable * rhssolvable( rhs.get() );
-      return rhssolvable && ::solvable_identical( _solvable, rhssolvable );
+      return rhssolvable && ( _solvable == rhssolvable || ::solvable_identical( _solvable, rhssolvable ) );
     }
 
     ///////////////////////////////////////////////////////////////////