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 To:

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 = cpnd_ckpt_sec_get_next(&cp_node->replica_