I've applied a modified version of your patch as http://hg.clusterlabs.org/pacemaker/stable-1.0/rev/deb967617b9e
Thanks for doing the initial legwork on this! On Sat, May 16, 2009 at 2:29 AM, Mark Hamzy <ha...@us.ibm.com> wrote: > Here is attempt #3: > > (See attached file: pacemaker.mark.patch) > > I chose not to use pe_working_set_t* in char2score and instead used a small > structure. I noticed a couple of things about crm_int_helper(). > 1) It doesn't have a way to indicate an error. It returns -1, but aren't > signed integers valid input? yes, not strictly ideal. i should probably fix that one day > 2) It returns long long but most of that is thrown away when converted to an > int., uint32, or uint64. yep, if the caller wants to throw the rest away thats fine. i just dont want it lost in the original conversion > > Questions/comments? > > Mark > > Common Information Model/Web-Based Enterprise Management at > http://www.openpegasus.org/ > Take a look at the Linux Omni Printer Driver Framework at > http://omniprint.sourceforge.net/ > > _______________________________________________ > Pacemaker mailing list > Pacemaker@oss.clusterlabs.org > http://oss.clusterlabs.org/mailman/listinfo/pacemaker > > _______________________________________________ Pacemaker mailing list Pacemaker@oss.clusterlabs.org http://oss.clusterlabs.org/mailman/listinfo/pacemaker