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

[coleen.phillimore at oracle.com]

Oh, ok.  Thanks for the explanation.  Your change looks complete to me then.
Coleen

On 2/13/18 2:11 AM, Marcus Larsson wrote:
> It's similar, but that one is using print() and not print_cr(), so it 
> actually needs the streams to keep everything on the same log line.
>
> Thanks,
> Marcus
>
> On 2018-02-12 17:38, coleen.phillimore at oracle.com wrote:
>>
>> 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
>>>>
>>>
>>
>