9736
33%

Ran 31 Aug 2018 08:16AM UTC

Jobs 1

Files 431

Run time 30s

Badge

Embed ▾

pending completion

Build # 9736

Build Type

push

travis-ci

Committed by

kevingranade

Commit Message

Code optimizations reported by static code analysis (2018-08-30) (#25257)

* Removed unnecessary ternary

`relevant` is always true due to following code block above:

```CPP
    if ( !relevant ) {
        relevant = &pc.weapon;
```

* Removed unused variable

* Removed implicit bool to float cast

* Fixed compare of int with float

* Simplified excessive check

```CPP
if( !A || ( A && B ) )
```

is equal to

```CPP
if( !A || B )
```

* Clarification comment

* Removed unused variable assignment

* Use pointers to avoid using expression repeatedly

Run Details

25333 of 109189 relevant lines covered (23.2%)

92977.54 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
2	9736.2 (COMPILER=g++-4.8 CODE_COVERAGE=true)	31 Aug 2018 08:16AM UTC	431	23.2	Travis Job 9736.2

CleverRaven / Cataclysm-DDA / 9736
33%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 9736

CleverRaven / Cataclysm-DDA / 9736 33%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 9736

CleverRaven / Cataclysm-DDA / 9736
33%

README BADGES
x