Review Request: Allow Link on Pasetbin Plugin Message Box

Aleix Pol Gonzalez aleixpol at gmail.com
Sun Dec 18 17:11:36 UTC 2011


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

Ship it!


Ship It!

- Aleix Pol Gonzalez


On Dec. 18, 2011, 4:31 a.m., David Narváez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103442/
> -----------------------------------------------------------
> 
> (Updated Dec. 18, 2011, 4:31 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> Similar to review 103139 (without enum namespaces as later noticed by Millian Wolff). Should be straightforward except for the OR'ed flags at the end, trying to keep the previous behavior (Notify is the default flag) and add the AllowLink flag. Sounds good?
> 
> 
> Diffs
> -----
> 
>   plugins/pastebin/pastebinplugin.cpp 7d59fd0 
> 
> Diff: http://git.reviewboard.kde.org/r/103442/diff/diff
> 
> 
> Testing
> -------
> 
> Posted a patch with pastebin plugin and then clicked on the link in the message box that pops up at the end of the process.
> 
> 
> Thanks,
> 
> David Narváez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20111218/dbf4c634/attachment.html>


More information about the KDevelop-devel mailing list