From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1VHylL-0005nJ-QN for mharc-grub-devel@gnu.org; Fri, 06 Sep 2013 12:19:59 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51446) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VHylI-0005iP-47 for grub-devel@gnu.org; Fri, 06 Sep 2013 12:19:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VHylG-00033o-CT for grub-devel@gnu.org; Fri, 06 Sep 2013 12:19:55 -0400 Received: from mail-qc0-x249.google.com ([2607:f8b0:400d:c01::249]:64727) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VHylG-00033g-1a for grub-devel@gnu.org; Fri, 06 Sep 2013 12:19:54 -0400 Received: by mail-qc0-f201.google.com with SMTP id e16so387373qcx.0 for ; Fri, 06 Sep 2013 09:19:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=o/LYy6kaHbqw6bXstLN080njdh3To5UW6jBhwNuW8pc=; b=HsnNZHo+o39JTvqG6IG1G883iIj3YHelsBreTdx6qo7e/+n/Zj9ruzT5mvmn618du1 02GiPYx8LZziEh591X9DCNM7ruM6PfrNuCjCX2uKYVtyyRGyh7bpux7frigZLZujucnC Y+yJRfcMTWOw3XXyurmHDgL6mH+IkioRgx5Xia1OuEYu5+Z0qjr03Z4BYJgl2uUO0nY5 es8vx4uTH/3dK38WpiZ5KRRoAVcfNFDMd4Oi3TmAc3vr5aVSng+Fe2qvu/JASBTgRIor /RIrytJf4N5Jp6lAgTczQZVPX2akjjtARU1i8dE5i9sBmz48jznbMjcRw86AnKrHeLS2 udNQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=o/LYy6kaHbqw6bXstLN080njdh3To5UW6jBhwNuW8pc=; b=kPfijp36x0tv3Gp11wwsG7Ss7J1NzcSTIwCjZr+LJ4CD7cHJHbc8FPp1RsfmluYcsN 1sw9f1abi/5otWRQCmm15nXpen+08woRTPO1lblltwFhREOKSeqA1EHfEfY0uZrygCTd JH8c5URJ6gHDFOI8GCNqFyuKRDwOxj37I67wgfsl3TAhZX3HvU7sAd0vNOdTGIQTcJBc UMqBT0Jf3TWy76//7R9hhTv4QSy7ceR12Yd82v3STTFX3SSRgP3yfGXGqgYt/XyGjLhP U+jsT1QUcxvoyoo1xvzMTDech1vLDTofICkrConeQBfjcgbnJ6XWo+08OHWDK4Iw6WmK vtLw== X-Gm-Message-State: ALoCoQmUybC2tdidrVdhcv6qXNNc6t+PeHPHH2f8bigsQj98uW/5T2UHRje18B01hdyE0PPYar4L/F7wcTVPw4751P4TerONx2ZVXBaHdcIvTISihccTHR7r+uJQe3BUVaVGafegOTtQ7qoadiP3i7EGAbDcfsvpAQLAyNbnOd6GZQDgvrOtKepq/ibF1SegvNE+lICaRuc0smVMsyGDB3QgF66uFYRIHA1E+kQp9oATEZTfNgm7yWY= X-Received: by 10.236.125.232 with SMTP id z68mr454967yhh.56.1378484393465; Fri, 06 Sep 2013 09:19:53 -0700 (PDT) Received: from corp2gmr1-1.hot.corp.google.com (corp2gmr1-1.hot.corp.google.com [172.24.189.92]) by gmr-mx.google.com with ESMTPS id a42si162460yhj.6.1969.12.31.16.00.00 (version=TLSv1.1 cipher=AES128-SHA bits=128/128); Fri, 06 Sep 2013 09:19:53 -0700 (PDT) Received: from yinz.mtv.corp.google.com (yinz.mtv.corp.google.com [172.17.81.122]) by corp2gmr1-1.hot.corp.google.com (Postfix) with ESMTP id 36DC131C1DC; Fri, 6 Sep 2013 09:19:53 -0700 (PDT) Received: by yinz.mtv.corp.google.com (Postfix, from userid 184367) id E7FDCC076E; Fri, 6 Sep 2013 09:19:52 -0700 (PDT) From: Jon McCune To: grub-devel@gnu.org Subject: [PATCH v2 1/5] style: indent --no-tabs --gnu-style grub-core/commands/loadenv.c Date: Fri, 6 Sep 2013 09:18:49 -0700 Message-Id: <1378484333-13577-2-git-send-email-jonmccune@google.com> X-Mailer: git-send-email 1.8.4 In-Reply-To: <1378484333-13577-1-git-send-email-jonmccune@google.com> References: <1378484333-13577-1-git-send-email-jonmccune@google.com> X-detected-operating-system: by eggs.gnu.org: Error: Malformed IPv6 address (bad octet value). X-Received-From: 2607:f8b0:400d:c01::249 Cc: Jon McCune X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: The development of GNU GRUB List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 06 Sep 2013 16:19:57 -0000 Added the option --no-tabs after observing that 'indent' did not result in consistent tab usage. An example of the inconsistent tab usage: the very first function in loadenv.c is open_envblk_file(), and it contains a line indented exclusively with tabs, and lines indented exclusively with spaces. The space-idented lines use two spaces for each level of indentation. In order for the tab-indented lines to format properly, the tabs must consume the equivalent of 8 spaces (i.e., four levels of indentation). To my knowledge, this is not consistent with the default GNU style. Signed-off-by: Jon McCune --- grub-core/commands/loadenv.c | 87 ++++++++++++++++++++++---------------------- 1 file changed, 44 insertions(+), 43 deletions(-) diff --git a/grub-core/commands/loadenv.c b/grub-core/commands/loadenv.c index c0a42c5..a431499 100644 --- a/grub-core/commands/loadenv.c +++ b/grub-core/commands/loadenv.c @@ -30,20 +30,19 @@ GRUB_MOD_LICENSE ("GPLv3+"); -static const struct grub_arg_option options[] = - { - /* TRANSLATORS: This option is used to override default filename - for loading and storing environment. */ - {"file", 'f', 0, N_("Specify filename."), 0, ARG_TYPE_PATHNAME}, - {0, 0, 0, 0, 0, 0} - }; +static const struct grub_arg_option options[] = { + /* TRANSLATORS: This option is used to override default filename + for loading and storing environment. */ + {"file", 'f', 0, N_("Specify filename."), 0, ARG_TYPE_PATHNAME}, + {0, 0, 0, 0, 0, 0} +}; static grub_file_t open_envblk_file (char *filename) { grub_file_t file; - if (! filename) + if (!filename) { const char *prefix; @@ -54,20 +53,21 @@ open_envblk_file (char *filename) len = grub_strlen (prefix); filename = grub_malloc (len + 1 + sizeof (GRUB_ENVBLK_DEFCFG)); - if (! filename) + if (!filename) return 0; grub_strcpy (filename, prefix); filename[len] = '/'; grub_strcpy (filename + len + 1, GRUB_ENVBLK_DEFCFG); - grub_file_filter_disable_compression (); + grub_file_filter_disable_compression (); file = grub_file_open (filename); grub_free (filename); return file; } else { - grub_error (GRUB_ERR_FILE_NOT_FOUND, N_("variable `%s' isn't set"), "prefix"); + grub_error (GRUB_ERR_FILE_NOT_FOUND, N_("variable `%s' isn't set"), + "prefix"); return 0; } } @@ -85,7 +85,7 @@ read_envblk_file (grub_file_t file) grub_envblk_t envblk; buf = grub_malloc (size); - if (! buf) + if (!buf) return 0; while (size > 0) @@ -104,7 +104,7 @@ read_envblk_file (grub_file_t file) } envblk = grub_envblk_open (buf, offset); - if (! envblk) + if (!envblk) { grub_free (buf); grub_error (GRUB_ERR_BAD_FILE_TYPE, "invalid environment block"); @@ -124,25 +124,25 @@ set_var (const char *name, const char *value) static grub_err_t grub_cmd_load_env (grub_extcmd_context_t ctxt, - int argc __attribute__ ((unused)), - char **args __attribute__ ((unused))) + int argc __attribute__ ((unused)), + char **args __attribute__ ((unused))) { struct grub_arg_list *state = ctxt->state; grub_file_t file; grub_envblk_t envblk; file = open_envblk_file ((state[0].set) ? state[0].arg : 0); - if (! file) + if (!file) return grub_errno; envblk = read_envblk_file (file); - if (! envblk) + if (!envblk) goto fail; grub_envblk_iterate (envblk, set_var); grub_envblk_close (envblk); - fail: +fail: grub_file_close (file); return grub_errno; } @@ -157,25 +157,25 @@ print_var (const char *name, const char *value) static grub_err_t grub_cmd_list_env (grub_extcmd_context_t ctxt, - int argc __attribute__ ((unused)), - char **args __attribute__ ((unused))) + int argc __attribute__ ((unused)), + char **args __attribute__ ((unused))) { struct grub_arg_list *state = ctxt->state; grub_file_t file; grub_envblk_t envblk; file = open_envblk_file ((state[0].set) ? state[0].arg : 0); - if (! file) + if (!file) return grub_errno; envblk = read_envblk_file (file); - if (! envblk) + if (!envblk) goto fail; grub_envblk_iterate (envblk, print_var); grub_envblk_close (envblk); - fail: +fail: grub_file_close (file); return grub_errno; } @@ -253,7 +253,7 @@ check_blocklists (grub_envblk_t envblk, struct blocklist *blocklists, return grub_errno; if (grub_memcmp (buf + index, blockbuf, p->length) != 0) - return grub_error (GRUB_ERR_FILE_READ_ERROR, "invalid blocklist"); + return grub_error (GRUB_ERR_FILE_READ_ERROR, "invalid blocklist"); } return GRUB_ERR_NONE; @@ -293,7 +293,7 @@ struct grub_cmd_save_env_ctx /* Store blocklists in a linked list. */ static void save_env_read_hook (grub_disk_addr_t sector, unsigned offset, unsigned length, - void *data) + void *data) { struct grub_cmd_save_env_ctx *ctx = data; struct blocklist *block; @@ -303,7 +303,7 @@ save_env_read_hook (grub_disk_addr_t sector, unsigned offset, unsigned length, return; block = grub_malloc (sizeof (*block)); - if (! block) + if (!block) return; block->sector = sector; @@ -315,7 +315,7 @@ save_env_read_hook (grub_disk_addr_t sector, unsigned offset, unsigned length, if (ctx->tail) ctx->tail->next = block; ctx->tail = block; - if (! ctx->head) + if (!ctx->head) ctx->head = block; } @@ -330,14 +330,14 @@ grub_cmd_save_env (grub_extcmd_context_t ctxt, int argc, char **args) .tail = 0 }; - if (! argc) + if (!argc) return grub_error (GRUB_ERR_BAD_ARGUMENT, "no variable is specified"); file = open_envblk_file ((state[0].set) ? state[0].arg : 0); - if (! file) + if (!file) return grub_errno; - if (! file->device->disk) + if (!file->device->disk) { grub_file_close (file); return grub_error (GRUB_ERR_BAD_DEVICE, "disk device required"); @@ -347,7 +347,7 @@ grub_cmd_save_env (grub_extcmd_context_t ctxt, int argc, char **args) file->read_hook_data = &ctx; envblk = read_envblk_file (file); file->read_hook = 0; - if (! envblk) + if (!envblk) goto fail; if (check_blocklists (envblk, ctx.head, file)) @@ -360,9 +360,10 @@ grub_cmd_save_env (grub_extcmd_context_t ctxt, int argc, char **args) value = grub_env_get (args[0]); if (value) { - if (! grub_envblk_set (envblk, args[0], value)) + if (!grub_envblk_set (envblk, args[0], value)) { - grub_error (GRUB_ERR_BAD_ARGUMENT, "environment block too small"); + grub_error (GRUB_ERR_BAD_ARGUMENT, + "environment block too small"); goto fail; } } @@ -373,7 +374,7 @@ grub_cmd_save_env (grub_extcmd_context_t ctxt, int argc, char **args) write_blocklists (envblk, ctx.head, file); - fail: +fail: if (envblk) grub_envblk_close (envblk); free_blocklists (ctx.head); @@ -383,24 +384,24 @@ grub_cmd_save_env (grub_extcmd_context_t ctxt, int argc, char **args) static grub_extcmd_t cmd_load, cmd_list, cmd_save; -GRUB_MOD_INIT(loadenv) +GRUB_MOD_INIT (loadenv) { cmd_load = grub_register_extcmd ("load_env", grub_cmd_load_env, 0, N_("[-f FILE]"), - N_("Load variables from environment block file."), - options); + N_("Load variables from environment block file."), + options); cmd_list = grub_register_extcmd ("list_env", grub_cmd_list_env, 0, N_("[-f FILE]"), - N_("List variables from environment block file."), - options); + N_("List variables from environment block file."), + options); cmd_save = grub_register_extcmd ("save_env", grub_cmd_save_env, 0, - N_("[-f FILE] variable_name [...]"), - N_("Save variables to environment block file."), - options); + N_("[-f FILE] variable_name [...]"), + N_("Save variables to environment block file."), + options); } -GRUB_MOD_FINI(loadenv) +GRUB_MOD_FINI (loadenv) { grub_unregister_extcmd (cmd_load); grub_unregister_extcmd (cmd_list); -- 1.8.4