[Openais] openais CsiSet Callback

[Steven Dake]

On Fri, 2006-11-17 at 15:48 +0100, Ola Lundqvist wrote:
> Hi
> 
> Venkat Chandra wrote:
> > I mean, the last parameter is listed as a pointer - but, the spec seems
> > to indicate otherwise...
> 
> To me it looks like the specification is broken, but it valid code and
> AMF do not follow it, so yes it can be a problem.
> 
> Regards,
> 
> // Ola
> 
> 
> > Venkat
It is my view the specification is broken in this regard which is why
the implementation varies from the specification.  If someone can
convince me how we could get it to work otherwise, we could change it to
match the spec.

Regards
-steve

> > 
> >> From: Hans Feldt <Hans.Feldt at ericsson.com>
> >> To: Venkat Chandra <venkatec at hotmail.com>
> >> CC: Openais at lists.osdl.org
> >> Subject: Re: [Openais] openais CsiSet Callback
> >> Date: Fri, 17 Nov 2006 07:51:21 +0100
> >>
> >> Venkat Chandra wrote:
> >>> Folks,
> >>>
> >>> The AIS spec indicates the following for the CsiSet callback...
> >>>
> >>>                     (SaInvocationT inv,
> >>>                      const SaNameT * comp_name,
> >>>                      SaAmfHAStateT ha_state, SaAmfCSIDescriptorT
> >>> csi_desc)
> >>>
> >>> However, openais uses *csi_desc for the 3rd parameter.
> >>>
> >>> Is this a violation of the spec ? Or, am I missing something here ?
> >>
> >> From whitetank/include/saAmf.h (same as trunk):
> >>
> >> typedef void (*SaAmfCSISetCallbackT) (
> >>     SaInvocationT invocation,
> >>     const SaNameT *compName,
> >>     SaAmfHAStateT haState,
> >>     SaAmfCSIDescriptorT *csiDescriptor);
> >>
> >> Same as the spec.
> >>
> >> Regards,
> >> Hans
> >>
> >>>
> >>> Venkat
> >>>
> >>> _________________________________________________________________
> >>> Get FREE company branded e-mail accounts and business Web site from
> >>> Microsoft Office Live
> >>> http://clk.atdmt.com/MRT/go/mcrssaub0050001411mrt/direct/01/
> >>>
> >>> _______________________________________________
> >>> Openais mailing list
> >>> Openais at lists.osdl.org
> >>> https://lists.osdl.org/mailman/listinfo/openais
> >>>
> >>
> > 
> > _________________________________________________________________
> > Get the latest Windows Live Messenger 8.1 Beta version. Join now.
> > http://ideas.live.com
> > 
> > 
> > ------------------------------------------------------------------------
> > 
> > _______________________________________________
> > Openais mailing list
> > Openais at lists.osdl.org
> > https://lists.osdl.org/mailman/listinfo/openais
> 
>