David Faure <faure@kde.org> writes:
> On Wednesday 11 March 2009, Jonathan Marten wrote:
>> Albert Astals Cid <aacid@kde.org> writes:
>> > You forgot to attach the patch :D
>> 
>> It's attached to the bug report
>> (https://bugs.kde.org/show_bug.cgi?id=186230).  Follows here too...
>
> No objections from me. (Can't be 100% sure that there are no regressions, of course,
> I don't know that code enough).

Thanks David for your comments.  OK to commit with your suggestions
implemented? - I've run through the test cases and will do so again
with these changes, and since this will only go into trunk any obscure
regressions will not affect the stable branch.

Regards, Jonathan

-- 
Jonathan Marten                         http://www.keelhaul.demon.co.uk
Twickenham, UK                          jjm2@keelhaul.demon.co.uk