<HTML><HEAD>
<META http-equiv=Content-Type content='text/html; charset=windows-1252'>
<title>Samsung Enterprise Portal mySingle</title>
<style> P, td, li {font-family:Arial, arial; font-size:9pt; margin-top:5px;margin-bottom:5px;} body{font-family:Arial, arial; font-size:9pt;}</style>
</HEAD><BODY><br><p> hi guys,</p>
<p>i m using DBUS daemon for our p2p communication but it seems to me that it
is bulky and latency is not predictable at heavy load condition what u people
think i m correct or not on the basis of that if it will be used for some other
constrained platform like mobile (which is running a version of linux kernel)it
will not perform properly or provide system slow responses etc. what you think
correct me if i m thinking in a wrong way.</p>
<p>looking for all listeners views.</p>
<p>thanks & regards,</p>
<p>deepesh</p>
<p> </p><br><br>------- <b>Original Message</b> -------<br><b>Sender</b> : Peter McCurdy<petermccurdy@alumni.uwaterloo.ca><br><b>Date</b> : Aug 26, 2008 11:36 (GMT+09:00)<br><b>Title</b> : Re: Deciding whether time values should be signed or unsigned<br><br>On Mon, Aug 25, 2008 at 2:50 PM, Havoc Pennington <hp@pobox.com> wrote:
<br>> Hi,
<br>>
<br>> On Mon, Aug 25, 2008 at 2:00 PM, Thiago Macieira <thiago@kde.org> wrote:
<br>>> You should use time_t, struct timeval, etc. internally, everywhere when
<br>>> dealing with the system calls. Don't try to invent your own time value.
<br>>>
<br>>> I also don't see the need to have sec/usec pairs everywhere. A struct would
<br>>> probably be better, and we have "timeval" for that. I understand the aversion
<br>>> of using any system headers in files that aren't "sysdep", so we could define
<br>>> our own structure.
<br>>
<br>> Cleaning up the internal API would not be bad, I think it's a separate
<br>> patch from just fixing the warnings though. (Meaning both, ideally it
<br>> is a separate patch for purposes of git history and patch review, and
<br>> that it's OK to go ahead and fix the warnings without a larger
<br>> refactoring)
<br>>
<br>> If cleaning up the internal API, I would be reluctant to use struct
<br>> timeval and time_t outside sysdeps as you say.
<br>
<br>You'd also want some sensible functions to add, subtract, compare,
<br>etc. these time values. The lack of these is part of what makes the
<br>current code bulkier than it needs to be.
<br>
<br>If you're going to make your own time struct anyway, you may want to
<br>consider just using a 64-bit count of microseconds, or something
<br>similar. You'd get the same precision as you currently have, have a
<br>useful range (+/- 300,000 years), not have to deal with creating
<br>structs and passing them around, and have an easier go with the time
<br>arithmetic. The drawback would be having to do some conversion
<br>whenever you interact with the outside world. Anyway, not really
<br>relevant right at the moment, just something to consider.
<br>
<br>>> I understand that the last two points don't apply to the patches you have, but
<br>>> they come as indications that signed is preferred.
<br>>
<br>> Right
<br>>
<br>
<br>Thiago's points make a lot of sense. So that makes it roughly 3-0 in
<br>favour of the signed patch, with justification and everything.
<br>
<br>Does anyone have any comments about the actual patch itself?
<br>
<br>Thanks.
<br>
<br>Peter.
<br>_______________________________________________
<br>dbus mailing list
<br>dbus@lists.freedesktop.org
<br>http://lists.freedesktop.org/mailman/listinfo/dbus
<br><p> </p><p> </p><!--SP:deepesh.t--><!--deepesh.t:EP--><p> </p><p> </p></BODY></HTML>