Review Request: Allow Link on Pasetbin Plugin Message Box

David Narváez david.narvaez at computer.org
Sun Dec 18 04:31:05 UTC 2011


-----------------------------------------------------------
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.


Changes
-------

Had an error uploading the patch on the first try.


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 (updated)
-----

  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/9e37d1b2/attachment.html>


More information about the KDevelop-devel mailing list