Hi Michael,
you are rising in interesting point.
 
We are trying to implement the SAF specifications, so OpenHPI should use
the return values as specified by SA Forum.
Actually SaHpi.h is the official SAF headerfile for the HPI.
But I agree with you that the definition of the saHpiSessionOpen lacks
some necessary return values. 
And I don't think OpenHPI really should be limited in this case, since
the listed return values are not enough.
 
So I would suggest, you rise that issue to the SA Forum, so they will
decide on the return values that should be used,
e.g. when the library cannot connect to a domain or has an internal
error - or maybe something else.
Probably they will follow what OpenHPI has currently implemented.
 
When you go to www.saforum.org, you can click on "SPECIFICATIONS" and
then see a button on the right: 
Provide Feedback on Specifications.
 
Cheers,
Uli
 
 


________________________________

        From: ext Thompson, Michael
[mailto:[email protected]] 
        Sent: Thursday, June 10, 2010 4:23 PM
        To: [email protected]
        Subject: [Openhpi-devel] Missing return value(s) for
saHpiSessionOpen()
        
        

        The description of the possible return values for
saHpiSessionOpen() in SaHpi.h is missing SA_ERR_HPI_NO_RESPONSE.

         

        From my very limited understanding of the code in
oh_client_session.cpp it looks like SA_ERR_HPI_INTERNAL_ERROR is also a
possible return value.

         

        Michael Thompson, Principal Engineer

        [email protected]
<mailto:[email protected]> 

        Tel: +1 401 535 4869

        Warwick, RI

         

         

<<image001.gif>>

------------------------------------------------------------------------------
ThinkGeek and WIRED's GeekDad team up for the Ultimate 
GeekDad Father's Day Giveaway. ONE MASSIVE PRIZE to the 
lucky parental unit.  See the prize list and enter to win: 
http://p.sf.net/sfu/thinkgeek-promo
_______________________________________________
Openhpi-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openhpi-devel

Reply via email to