Re: [PATCH 0/6 v2] logging refactoring

[Colin McCabe <cmccabe@xxxxxxxxxxxxxx>]

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