Re: [devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs [#580]

2013-12-02 Thread Anders Widell
1 PM >> To: Ramesh Babu Betham >> Cc: opensaf-devel@lists.sourceforge.net >> Subject: [devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs >> [#580] >> >> osaf/libs/core/common/Makefile.am |2 + >&g

Re: [devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs [#580]

2013-11-29 Thread Anders Widell
if (result == 1) { > > Just print fd, revents, strerror(errno) > > Cheers, > Mathi. > >> -Original Message- >> From: Anders Widell [mailto:anders.wid...@ericsson.com] >> Sent: Tuesday, October 08, 2013 3:51 PM >> To: Ramesh Babu Betham >>

Re: [devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs [#580]

2013-11-29 Thread Mathivanan Naickan Palanivelu
y, October 08, 2013 3:51 PM > To: Ramesh Babu Betham > Cc: opensaf-devel@lists.sourceforge.net > Subject: [devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs > [#580] > > osaf/libs/core/common/Makefile.am |2 + > osaf/libs/core/common/include/osaf_poll.

Re: [devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs [#580]

2013-10-30 Thread Anders Widell
2013-10-30 12:49, Ramesh Betham skrev: > Hi Anders, > > >> [Ramesh]: Above function should take care of comparing i_end and > i_start tmr values and should take appropriate action. > > [AndersW]: Do you mean that the function... > > For the above comment I'll make a direct statement., what i

Re: [devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs [#580]

2013-10-30 Thread Ramesh Betham
Hi Anders, >> [Ramesh]: Above function should take care of comparing i_end and i_start tmr values and should take appropriate action. > [AndersW]: Do you mean that the function... For the above comment I'll make a direct statement., what if user gives "i_end" timer values lesser-than "i_

Re: [devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs [#580]

2013-10-30 Thread Anders Widell
Hi! Thanks for the comments. Se my replies inline below. regards, Anders Widell 2013-10-30 10:00, Ramesh Betham skrev: > Hi Anders, > > I am not sure on the motivation behind adding more utilities to > core/leap area. > > Also to what extent all these utility functions are used by OpenSAF > se

Re: [devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs [#580]

2013-10-30 Thread Ramesh Betham
Hi Anders, I am not sure on the motivation behind adding more utilities to core/leap area. Also to what extent all these utility functions are used by OpenSAF services?. Because on behalf of ticket# 537, I am cleaning up the unused code from leap area and adding more utilities with out actuall

[devel] [PATCH 1 of 1] base: Add osaf_poll.h and osaf_time.h APIs [#580]

2013-10-08 Thread Anders Widell
osaf/libs/core/common/Makefile.am |2 + osaf/libs/core/common/include/osaf_poll.h | 97 +++ osaf/libs/core/common/include/osaf_time.h | 371 ++ osaf/libs/core/common/osaf_poll.c | 130 ++ osaf/libs/core/common/osaf_time.c |