Patch attached. What I have changed:
* Replaced `tok` and `ctok` functions with `skip` and (already
existing, but a little different) `eat`
* Changed parsing of separators (<SPACE> from RFC) in server
responses. Now tabs are not skipped. Also no more than one space is
skipped. RFC says there can be more than one space but I have seen
no server that will add extra separator characters. It is not hard
to parse but simplicity > correctness.
* Removed always true `if(msg[1])` check
* Added output of command parameters:
- pout(usr, ">< %s: %s", cmd, txt);
+ pout(usr, ">< %s (%s): %s", cmd, msg, txt);
Now it is possible to use such commands as "LIST"
* Added function `trim` for removing trailing whitespace.
It is now used on `msg` variable before outputting it so instead of
reticulum.oftc.net: 04/15/10 17:45 >< NOTICE (AUTH ): *** Looking up your hostname...
you get
reticulum.oftc.net: 04/15/10 17:45 >< NOTICE (AUTH): *** Looking up your hostname...
It is not a problem when you output it with "%-12s" format like
first argument of pout, but it is easy to see the problem when you
output it in parenthesis.
Code is one line less now.
This archive was generated by hypermail 2.2.0 : Thu Apr 15 2010 - 14:00:02 UTC