Task #1479: Core get ride of "extra ;" warning with pedantic mode - U++ - Redmine

Task #1479

Core get ride of "extra ;" warning with pedantic mode

Added by Zbigniew Rebacz almost 9 years ago. Updated over 4 years ago.

Status:

Patch ready

Start date:

06/25/2016

Priority:

Low

Due date:

Assignee:

Miroslav Fidler

% Done:

Category:

Core

Spent time:

Target version:

Release 2021.1 - NTH

Description

Compile with GCC and -Wpedantic to detects all warning related to "extra ;"

uppsrc.diff - Proposed patch for uppsrc directory (since 13633 revision) (47 KB) Sender Ghost, 10/08/2019 04:59 PM

uppsrc_excluded_macros.diff - Proposed patch for uppsrc directory (excluded changes for macros, since 13635 revision) (18 KB) Sender Ghost, 10/09/2019 01:41 PM

History

#1 Updated by Zbigniew Rebacz almost 9 years ago

Tracker changed from Bug to Task

#2 Updated by Zbigniew Rebacz about 8 years ago

Target version changed from Release 2017.1 - NTH to Release 2017.2 - NTH

#3 Updated by Zbigniew Rebacz over 7 years ago

Target version changed from Release 2017.2 - NTH to Release 2018.1 - NTH

#4 Updated by Zbigniew Rebacz about 7 years ago

Target version changed from Release 2018.1 - NTH to Release 2019.1 - NTH

#5 Updated by Zbigniew Rebacz almost 6 years ago

Target version changed from Release 2019.1 - NTH to Release 2019.2 - NTH

#6 Updated by Sender Ghost over 5 years ago

File uppsrc.diff added

The GCC 8.3.0 compiler identified following cases with extra ';' character:

namespace * {*};
extern "C" {*};
INITIALIZE(*);
INITBLOCK {*};
rval_default(*);
CH_VALUE(*);
CH_IMAGE(*);
CH_COLOR(*);
CH_INT(*);
NTL_MOVEABLE(*);
SVO_FN(*);
VALUE_XMLIZE(*);
__Expand(*);
INI_TYPE(*);
INI_BOOL(*);
INI_INT(*);
INI_STRING(*);

and where INI_TYPE define was used manually:

INI_INT64(*);
INI_DOUBLE(*);

Patch attached.
Please use "svn patch" (or patch, etc.) utility to apply patches, because current (at 13633 revision) version of TheIDE's "Assist -> Display/apply patch.." function doesn't recognize "\ No newline at end of file" line(s) in patches, created by diff, "git diff" or "svn diff" programs.

#7 Updated by Sender Ghost over 5 years ago

Status changed from New to Patch ready

#8 Updated by Miroslav Fidler over 5 years ago

Yeah, well, but I would like to actually have ';' after many of these macros, for two reasons:

- it is more readable (; being a natural item ending character in C++)
- futureproofing - it is not unlikely that some day in the future implementation changes. well, unlikely, but still

Maybe we should instead find a way how to require ';' there?