One comment is that the usual pattern is to use `inherits` here so the rest of the module can reference `$xinetd::package_name` (etc.) instead of `$xinetd::params::package_name` and parameters can be added to the base class without editing the code elsewhere in the module.