Re: [devel] [PATCH 1 of 1] imported patch 1967_fix_headless_error_cppcheck.patch

2016-08-24 Thread A V Mahesh
ACK not tested -AVM On 8/25/2016 10:06 AM, Vo Minh Hoang wrote: > Dear Mahesh, > > Logic of this part is complicated so I modified it wrong. > This time I correct it, not a roll back. > That part is subsequence of if clause above. > > Sincerely, > Hoang > > -Original Message- > From: A V

Re: [devel] [PATCH 1 of 1] imported patch 1967_fix_headless_error_cppcheck.patch

2016-08-24 Thread Vo Minh Hoang
Dear Mahesh, Logic of this part is complicated so I modified it wrong. This time I correct it, not a roll back. That part is subsequence of if clause above. Sincerely, Hoang -Original Message- From: A V Mahesh [mailto:mahesh.va...@oracle.com] Sent: Thursday, August 25, 2016 11:21 AM

Re: [devel] [PATCH 1 of 1] imported patch 1967_fix_headless_error_cppcheck.patch

2016-08-24 Thread A V Mahesh
Hi Hoang, Is this rollback of `ckpt: fix cppcheck warning [#1874]` ? === @@ -4766,6 +4758,13 @@ static uint32_t cpnd_transfer_replica(CP total_num++; tmp_sec_info =

[devel] [PATCH 1 of 1] imported patch 1967_fix_headless_error_cppcheck.patch

2016-08-24 Thread Hoang Vo
osaf/services/saf/cpsv/cpd/cpd_proc.c | 4 ++-- osaf/services/saf/cpsv/cpnd/cpnd_evt.c | 11 ++- 2 files changed, 4 insertions(+), 11 deletions(-) diff --git a/osaf/services/saf/cpsv/cpd/cpd_proc.c b/osaf/services/saf/cpsv/cpd/cpd_proc.c --- a/osaf/services/saf/cpsv/cpd/cpd_proc.c