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

[Marcus Larsson]

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
>>>
>>
>