On Sun, 11 Nov 2012, Phil Dibowitz wrote:

> num_stages seems unnecessary, we already provide num_stages on the very first
> callback by passing a stage_id of LC_CB_STAGE_NUM_STAGES.
>
> AFAIK, this is implemented in every entry point of libconcord.
>
> This patch hard-codes it a second time, and also doesn't take into account
> things like no-reset which the existing code does.
>
> The only piece is that *which* stages there are.
>
> What if... hmm.
>
> What if we alter the API slightly to say that on LC_CB_STAGE_NUM_STAGES, 'arg'
> is not an opague object, but instead a pointer to the list of stages?

Actually, I thought about this some more and I don't think it will work. 
The 'arg' pointer is for passing a pointer from the calling function, eg 
the code that calls update_configuration(), to the callback, not for us to 
pass information to the callback.  Congruity uses this to pass a pointer 
to the class that the callback needs to operate on, for example.  So I 
don't think we can go this way unless we add another argument to the 
callback signature.

Scott

------------------------------------------------------------------------------
Monitor your physical, virtual and cloud infrastructure from a single
web console. Get in-depth insight into apps, servers, databases, vmware,
SAP, cloud infrastructure, etc. Download 30-day Free Trial.
Pricing starts from $795 for 25 servers or applications!
http://p.sf.net/sfu/zoho_dev2dev_nov
_______________________________________________
concordance-devel mailing list
concordance-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/concordance-devel

Reply via email to