google / riegeli

Riegeli/records is a file format for storing a sequence of string records, typically serialized protocol buffers.
Apache License 2.0
418 stars 53 forks source link

base/buffer.h: Type qualifiers ignored on function return type #4

Closed CodeArno closed 5 years ago

CodeArno commented 5 years ago

Hi,

Compiling Riegeli in our project setup creates the following GCC warning/error: riegeli/base/buffer.h:52:3: type qualifiers ignored on function return type [-Werror=ignored-qualifiers]

The reason is that the first const in the line has no effect, since it returns by value: const size_t size() const { return size_; }

It would be great if you could just merge it - I've created https://github.com/google/riegeli/pull/3, but Googlebot doesn't like me. :)

Thanks, Arno

QrczakMK commented 5 years ago

Thank you!

This is now fixed: https://github.com/google/riegeli/commit/29d474f38bffcb2e503cd7f9f757a413e89ab4e8

On Fri, 9 Aug 2019 at 15:34, Arno Fleck notifications@github.com wrote:

Hi,

Compiling Riegeli in our project setup creates the following GCC warning/error: riegeli/base/buffer.h:52:3: type qualifiers ignored on function return type [-Werror=ignored-qualifiers]

The reason is that the first const in the line has no effect, since it returns by value: const sizet size() const { return size; }

It would be great if you could just merge it - I've created #3 https://github.com/google/riegeli/pull/3, but Googlebot doesn't like me. :)

Thanks, Arno

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/google/riegeli/issues/4?email_source=notifications&email_token=ACAUJJAHUCSYS6DKXZM7QUTQDVW6HA5CNFSM4IKUCLA2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HEMZZPA, or mute the thread https://github.com/notifications/unsubscribe-auth/ACAUJJA3CDWYI3UV5FD5EX3QDVW6HANCNFSM4IKUCLAQ .

-- Marcin Kowalczyk