Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compile warning under linux gcc 4.8.4 #53

Closed
tpeterka opened this issue Jan 26, 2018 · 2 comments
Closed

Compile warning under linux gcc 4.8.4 #53

tpeterka opened this issue Jan 26, 2018 · 2 comments

Comments

@tpeterka
Copy link
Member

tpeterka commented Jan 26, 2018

In file included from /homes/tpeterka/software/diy/include/diy/collection.hpp:7:0,
                 from /homes/tpeterka/software/diy/include/diy/master.hpp:12,
                 from /homes/tpeterka/software/diy/tests/swap-reduce.cpp:10:
/homes/tpeterka/software/diy/include/diy/storage.hpp: In member function ‘virtual void diy::detail::FileBuffer::load_binary_back(char*, size_t)’:
/homes/tpeterka/software/diy/include/diy/storage.hpp:31:134: warning: ignoring return value of ‘size_t fread(void*, size_t, size_t, FILE*)’, declared with attribute warn_unused_result [-Wunused-result]
       virtual inline void load_binary_back(char* x, size_t count) override    { fseek(file, tail, SEEK_END); fread(x, 1, count, file); tail += count; fseek(file, head, SEEK_SET); }
                                                                                                                                      ^
/homes/tpeterka/software/diy/include/diy/storage.hpp: In member function ‘virtual void diy::FileStorage::get(int, diy::MemoryBuffer&, size_t)’:
/homes/tpeterka/software/diy/include/diy/storage.hpp:120:41: warning: ignoring return value of ‘ssize_t read(int, void*, size_t)’, declared with attribute warn_unused_result [-Wunused-result]
         read(fh, &bb.buffer[0], fr.size);
@mathstuf
Copy link
Contributor

mathstuf commented Feb 1, 2018

See #57 for more detail.

@mrzv
Copy link
Member

mrzv commented Feb 1, 2018

I'm going to close this in favor of #57.

@mrzv mrzv closed this as completed Feb 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants