[Openais] First B.01.01 service - cluster membership (Now event service)

Steven Dake sdake at mvista.com
Thu Feb 3 15:20:54 PST 2005


On Thu, 2005-02-03 at 16:16, Mark Haverkamp wrote:
> On Thu, 2005-02-03 at 15:28 -0700, Steven Dake wrote:
> > Mark
> > 
> > I have had a thorough review of this patch and it looks great!
> > 
> > I only have one comment exec/evt.h includes ipc_evt.h.  I was hoping
> > that the ipc_evt.h file would only be included by exec/evt.c and
> > lib/evt.c.  No sense corrupting the rest of the namespace with the ipc
> > data structures since they are really private to the ipc in the event
> > service.
> 
> I can include saEvt.h instead.
> 

aha I see this makes more sense.

Thanks
-steve

> Mark.




More information about the Openais mailing list