Re: [Powertop] [PATCH v2 8/8] Adding a clean_shutdown function

[Sergey Senozhatsky <sergey.senozhatsky at gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 1107 bytes --]

On (10/14/14 11:09), Joe Konno wrote:
>  src/main.cpp | 15 ++++++++++++---
>  1 file changed, 12 insertions(+), 3 deletions(-)
> 
> diff --git a/src/main.cpp b/src/main.cpp
> index d3963ed..b6ce090 100644
> --- a/src/main.cpp
> +++ b/src/main.cpp
> @@ -368,6 +368,17 @@ static void powertop_init(void)
>  	initialized = 1;
>  }
>  
> +void clean_shutdown()
> +{
> +	close_results();
> +	close_display();
> +	clean_open_devices();
> +	clear_all_devices();
> +	clear_all_cpus();
> +
> +	return;
> +}
> +

I would agree on that refactoring if we would call this function
from several places. otherwise, I see no actual reason.

	-ss

>  int main(int argc, char **argv)
>  {
> @@ -498,9 +509,7 @@ int main(int argc, char **argv)
>  	clear_tuning();
>  	reset_display();
>  
> -	clean_open_devices();
> -	clear_all_devices();
> -	clear_all_cpus();
> +	clean_shutdown();
>  
>  	return 0;
>  }
> -- 
> 2.1.2
> 
> _______________________________________________
> PowerTop mailing list
> PowerTop(a)lists.01.org
> https://lists.01.org/mailman/listinfo/powertop
>