----- Original Message -----

> In devassistant/config_manager.py:
> > +    launches of devassistant GUI.
> > +    It provides saving and loading configuration values from a file."""
> > +
> > +    def __init__(self):
> > +        self.config_dict = dict()
> > +        self.config_file = settings.CONFIG_FILE
> > +        self.config_changed = False
> > +
> > +    def load_configuration_file(self):
> > +        """Load all configuration from file
> > +        """
> > +        if not os.path.exists(self.config_file):
> > +            return
> > +        with open(self.config_file,'r') as file:
> > +            for line in file:
> > +                key, sep, value = line.rstrip('\n').partition('=')

> Eh... To be honest, I really did not expect there would ever be any parameter
> with "=" in its name. Do you expect so?
> Actually, many common configuration files use the pattern "key=value". I just
> looked into the /etc directory into the .conf files and majority of them
> really used "=" as a separator. So why complicate things and not follow the
> usual and widely-used style? I used the csv library to read and write the
> lines (so they will take care about "=" in both key and value) but used the
> "=" delimiter anyway.
Just to make this clear: The point I was trying to make here is that if you use your own solution, then no matter how simple it seems, you'll almost always forget about some corner cases - this one is far fetched, but maybe there are some others that neither of us though of. That's why I requested use of csv or other standardized modules - a tested solution that many people use and has proven to be functioning well. 

Hope this makes sense. 

> Btw I was thinking about adding support for comments, adding check for "#" in
> the beginning of line. But it seemed rather pointless because the file is
> always automatically re-generated and is actually not expected to be ever
> hand-edited.

> —
> Reply to this email directly or view it on GitHub .
-- 
Regards, 
Bohuslav "Slavek" Kabrda.