Cleanup codecheck to be compatible with clang-format

Bug #1610731 reported by GunChleoc
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Undecided
Unassigned

Bug Description

After running clang-format-3.8 over the code base, some codecheck errors get triggered. We need to cleanup the codecheck rules.

The lines triggered are:

src/logic/backtrace.cc:39: A space is mandatory after '*' or '&' in access type declaration.
src/editor/ui_menus/player_menu.h:52: A space is mandatory after '*' or '&' in access type declaration.
src/logic/map_objects/tribes/production_program.cc:70: A space is mandatory after '*' or '&' in access type declaration.
src/logic/map_objects/tribes/production_program.cc:116: A space is mandatory after '*' or '&' in access type declaration.
src/editor/tools/tool.h:105: A space is mandatory after '*' or '&' in access type declaration.
src/economy/test/test_routing.cc:545: A space is mandatory after '*' or '&' in access type declaration.
src/io/fileread.cc:99: A space is mandatory after '*' or '&' in access type declaration.

This happens with pointer variable declaration.

GunChleoc (gunchleoc)
summary: - Cleanup codecheck fo be compatible with clang-format
+ Cleanup codecheck to be compatible with clang-format
GunChleoc (gunchleoc)
Changed in widelands:
status: New → Confirmed
milestone: none → build20-rc1
Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed by automating clang-format.

tags: added: codecheck
Changed in widelands:
status: Confirmed → Fix Committed
Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed in build20-rc1

Changed in widelands:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.