(no subject)

[David Zeuthen]

>Small problem with this (otherwise cool code): %dc|p is taken as one
>integer with a modifier, rather than %d[invisiblespace]c|p - I've added
>another variable &type which lets me refactor things slighty. Tell me
>if there's an easier way to convice sscanf to do it's thing.

Looks good.

<snip>

>> As a general issue, please send patches as attachments, not inline,
>and
>
>Mmmookay, but that means using Outlook Web Mail. Which won't send using
>plain text. Rawhide Evolution doesn't like newsgroups like it used to.
>Tell me if there's a problem with the patch.

HTML mails, broken threading and no subject is considered worse by most
people though :-) - Why don't you just subscribe to the mailing list and
use Evolution instead?
>
>> Also, you need to
>> write an ChangeLog entry and include that in the patch.
>
>Done. Feel free to edit any of the patch. Note I've also updated my
>email in the
>ChangeLog too - ee21rh at surrey.ac.uk is the account I use to send,
>richard at hughsie.com as the account to receive. It's just a redirect,
>but
>lets me get keep my email address when I move mailboxes.

I've committed this and I've changed the ChangeLog entry to somewhat
adhere to

 http://www.gnu.org/prep/standards/html_node/Change-Logs.html

Thanks for your contribution.

Cheers,
David


_______________________________________________
hal mailing list
hal at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/hal