[aur-dev] [PATCH] xhtml validation fixes for login_form.php

[Michael Klier]

Loui wrote:
> There are a couple worthwhile things in here but most of this just
> looks like messing with the indentation and such. Overall tabs
> are actually used for indentation, though it's not written anywhere.

I was actually wondering about that because I haven't found a real refernce
and I've seen files with a mix of tabs and whitespace. And you're right, most
of it is just messing with the indentation. But I'll use tabs from now on :).

> You should try to write shorter patches that don't mess with
> everything in the file. It's less for the maintainers to read through
> and less for you to write. Then we can get more patches through in less
> time and you can write more. :D Win win!

I see your point. In case of the search_pkg_form.php patch, well, this was a
complete rewrite/restructuring (preserving it's functionality of course). With
the end result in mind it's kinda difficult to do such a rewrite in smaller
chunks (and I dreamed about the promised cookie already :D).

Anyway, thanks for your reply, I appreaciate your advice and will do my best
to conform to these guidelines in the future :).

    Michael

-- 
Michael Klier
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://archlinux.org/pipermail/aur-dev/attachments/20080606/0ca5fbf1/attachment.pgp>