fix hotspot build failures with -D_FORTIFY_SOURCE=2 and -Wformat=1

[Andrew Haley]

Kelly O'Hair wrote:
> 
> 
> David Holmes - Sun Microsystems wrote:
>> Hi Matthias,
>>
>> Can't we get rid of all the dummy variables here:
>>
>>    count = sscanf(s,"%c %d %d %d %d %d %lu %lu %lu %lu %lu %lu %lu",
>> -                 &idummy, &idummy, &idummy, &idummy, &idummy, &idummy,
>> +                 &cdummy, &idummy, &idummy, &idummy, &idummy, &idummy,
>>                   &ldummy, &ldummy, &ldummy, &ldummy, &ldummy,
>>                   &user_time, &sys_time);
>>
>> by using the %*d %*c format specifiers to parse but not store the value.
>>
>> And here:
>>
>> -    ::write(defaultStream::output_fd(), s, (int)strlen(s));
>> +    ssize_t rv = ::write(defaultStream::output_fd(), s, (int)strlen(s));
>>
>> if we are ignoring the return value anyway (which might be the real
>> error here!) wouldn't a cast to void be a cleaner fix?
> 
> I tend to agree on the (void) cast, and seems like we run the risk of a
> 'local variable assigned to but not used' warning...   or do any C/C++
> compilers do that?

Yeah, they do.  gcc has attribute ((unused)) for unused decls,
which one might as well use in Linux-specific code.

Andrew.