code checker

Bug #553197 reported by Victor Pelt
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Medium
SirVer

Bug Description

Code checker doesn't seem to be aware of #define macro's
see the following code from warehouse.cc (around line 288)

#define SET_WORKER_WITHOUT_COST_SPAWNS(nr, value) \
   for \
      (wl_index_range<uint32_t *> \
       i( \ <- this line gives an error about trailing whitespaces
          m_next_worker_without_cost_spawn, \
          m_next_worker_without_cost_spawn + nr \
  ); \
       i; \
       ++i) \
      *i.current = value; \

Revision history for this message
SirVer (sirver) wrote :

which codechecker? there are two of them

Revision history for this message
Victor Pelt (victor-pelt) wrote :

the one accessed from cmake/make codecheck
(qcs cmake branch)

Revision history for this message
Jens Beyer (qcumber-some) wrote :

This is the Python codechecker, merge lp:~qcumber-some/widelands/spice-up-cmake into your trunk if you want to have it.

Revision history for this message
SirVer (sirver) wrote :

I will care for this as soon as it is in trunk. Will not have much time next week.

Changed in widelands:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → SirVer (sirver)
milestone: none → build16-rc1
Revision history for this message
SirVer (sirver) wrote :

Oh, well I just fixed it. trunk 5186

Changed in widelands:
status: Confirmed → Fix Committed
milestone: build16-rc1 → build15
Revision history for this message
SirVer (sirver) wrote :

Released in build-15

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.