To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=74838





------- Additional comments from [EMAIL PROTECTED] Mon Feb 26 09:29:55 +0000 
2007 -------
bridges/source/remote/urp/urp_log.cxx:1.4.42.1:  For one, C-style casts are
frowned upon (e.g., Item 95 of Sutter/Alexandrescu "C++ Coding Standards"), and
for another, there might be more sensible printf type specifiers to use.  For
example,

  "%d", (int) osl_getGlobalTimer()

would probably better be

  "%u", static_cast< unsigned int >(osl_getGlobalTimer())

(The

  "%lu", static_cast< unsigned long >(osl_getGlobalTimer())

attempt at max portability might be considered overkill, depending on personal
taste.)

The rest looks fine.

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to