FrequencyMap: is this correct or a bug?

[facontidavide]

I was looking at this code:

bool lama::FrequencyOccupancyMap::setOccupied(const Vector2ui& coordinates)
{
    // ...
    bool occupied = prob(*cell) > occ_thresh;
    cell->occupied++;
    cell->visited++;
    // ...
}

Shouldn't it be instead:

    bool occupied = prob(*cell) > occ_thresh;
    if( occupied ) cell->occupied++;
    cell->visited++;
}

[facontidavide]

oh, I think I figured it out, never mind

[eupedrosa]

I was writting my reply to you, but it seems that it solved by it self :)