Mailing-List: contact celix-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: celix-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of gerritbinnenmars@gmail.com
 designates 209.85.215.171 as permitted sender)
Message-ID: <52AA0C64.9090105@gmail.com>
Date: Thu, 12 Dec 2013 20:20:04 +0100
From: Gerrit Binnenmars <gerritbinnenmars@gmail.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64;
 rv:24.0) Gecko/20100101 Thunderbird/24.2.0
MIME-Version: 1.0
To: celix-dev@incubator.apache.org
Subject: Re: celix log levels
References: <5fd2842bb494b7f8330d4c62f002e5fa@imap.sundevil.de>
In-Reply-To: <5fd2842bb494b7f8330d4c62f002e5fa@imap.sundevil.de>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

Hello Bjorn,

It is a good proposal, so just do it and make a patch for Celix.
I can imagine that there are a lot more "prefixes"missing?

Greetings Gerrit
>
>
> Hi all,
>
> While coupling a log_writer to syslog, I recognized that celix has 
> some log levels (as enum in log_service.h) which are already defined 
> within syslog. As I want to prevent strange #undefine's in my code and 
> changing syslog is obviously not a solution, I would propose to add a 
> "CELIX_" prefix to the log levels.
>
> As this implies a change to several source-files, I would like to ask 
> whether there are any objections or other proposals?
>
> Regards,
>   Bjoern