Rudder

I falled on that line of code:

https://github.com/Normation/rudder/blob/master/rudder-core/src/main/scala/com/normation/rudder/services/reports/ReportingServiceImpl.scala#L105

....
        case Some(serial) if ((serial == newSerial)&&(configs.size > 0)) =>
            // no change if same serial and some config appliable
            logger.debug("Same serial %s for ruleId %s, and configs presents".format(serial, ruleId))
            currentConfigurationsToRemove.remove(ruleId)

        case Some(serial) if ((serial == newSerial)&&(configs.size == 0)) => // same serial, but no targets
            // if there is not target, then it need to be closed
          logger.debug("Same serial, and no configs present")

        case Some(serial) => // not the same serial
          logger.debug("Not same serial")
            confToCreate += conf
            confToClose += ruleId

            currentConfigurationsToRemove.remove(ruleId)
      }

The second pasted case is strange, because the comment seems to say "No config present, I need to do something!" and then, nothing is done. Is that what is expected ?