RFR(S): 8194994: Remove redundant string streams used for logging

[coleen.phillimore at oracle.com]

I found another one of these in instanceKlass.cpp in 
print_class_load_logging.  Should this be fixed in the same way?

Thanks,
Coleen

On 2/12/18 6:40 AM, Marcus Larsson wrote:
> Hi Ioi,
>
>
> On 2018-02-12 12:06, Ioi Lam wrote:
>> Hi Marcus,
>>
>> Looks good. Some small nits:
>>
>> compactHashtable.cpp: copyright date
>>
>> filemap.cpp: the indentation  needs to be fixed:
>>
>>  414  LogMessage(cds) msg;
>>  415   if (msg.is_info()) {
>>  416     msg.info("Dumping shared data to file: ");
>>  417     msg.info("   %s", _full_path);
>>  418   }
>
> Good catch, will fix before pushing!
>
> Thanks for reviewing!
> Marcus
>
>>
>> Thanks
>> - Ioi
>>
>> On 2/11/18 11:13 PM, Marcus Larsson wrote:
>>> Hi,
>>>
>>> Please review the following patch to remove some redundant uses of 
>>> string streams used for logging.
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~mlarsson/8194994/webrev.00/
>>>
>>> Issue:
>>> https://bugs.openjdk.java.net/browse/JDK-8194994
>>>
>>> Tested with hs-tier 1-2.
>>>
>>> Thanks,
>>> Marcus
>>
>