<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/100126/">http://git.reviewboard.kde.org/r/100126/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I like the changes you did, in general. Please, take care of the things listed below and then it's ok for me.</pre>
<br />
<p>- Andrea</p>
<br />
<p>On November 1st, 2010, 4:34 p.m., Jon Ander Peñalba wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for rekonq and Yoann Laissus.</div>
<div>By Jon Ander Peñalba.</div>
<p style="color: grey;"><i>Updated 2010-11-01 16:34:59</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I've done some clean-up to the bookmark API.
There are still some things I want to improve, I hope to work on it soon, but I've been busier than expected lately.
This diff is more than one commit, if you want to see each commit look at my personal clone: http://gitorious.org/~Jonan/rekonq/jonans-mainline/commits/bookmark</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/bookmarks/bookmarkowner.h <span style="color: grey">(573e4ee)</span></li>
<li>src/bookmarks/bookmarkowner.cpp <span style="color: grey">(77af9b1)</span></li>
<li>src/bookmarks/bookmarkprovider.cpp <span style="color: grey">(ccdc326)</span></li>
<li>src/bookmarks/bookmarkscontextmenu.h <span style="color: grey">(d669eef)</span></li>
<li>src/bookmarks/bookmarkscontextmenu.cpp <span style="color: grey">(93df6ce)</span></li>
<li>src/bookmarks/bookmarkspanel.h <span style="color: grey">(f042d4c)</span></li>
<li>src/bookmarks/bookmarkspanel.cpp <span style="color: grey">(f732cb2)</span></li>
<li>src/bookmarks/bookmarkstoolbar.cpp <span style="color: grey">(4449d92)</span></li>
<li>src/history/historypanel.h <span style="color: grey">(0a5ae21)</span></li>
<li>src/history/historypanel.cpp <span style="color: grey">(28f7f4e)</span></li>
<li>src/urlpanel.h <span style="color: grey">(722eeae)</span></li>
<li>src/urlpanel.cpp <span style="color: grey">(b3e8b2f)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100126/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>