Review Request: Add support for parsing relative present actions in notification contexts

[Jacopo De Simoi]

> On 2010-10-25 12:12:45, Olivier Goffart wrote:
> > trunk/KDE/kdebase/runtime/knotify/knotify.cpp, line 180
> > <http://svn.reviewboard.kde.org/r/5631/diff/1/?file=39314#file39314line180>
> >
> >     I do not really like this JavaStyle iterator.    I think foreach would be better and more intuitive in that case.

I guess my concern using foreach was that this way we check the first element of the list twice, but that's not such a big deal after all. The code is much more readable now.


- Jacopo


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5631/#review8339
-----------------------------------------------------------


On 2010-10-15 11:08:19, Jacopo De Simoi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5631/
> -----------------------------------------------------------
> 
> (Updated 2010-10-15 11:08:19)
> 
> 
> Review request for kdelibs and Olivier Goffart.
> 
> 
> Summary
> -------
> 
> This patch allows to specify, for some specific notification contexts, present actions that should be either added or removed with 
> respect to the default (i.e. without context) ones. E.g. one could write notifyrc files containing:
> 
> [Event/someevent/presentation/fullscreen]
> Action=-Sound|-Popup
> 
> [Event/someevent/quietbutverbose/whatever]
> Action=+Popup|-Sound
> 
> [Event/someevent/morespecificnotifier/present]
> Action=-Popup
> 
> This lets the user to customize non-contextualized actions via the usual interface, but at the same time allows some control by the
> developer in some particular situations.
> 
> The patch changes knotifyconfig.cpp in order to parse contextualized actions also in the system-wide file and includes some basic 
> validity check in such a way to recognize invalid expressions such as:
> 
> Action=-Popup|Sound
> 
> I don't know where to document this behaviour; any help on that is most appreciated.
> Thanks!
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdebase/runtime/knotify/knotify.cpp 1185812 
>   trunk/KDE/kdebase/runtime/knotify/knotifyconfig.cpp 1185812 
> 
> Diff: http://svn.reviewboard.kde.org/r/5631/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jacopo
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20101026/ad10e4f0/attachment.htm>