Overview
Request 816915 superseded
- Update to 1.5.2:
* Fix #83, output formats "sed" and "gsed" to auto-fix some issues
* Fix #92, new category "build/namespaces_headers" for unnamed namespaces in header file
* Sort list of files before processing
* Fix #144 Falso positive for indent when using QT macros "signals" and "slots"
* Fix #76 Parsing of class decorators that also use digits
* Fix #139 Add message "Relative paths like . and .. are not allowed"
* Revert #43 behavior change for include order from 1.5.0, and hide it behind command-line-flag --includeorder=standardcfirst. It turns out there is no easy objective way to tell c system headers from certain c++ library headers, and Google cpplint intentionally classifies some C++ header includes as C system header for simplicity.
* Libraries considered as C system headers using --includeorder=standardcfirst now also includes linux-specifc headers (glibc-devel, glibc-kernheaders, linux-libc-dev).
* Fix #43 false positives in header include order by checking includes against a list of c headers. Since this interprets certain include lines different than before, output about header include order changes.
* Fix #135: allow 'if constexpr' in readability/braces.
* Fix runtime warning: Close files after reading contents
- Created by StevenK
- In state superseded
- Superseded by 817961
- Open review for factory-staging
Request History
StevenK created request
- Update to 1.5.2:
* Fix #83, output formats "sed" and "gsed" to auto-fix some issues
* Fix #92, new category "build/namespaces_headers" for unnamed namespaces in header file
* Sort list of files before processing
* Fix #144 Falso positive for indent when using QT macros "signals" and "slots"
* Fix #76 Parsing of class decorators that also use digits
* Fix #139 Add message "Relative paths like . and .. are not allowed"
* Revert #43 behavior change for include order from 1.5.0, and hide it behind command-line-flag --includeorder=standardcfirst. It turns out there is no easy objective way to tell c system headers from certain c++ library headers, and Google cpplint intentionally classifies some C++ header includes as C system header for simplicity.
* Libraries considered as C system headers using --includeorder=standardcfirst now also includes linux-specifc headers (glibc-devel, glibc-kernheaders, linux-libc-dev).
* Fix #43 false positives in header include order by checking includes against a list of c headers. Since this interprets certain include lines different than before, output about header include order changes.
* Fix #135: allow 'if constexpr' in readability/braces.
* Fix runtime warning: Close files after reading contents
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
dimstar_suse added as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:58"
dimstar_suse accepted review
Picked "openSUSE:Factory:Staging:adi:58"
dimstar_suse added factory-staging as a reviewer
Being evaluated by group "factory-staging"
dimstar_suse accepted review
Unstaged from project "openSUSE:Factory:Staging:adi:58"
dimstar_suse declined review
sr#816915 has same source and is already staged
dimstar_suse declined request
sr#816915 has same source and is already staged
dimstar_suse reopened request
Thanks OBS for going to reboot mids of a staging action
dimstar_suse added as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:85"
dimstar_suse accepted review
Picked "openSUSE:Factory:Staging:adi:85"
dimstar accepted review
dimstar declined review
FTBFS
dimstar declined request
FTBFS
dimstar_suse reopened request
Reopened via staging workflow.
dimstar_suse added factory-staging as a reviewer
Being evaluated by group "factory-staging"
dimstar_suse accepted review
Unstaged from project "openSUSE:Factory:Staging:adi:85"
dimstar_suse declined request
Declined via staging workflow.
superseded by 817961