[Nepomuk] Re: Review Request: Extra function for SimpleResource and SimpleResourceGraph
Artem Serebriyskiy
v.for.vandal at gmail.com
Thu Jul 7 10:54:30 CEST 2011
> On July 7, 2011, 8:26 a.m., Vishesh Handa wrote:
> > nepomuk/services/storage/lib/simpleresourcegraph.h, line 76
> > <http://git.reviewboard.kde.org/r/101856/diff/2/?file=26199#file26199line76>
> >
> > Please implement this in the .cpp file.
Why ? This way this function is inline and using X.size() or X.count() will have no difference. If I put it to .cpp file, then it will be one useless function call more.
- Artem
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101856/#review4479
-----------------------------------------------------------
On July 7, 2011, 8:44 a.m., Artem Serebriyskiy wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101856/
> -----------------------------------------------------------
>
> (Updated July 7, 2011, 8:44 a.m.)
>
>
> Review request for Nepomuk.
>
>
> Summary
> -------
>
> Extra functions to SimpleResource|Graph
> 1. Operator comparison for Graph
> 2. addGraph for Graph - merges 2 graphs into one
> 3. Serialization operators to QDataStream to both classes.
> 4. Tests for simpleresourcetest
>
>
> Diffs
> -----
>
> nepomuk/services/storage/lib/simpleresource.h c6c3b04
> nepomuk/services/storage/lib/simpleresource.cpp 3d529bb
> nepomuk/services/storage/lib/simpleresourcegraph.h 6a3a0ee
> nepomuk/services/storage/lib/simpleresourcegraph.cpp 419a949
> nepomuk/services/storage/test/CMakeLists.txt f8d00de
> nepomuk/services/storage/test/simpleresourceclassestest.h PRE-CREATION
> nepomuk/services/storage/test/simpleresourceclassestest.cpp PRE-CREATION
>
> Diff: http://git.reviewboard.kde.org/r/101856/diff
>
>
> Testing
> -------
>
> Provide tests for this operations. Currently failing addGraph test, because of SimpleResource::addProperties behaviour. But may be it is intended, then I will rewrite test case.
>
>
> Thanks,
>
> Artem
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/nepomuk/attachments/20110707/1deea236/attachment-0001.htm
More information about the Nepomuk
mailing list