Skip to content

Daniel Marjamäki
danmar

  • Sweden, Stockholm
  • Joined on
@danmar
@danmar
  • @danmar 6c32401
    Use MAXTIME in templatesimplifier
@danmar
  • @danmar 087b233
    Daca2: don't skip flite anymore. Analysis without MAXTIME is still ve…
@danmar
danmar commented on pull request danmar/cppcheck#771
@danmar

fair enough.. I don't volounteer to profile it.. so feel free to push it.

danmar commented on pull request danmar/cppcheck#771
@danmar

This looks good. Feel free to push it. I suggest that: const Token *index = tok->next()->astOperand2(); if (... && index->getValueGE(0x80, _setting…

@danmar
@danmar
@danmar
  • @danmar 1ab5805
    Fixed #6616 (valueFlowForward: for loop, variable in 3rd expression i…
@danmar
  • @danmar ec9facb
    rundaca: allow that start folder is set on command line, can be used …
@danmar

yes thanks.. I don't like those neither.

@danmar
@danmar
  • @danmar 34b5e0c
    Fixed #6873 (False positive uninitvar - variable initialized via poin…
@danmar
  • @danmar 5fef7cc
    Fixed #4973 (wrong enum simplification of shadow struct variable)
@danmar

sorry for late reply.. in case from line 124 we know the value of the char variable. It is ok to write the message when the char variable has a dan…

@danmar
  • @danmar 6c1012a
    Fixed #4625 (wrong enum simplification in shadowed enum declaration)
@danmar
  • @danmar f781f13
    Fixed #6806 (wrong enum simplification in initialization list)
@danmar
  • @danmar 42278dd
    Tokenizer: skip constant folding in normal token list for C code. In …
@danmar
  • @danmar 7bd034c
    Fixed #7369 (False positive knownConditionTrueFalse - assertions)
danmar commented on commit danmar/cppcheck@618ea498e9
@danmar

I tried a quick fix for this .. but we then get FN for such C code: const int *p = n > 4; // <- should warn about assigning bool to pointer

danmar commented on commit danmar/cppcheck@618ea498e9
@danmar

ok that is wrong behaviour :-( should be fixed.

@danmar

we don't warn about division by zero / array index out of bounds / null pointer dereference / etc using function parameter unless we can see bad fu…

@danmar
danmar commented on commit danmar/cppcheck@35317695d4
@danmar

I would prefer {} around this for loop body.

@danmar
  • @danmar b45c285
    daca2: changed checking of test code again. people might want to run …
@danmar
  • @danmar 1d9537b
    daca2: don't run cppcheck on test code, that is rarelly meant to be v…
@danmar
  • @danmar 07f3930
    report progress in TemplateSimplifier::simplifyTemplateInstantiations…
danmar commented on pull request danmar/cppcheck#768
@danmar

I don't see any benefit in those CWE-constant variables - but I guess now it's too late for that comment Right now the benefit is that it's more…

@danmar

if foo is called like this , then there is no bug: foo('x');

@danmar

ok I buy this

Something went wrong with that request. Please try again.