Cc pdlogs 61

Updated after review comments fro the ENT feature..  
Modified the messages to capitalize acronyms.  
Removed unnecessary whitespace.  
Added full stop to end of each part of the message.  
Actions are numbered if there is more than one action.  This separates the individual actions.
Separated the log definitions with a blank line to make them more readable.  
Ran astyle (according to the coding standards) against chronos_ent_definitions.h.
Fixed header file inconsistencies.
The review comment had: "A lot of the logs still say 'report issue to support' or something similar. This just isn't suitable for Project Clearwater."
The language was kept the same as this is what is done for the SNMP alarm definitions.  The associated documentation should identify the Metaswitch support team for ClearwaterCore and the Clearwater mailing list for Clearwater.
Removed signalnames.cpp.  The signal call name is now obtained from the strsignal call.
Fixed.

From: Eleanor Merry [mailto:notifications@github.com]
Sent: Thursday, December 18, 2014 10:41 AM
To: Metaswitch/chronos
Cc: David Collins
Subject: Re: [chronos] Cc pdlogs 61 (#92)


In src/include/chronos_ent_definitions.h<https://github.com/Metaswitch/chronos/pull/92#discussion-diff-22053413>:

> + * respects for all of the code used other than OpenSSL.

> + * "OpenSSL" means OpenSSL toolkit software distributed by the OpenSSL

> + * Project and licensed under the OpenSSL Licenses, or a work based on such

> + * software and licensed under the OpenSSL Licenses.

> + * "OpenSSL Licenses" means the OpenSSL License and Original SSLeay License

> + * under which the OpenSSL Project distributes the OpenSSL toolkit software,

> + * as those licenses appear in the file LICENSE-OPENSSL.

> + */

> +

> +#ifndef _CHRONOS_ENT_DEFINITIONS_H__

> +#define _CHRONOS_ENT_DEFINITIONS_H__

> +

> +#include <string>

> +#include "pdlog.h"

> +

> +// Defines instances of PDLog for the chronos module

Can you just use include lines 45-53 (down to Action), and do the same for the other repos

—
Reply to this email directly or view it on GitHub<https://github.com/Metaswitch/chronos/pull/92/files#r22053413>.