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





------- Additional comments from [EMAIL PROTECTED] Mon Jan 30 01:56:03 -0800 
2006 -------
mox: the correct line would be

fprintf( stderr, "getWindowManagerName(): \"%s\" \n",
rtl::OUStringToOString( pSalDisplay->getWMAdaptor()->getWindowManagerName(),
osl_getThreadTextEncoding() ).getStr() );

also please surround the creation of MACOSXIntegrator with #ifdef MACOSX like
you did with the #include directive, else the code will not compile on not
MacOSX. Also you might want to make the compilation of macosxint.cxx conditional
for MacOSX (like the compilation for CDEIntegrator is conditional for Solaris,
please have a look at makefile.mk

Aside from these minor technicalities the patch looks good to me, if you want to
go with an RC file. Are there possibly methods to ask the Mac API for these
colors instead of using a file ?

---------------------------------------------------------------------
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