Code review comment for lp:~danrabbit/midori/fix-1172589

Revision history for this message
Paweł Forysiuk (tuxator) wrote :

Please clean up this patch before proposing to merge.

You should not change code style (adding brackets, changing where they appear) unless it really helps readability. Your style changes makes this patch way bigger than it needs to be and makes it hard to see real changes. Patches should be as small as possible generally. Also there is a typo in line 157 - "Boomarks".

review: Needs Fixing

« Back to merge proposal