RFR: 8141211: Convert TraceExceptions to Unified Logging

[Rachel Protacio]

Oh actually that was my fault! I cleaned up the code a bit and left that 
sitting around. Thanks for catching that - I'll remove it.
Rachel

On 12/8/2015 1:35 PM, harold seigel wrote:
> Hi Max,
>
> Is "int j = 0;" needed in lookup_logging_aliases() ?
>
> *+ bool Arguments::lookup_logging_aliases(const char* arg, char* 
> buffer) {*
> *+ int j = 0;*
> *+ for (size_t i = 0; aliased_jvm_logging_flags[i].alias_name != NULL; 
> i++) {*
> *+ const AliasedFlag& flag_status = aliased_jvm_logging_flags[i];*
> *+ if (strcmp(flag_status.alias_name, arg) == 0) {*
> *+ strcpy(buffer, flag_status.real_name);*
> *+ return true;*
> *+ }*
> *+ }*
> *+ return false;*
> *+ } *
>
> Thanks, Harold
>
>
>
> On 12/8/2015 10:42 AM, Rachel Protacio wrote:
>> Hello,
>>
>> Please review my conversion of -XX:+TraceExceptions to 
>> -Xlog:exceptions=info. The existing (product) flag is aliased to the 
>> logging flag at the info level.
>>
>> If you have any questions on the alias table (in the arguments.cpp 
>> and .hpp files), Max will chime in as he is the one who implemented 
>> that portion.
>>
>> Open webrev: http://cr.openjdk.java.net/~rprotacio/8141211/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8141211
>>
>> Testing: jtreg, JPRT, jck vm tests, refworkload performance tests, 
>> rbt quick & non-colo tests
>>
>> Thank you!
>> Rachel
>