[igt-dev] [PATCH i-g-t v2 06/13] scripts/test_list.py: close config file before start processing

Kamil Konieczny kamil.konieczny at linux.intel.com
Thu Jul 13 10:20:13 UTC 2023


Hi Mauro,

On 2023-07-13 at 07:47:15 +0200, Mauro Carvalho Chehab wrote:
> On Wed, 12 Jul 2023 20:38:59 +0200
> Kamil Konieczny <kamil.konieczny at linux.intel.com> wrote:
> 
> > Hi Mauro,
> > 
> > On 2023-07-12 at 15:58:47 +0200, Mauro Carvalho Chehab wrote:
> > > From: Mauro Carvalho Chehab <mchehab at kernel.org>
> > > 
> > > Reduce the identation level of the code which parses the config
> > > file, as this can be done after closing the file.  
> > ----- ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > Where it is? Maybe should be dropped?
> > 
> > > 
> > > This is a preparation for the next patch that will offer a different
> > > way to pass the configuration.  
> > 
> > Could you change description? Now it is a little confusing,
> > you only change indentation but it state:
> > 
> > v2 06/13 scripts/test_list.py: close config file before start processing
> > ------------------------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > 
> > Maybe something like: change indentation of code
> 
> I opted to describe the functional change, and not the physical one.
> 
> This is something I don't like on python: changing indentation actuall
> changes the code itself, as it doesn't have explicit block markups like
> C's { } or Pascal's begin ... end. 
> 
> What was happening before was equivalent to this (in C code format):
> 
> 	fp = fopen(...)
> 	if (!fp) exit();	// implicit, as this is python normal behavior
> 	{
> 		fread()
> 		// do a lot of things (unrelated to fp itself)
> 	}
> 	fclose(fp)
> 
> After the change, the code is it now:
> 
> 	fp = fopen(...)
> 	if (!fp) exit();
> 	fread()
> 	fclose(fp)
> 	// do a lot of things
> 	
> Yeah, as a side effect, we have one less level of indentation, which is
> indeed a good thing.
> 
> Regards,
> Mauro

Thank you for explanation, I missed this in those massive un-indentation.

Regards,
Kamil



More information about the igt-dev mailing list