Hi core-libs-dev,

Here's another approach to just add a defined(AIX) check to make sure it works on Aix platform.
http://cr.openjdk.java.net/~luchsh/timezone_md_ret_check/

Any suggestions?

Cheers,
- Jonathan

On 11/02/2011 04:56 PM, Jonathan Lu wrote:
Hi core-libs-dev,

In jdk/src/solaris/native/java/util/TimeZone_md.c, starting from line 626, I found that the scope of "#ifdef __solaris__" might be too narrow, since it also works for some kind of OS which I'm currently working on, such as AIX. So I suggest to just remove the '#ifdef __solaris__' and leave the "#else" to accommodate more conditions, see attachment 'patch.diff'. I think this may enhance the cross-platform ability, any ideas about this modification?

Regards
- Jonathan Lu

Reply via email to