diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2013-08-15 19:53:11 +0200 |
|---|---|---|
| committer | Julien Nabet <serval2412@yahoo.fr> | 2013-08-15 19:53:11 +0200 |
| commit | 3b51ba24bed97d9f40a0b0e3e3e1c3ae15db1629 (patch) | |
| tree | 478562a855aeb2d2311e4f2906dd9b7c6de538d3 /padmin | |
| parent | 507169c13a95705dc8a82dc8f4ca7172c8d671ac (diff) | |
cppcheck: fix duplicateBranch
Git history shows it was like this since first commit (2009-12-15)
Moreover I noticed these lines:
220 if ( mpCGM->pElement->nAspectSourceFlags & ASF_FILLCOLOR )
221 nFillColor = mpCGM->pElement->pFillBundle->GetColor();
222 else
223 nFillColor = mpCGM->pElement->aFillBundle.GetColor();
even if nFillColor can have another value in case below
249 case FIS_GEOPATTERN :
250 {
251 if ( mpCGM->pElement->eTransparency == T_ON )
252 nFillColor = mpCGM->pElement->nAuxiliaryColor;
253 eFS = drawing::FillStyle_NONE;
254 }
this change is still safe since it's just a simplification.
Change-Id: Icf41dbeee6405780483649e0968dd30e8a533882
Diffstat (limited to 'padmin')
0 files changed, 0 insertions, 0 deletions