Thanks for reviewing, Danek!
See below.
April
On 2/9/2012 4:08 PM, Danek Duvall wrote:
april.c...@oracle.com wrote:
Could I please get a code review to fix:
7127211 problem with ruby
http://monaco.us.oracle.com/detail.jsf?cr=7127211
The webrev is here:
http://http://jurassic.us.oracle.com/net/sfwcfi/builds/apchin/s11-u1-ruby/webrev-1/
02-code_objects-html_generator.patch:
- Is this affected by the locale? That is, do you get a different result
if you use C rather than en_US.UTF-8? That's definitely been an issue
for sorting in the past.
Good question. I'll check this out and let you know.
- line 34, 46: I wouldn't keep commented-out code in the file.
Okay, I'll remove the commented out old code.
Danek
_______________________________________________
userland-discuss mailing list
userland-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/userland-discuss
