Opened 2 years ago

Closed 2 years ago

#859 closed defect (fixed)

turn on all warnings when compiling rasdaman

Reported by: dmisev Owned by: dmisev
Priority: major Milestone: 9.0.x
Component: build system Version: development
Keywords: Cc: pbaumann
Complexity: Medium

Description

We should add a --enable-strict option to the configure script, which will add g++ options that turn on warnings and make them errors that halt compilation.

This option should then be regularly used by developers, once the code is cleaned from current warnings.

Change History (4)

comment:1 Changed 2 years ago by pbaumann

should we make this behavior default maybe?

comment:2 Changed 2 years ago by dmisev

It's a bad idea from a user's point of view. Even if we fix all warnings on our compiler versions, users may have different (usually newer) that throw new warnings and fail their make. But users usually don't know about the configure option to disable it, so it's not very user-friendly.

I think it's better that developers are aware of it and consciously turn it on.

comment:3 Changed 2 years ago by pbaumann

hm, from the perspective of gcc changing behavior more often than one would think this is a good thought indeed. Hence, agreeing with you.

comment:4 Changed 2 years ago by dmisev

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.