Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1698491 > unrolled thread

[PATCH] init:main.c: Fixed issues in Block comments and removed braces in single statement if block

Back to article view | Back to linux.kernel

Contents

  [PATCH] init:main.c: Fixed issues in Block comments and removed braces in single statement if block  janani-sankarababu <jananis37@gmail.com> - 2017-07-28 08:20 +0200
    Re: [PATCH] init:main.c: Fixed issues in Block comments and removed  braces in single statement if block Steven Rostedt <rostedt@goodmis.org> - 2017-07-28 14:30 +0200

#1698491 — [PATCH] init:main.c: Fixed issues in Block comments and removed braces in single statement if block

Signed-off-by: Janani S <jananis37@gmail.com>

---
 init/main.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/init/main.c b/init/main.c
index 052481f..f8eb4966 100644
--- a/init/main.c
+++ b/init/main.c
@@ -181,7 +181,8 @@ static bool __init obsolete_checksetup(char *line)
 				/* Already done in parse_early_param?
 				 * (Needs exact match on param part).
 				 * Keep iterating, as we can have early
-				 * params and __setups of same names 8( */
+				 * params and __setups of same names
+				 */
 				if (line[n] == '\0' || line[n] == '=')
 					had_early_param = true;
 			} else if (!p->setup_func) {
@@ -693,9 +694,9 @@ asmlinkage __visible void __init start_kernel(void)
 	arch_post_acpi_subsys_init();
 	sfi_init_late();
 
-	if (efi_enabled(EFI_RUNTIME_SERVICES)) {
+	if (efi_enabled(EFI_RUNTIME_SERVICES))
 		efi_free_boot_services();
-	}
+
 
 	/* Do the rest non-__init'ed, we're now alive */
 	rest_init();
-- 
1.9.1

[toc] | [next] | [standalone]

#1698729 — Re: [PATCH] init:main.c: Fixed issues in Block comments and removed braces in single statement if block

On Fri, 28 Jul 2017 11:37:33 +0530
janani-sankarababu <jananis37@gmail.com> wrote:

> Signed-off-by: Janani S <jananis37@gmail.com>
> 
> ---
>  init/main.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/init/main.c b/init/main.c
> index 052481f..f8eb4966 100644
> --- a/init/main.c
> +++ b/init/main.c
> @@ -181,7 +181,8 @@ static bool __init obsolete_checksetup(char *line)
>  				/* Already done in parse_early_param?
>  				 * (Needs exact match on param part).
>  				 * Keep iterating, as we can have early
> -				 * params and __setups of same names 8( */
> +				 * params and __setups of same names

Why did you get rid of the "8(" part? Yeah, it's a kind of an old
fashion emoji, but it helps make the code stand out as something that
we can improve on in the future.

> +				 */
>  				if (line[n] == '\0' || line[n] == '=')
>  					had_early_param = true;
>  			} else if (!p->setup_func) {
> @@ -693,9 +694,9 @@ asmlinkage __visible void __init start_kernel(void)
>  	arch_post_acpi_subsys_init();
>  	sfi_init_late();
>  
> -	if (efi_enabled(EFI_RUNTIME_SERVICES)) {
> +	if (efi_enabled(EFI_RUNTIME_SERVICES))
>  		efi_free_boot_services();
> -	}
> +

Honestly, I'm not too thrilled with these patches. It causes churn to
the code history with very little benefit. Usually these types of clean
ups are done when the code is modified for functional reasons.

Brackets for single instruction blocks isn't needed. But it's not
something we should go out of our way to "fix".

-- Steve


>  
>  	/* Do the rest non-__init'ed, we're now alive */
>  	rest_init();

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web