Duly noted. I added the other stuff
"Now passes checkpactch.pl and has no obvious style problems and is ready for
submission."
as a way of saying I ran the tests. Wasn't sure where to put it without
breaking the git import.
On 11-03-28 03:25 PM, Alexandre Montplaisir wrote:
On 11-03-28 01:12 PM, Matthew Khouzam wrote:
Added a check in get online pids to only send the pids that are
currently online.
Now passes checkpactch.pl and
has no obvious style problems and is ready for submission.
Signed-off-by: Matthew Khouzam<[email protected]>
---
I have to nitpick here (we can't let you go can we ;) )
The first line of git messages should be at the present tense ("Add a
check...") and not go over 50 characters.
The second line should ALWAYS be empty.
If you use a real editor to generate the commits (like vim, not the one
in Eclipse for example...) it will warn you of those things.
Handy link:
http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html
_______________________________________________
ltt-dev mailing list
[email protected]
http://lists.casi.polymtl.ca/cgi-bin/mailman/listinfo/ltt-dev
