[Kde-pim] Re: Review Request: New NoteUtils namespace with NoteMessageWrapper class

[Christophe Giboudeaux]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102076/#review5067
-----------------------------------------------------------



akonadi/notes/CMakeLists.txt
<http://git.reviewboard.kde.org/r/102076/#comment4528>

    If you export it, also add it to KdepimLibsConfig.cmake.in



akonadi/notes/noteutils.h
<http://git.reviewboard.kde.org/r/102076/#comment4529>

    no camelcase in kdepimlibs


- Christophe


On July 24, 2011, 8:23 p.m., Christian Mollekopf wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102076/
> -----------------------------------------------------------
> 
> (Updated July 24, 2011, 8:23 p.m.)
> 
> 
> Review request for Akonadi, KDEPIM-Libraries and Stephen Kelly.
> 
> 
> Summary
> -------
> 
> NoteMessageWrapper is a convenience class to assemble/disassemble Kmime messages into notes(text message).
> This is a first step to standardize how notes are handled with akonadi/akonotes resource.
> 
> The class will be used by the upcoming akonotes-nepomuk-feeder as well as MindMirror.
> (Of course also kjots and any other app working with the akonotes resource might make use of it)
> 
> I put the files in a new notes directory, so we can put there future efforts like an even more convenient Note class,
> which I briefly discussed with skelly.
> 
> The NoteUtils namespace can be used for further note related helper functions/classes.
> 
> As skelly already had a look at it I will commit if noone objects within the next days.
> 
> 
> Diffs
> -----
> 
>   akonadi/notes/noteutils.h PRE-CREATION 
>   akonadi/notes/noteutils.cpp PRE-CREATION 
>   akonadi/notes/akonadi-notes_export.h PRE-CREATION 
>   akonadi/CMakeLists.txt cf63a0db5d7139d29cad268cd7aeaaa9a00e3cf1 
>   akonadi/notes/CMakeLists.txt PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/102076/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Christian
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/