<br><br><div class="gmail_quote">On Tue, Mar 24, 2009 at 11:52 PM, Thiago Macieira <span dir="ltr"><<a href="mailto:thiago@kde.org">thiago@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im"><br>
</div>Patch looks good, but before you commit, can you fix your indentation<br>
problems? Please use the same indentation as the rest of Dolphin (Dolphin<br>
appears to be using 4-space indentation but your new file is doing 2-then-4<br>
and some braces are in weird positions).</blockquote><div> </div><div>Yes, I know, I will clean it. Its because the kremoteencodingplugin.cpp file is abit out-dated doesn't fit KDE coding standarts :)<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Wasn't there a way to share this code with Konqueror?</blockquote><div><br>There isn't a way for Dolphin to share Konqueror code, But when I commit this there will be a way for Konqueror to share Dolphin code ;-)<br>
I will make DolphinPart to use DolphinRemoteEncoding So Konqueror will share same code. Thus we can get rid off KRemoteEncodingPlugin in Konqueror.<br></div></div><br><br><br><br clear="all"><br>-- <br>Rahman Duran<br><br>
Software Engineer<br>Turkey<br><br>How many apples fell on Newton's head before he took the hint! <br><br>