[devel] [PATCH 3 of 4] log: update logsv to support long DN [#1315]

2016-07-01 Thread Vu Minh Nguyen
osaf/services/saf/logsv/lgs/Makefile.am |1 + osaf/services/saf/logsv/lgs/lgs.h |4 +- osaf/services/saf/logsv/lgs/lgs_amf.cc | 14 +- osaf/services/saf/logsv/lgs/lgs_config.cc | 30 +- osaf/services/saf/logsv/lgs/lgs_evt.cc | 77 -- osaf/services/saf/lo

Re: [devel] [PATCH 3 of 4] log: update logsv to support long DN [#1315]

2016-08-03 Thread Lennart Lund
Hi Vu Ack with comments and after static checks done See comments inline [Lennart] Also if not done already, run the static checks (cppcheck and cpplint) and fix found problems for at least the new code. Thanks Lennart > -Original Message- > From: Vu Minh Nguyen [mailto:vu.m.ngu...@de

Re: [devel] [PATCH 3 of 4] log: update logsv to support long DN [#1315]

2016-08-03 Thread Vu Minh Nguyen
Hi Lennart, Thanks for your comments. Please see my responses inline [Vu]. Regards, Vu > -Original Message- > From: Lennart Lund [mailto:lennart.l...@ericsson.com] > Sent: Wednesday, August 3, 2016 8:57 PM > To: Vu Minh Nguyen ; > mahesh.va...@oracle.com > Cc: opensaf-devel@lists.sourcef