[elbe-devel] [PATCH 2/5] Add ISO files options for source cdroms

Olivier Dion dion at linutronix.de
Wed Jul 17 10:49:56 CEST 2019


On 2019-07-17T09:38:09+0200, Kurt Kanzenbach wrote:

> Why not use hashes consistently? I'd prefer something like

>  str_type = encoding[iso_options[opt_name]["str_type"]]

> > +    if len(text) > iso_options[opt_name][1]:

>  if len(text) > iso_options[opt_name]["len"]:

> and so on. It makes it easier to read.

Old C habit I guess.  I can either change it to a dict if you want.

--
Olivier Dion
Linutronix GmbH | Bahnhofstrasse 3 | D-88690 Uhldingen-Mühlhofen



More information about the elbe-devel mailing list