Re: [boost] [PATCH 1/2] Fixed warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)'

Subject: Re: [boost] [PATCH 1/2] Fixed warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)'
From: Steven Watanabe (watanabesj_at_[hidden])
Date: 2010-12-29 23:54:21

AMDG

On 12/21/2010 1:50 AM, Vasyl' wrote:
> diff --git a/boost/uuid/seed_rng.hpp b/boost/uuid/seed_rng.hpp
> index 3090197..4196536 100644
> --- a/boost/uuid/seed_rng.hpp
> +++ b/boost/uuid/seed_rng.hpp
> @@ -150,7 +150,8 @@ private:
>
>              if(random_)
>              {
> -                std::fread( buffer, 1, 20, random_ );
> +                size_t rnd_read_size = std::fread( buffer, 1, 
> sizeof(buffer), random_ );
> +                BOOST_ASSERT(rnd_read_size == sizeof(buffer));
>              }
>
>              // using an uninitialized buffer[] if fopen fails

If we're going to be picky about warnings at this
level, this will produce an unused variable warning
when asserts are disabled.

In Christ,
Steven Watanabe