I'd prefer having here a logfile_enabled = True|False for consistency (So far we don't enable/disable by commenting) but i don't see this as a show stopper