[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: kdesupport/soprano/soprano/inference
From:       Sebastian =?utf-8?q?Tr=C3=BCg?= <strueg () mandriva ! com>
Date:       2008-09-25 18:19:48
Message-ID: 200809252019.48532.strueg () mandriva ! com
[Download RAW message or body]

Actually, the qHash method is not intended to be public API. Thus, if 
possible, can it be removed from the header. Back when I implemented it, I 
just did not know the namespacing issue and found having a declaration in the 
header the only solution.

Cheers,
Sebastian

On Thursday 25 September 2008 16:13:10 Daniel Molkentin wrote:
> SVN commit 864795 by danimo:
>
> mea culpa
>
>
>  M  +0 -10     inferencemodel.cpp
>
>
> --- trunk/kdesupport/soprano/soprano/inference/inferencemodel.cpp
> #864794:864795 @@ -37,11 +37,6 @@
>
>  // FIXME: add error handling!
>
> -uint qHash(  const Soprano::Node& node )
> -{
> -    return qHash(  node.toString() );
> -}
> -
>  static Soprano::Node compressStatement( const Soprano::Statement&
> statement ) {
>      // There should be some method Statement::toXXXString for this
> @@ -73,11 +68,6 @@
>  {
>      return qHash( node.toString() );
>  }
> -
> -uint qHash( const Soprano::Node& node )
> -{
> -    return qHash( node.toString() );
> -}
>  //
> ---------------------------------------------------------------------------
>--------------------------------------------
[prev in list] [next in list] [prev in thread] [next in thread]