Hi all, I think I'm going to respin this patch to be a lot smaller. Just refactor one or two things, but leave the existing "verbose" and "applog" system in place. Let me know what you think-- do you think I should push ahead with HAIL_DEBUG etc., cheers. Colin On Tue, Dec 8, 2009 at 4:11 PM, Colin McCabe <cmccabe@xxxxxxxxxxxxxx> wrote: > This patch series creates a common.h and populates it with some function > prototypes and logging stuff. > > version 2: > * Rename CLD_DEBUG, etc. to HAIL_DEBUG > * use angle brackets for including common.h from include/ > > Colin McCabe (6): > cld: Declare common.c functions in common.h > cld: create logging macros > cld: modify client code to use logging macros > cld: modify server code to use logging macros > cld: modify cld-dns to use logging macros > cld: Tweak some log levels in cldc dns > > include/cld_msg.h | 10 ----- > include/cldc.h | 12 +++---- > include/common.h | 38 ++++++++++++++++++++ > lib/cldc-dns.c | 82 +++++++++++++++++--------------------------- > lib/cldc.c | 98 +++++++++++++++++++++------------------------------- > server/cld.h | 4 +- > server/cldb.c | 6 ++-- > server/msg.c | 49 +++++++++++--------------- > server/server.c | 86 ++++++++++++++++++++-------------------------- > server/session.c | 57 ++++++++++++------------------- > server/util.c | 27 +++++++------- > tools/cldcli.c | 14 +++++--- > 12 files changed, 223 insertions(+), 260 deletions(-) > create mode 100644 include/common.h > > -- To unsubscribe from this list: send the line "unsubscribe hail-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
