Re: [devel] [PATCH 1 of 1] amfd: Convert remove_csi and add_csi to static member functions [#713]

2014-06-26 Thread Hans Feldt
Each SI _instance_ do have a list of CSIs, what do you mean? I think this patch is wrong... Thanks, Hans -Original Message- From: Gary Lee [mailto:gary@dektech.com.au] Sent: den 26 juni 2014 04:11 To: Hans Feldt; Hans Nordebäck; nagendr...@oracle.com Cc:

Re: [devel] [PATCH 1 of 1] amfd: Convert remove_csi and add_csi to static member functions [#713]

2014-06-26 Thread Gary Lee
Hi Hans Yeah you're right. I should just refactor add_csi and remove_csi. Will post a replacement patch. Thanks Gary On 26/06/14 20:49, Hans Feldt wrote: Each SI _instance_ do have a list of CSIs, what do you mean? I think this patch is wrong... Thanks, Hans -Original Message-

[devel] [PATCH 1 of 1] amfd: Convert remove_csi and add_csi to static member functions [#713]

2014-06-25 Thread Gary Lee
osaf/services/saf/amf/amfd/csi.cc | 4 ++-- osaf/services/saf/amf/amfd/include/si.h | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) These functions don't operate directly on this SI instance. Let's make them static member functions for now. diff --git