Re: [devel] [PATCH 1 of 1] amfd: Add support for google unit test framework V2 [#1142]

2015-05-26 Thread Hans Feldt
On 2015-05-26 07:23, praveen malviya wrote: On 08-May-15 1:41 PM, Hans Nordeback wrote: 00-README.unittest | 32 configure.ac| 1 + osaf/services/saf/amf/amfd/Makefile.am | 2 +-

Re: [devel] [PATCH 0 of 1] Review Request for amfd: Add support for google unit test framework V2 [#1142]

2015-05-26 Thread Anders Widell
Ack (for both patches on this ticket), with some comments: * Boiler plates (licence copyright) are missing in the new files * The README assumes the developer uses Ubuntu and has sudo rights (root access). Instead, I think it would be better to describe how to download the Google test

[devel] [PATCH 0 of 1] Review Request for IMM: Detour of accessorGet into ImmModel::searchInitialize removed [#674]

2015-05-26 Thread Anders Bjornerstedt
Summary: IMM: Detour of accessorGet into ImmModel::searchInitialize removed [#674] Review request for Trac Ticket(s): 674 Peer Reviewer(s): Neel; Zoran Pull request to: Affected branch(es): default(4.7) Development branch: default(4.7) Impacted area Impact

[devel] [PATCH 1 of 1] IMM: Detour of accessorGet into ImmModel::searchInitialize removed [#674]

2015-05-26 Thread Anders Bjornerstedt
osaf/services/saf/immsv/immnd/ImmModel.cc | 30 +++--- osaf/services/saf/immsv/immnd/immnd_evt.c | 2 +- 2 files changed, 16 insertions(+), 16 deletions(-) The detour seemed mostly to deal with handling of the SaImmScopeT type. The scope of SA_IMM_ONE is only relevant

Re: [devel] [PATCH 1 of 1] amfd: ignore invalid modification of saAmfSGMaxActiveSIsperSU/saAmfSGMaxStandbySIsperSU [#1361]

2015-05-26 Thread Hans Nordebäck
Ack, code review only/Thanks HansN -Original Message- From: nagendr...@oracle.com [mailto:nagendr...@oracle.com] Sent: den 21 maj 2015 09:21 To: Hans Nordebäck; praveen.malv...@oracle.com Cc: opensaf-devel@lists.sourceforge.net Subject: [PATCH 1 of 1] amfd: ignore invalid modification of