<br><div class="gmail_quote">On Fri, Jul 20, 2012 at 11:50 AM, Alex Fiestas <span dir="ltr"><<a href="mailto:afiestas@kde.org" target="_blank">afiestas@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
> If it's ok I can start pushing refactored code to master starting tomorrow.<br>
</div>That was fast! :p<br></blockquote><div> </div><div>I was in Tibet the last 4 years, i learned some secret refactoring tecniques :P Also, these two days i don't have much stuff to do at work.. that might have helped too ;)</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The common way of doing this is by creating a reviewrequest on<br>
<a href="https://git.reviewboard.kde.org" target="_blank">https://git.reviewboard.kde.org</a> or if the diff contains different commits and<br>
you consider that a diff would be difficult to review, then you can just email<br>
here a branch with your changes.<br></blockquote><div><br></div><div>I think i'll go for the second.. I mostly worked by extracting classes (SRP), and i also added unit tests, i guess a branch would be more easy to review.</div>

<div>I'll push later today, no ssh here :(</div><div>Cheers!</div><div>Cheers</div></div>