git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] compat/regex: fix typos in comments.

From
Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Date
May 6, 2012, 15:07 UTC
Message-ID
<4FA693A9.20101@in.waw.pl>
In-Reply-To
<1336308300-4858-1-git-send-email-semtlenori@gmail.com>
On 05/06/2012 02:45 PM, Yi, EungJun wrote:
Show 20 quoted lines
> From: "Yi, EungJun" <semtlenori@gmail.com>
> 
> occured => occurred
> 
> Signed-off-by: Yi, EungJun <semtlenori@gmail.com>
> ---
>  compat/regex/regcomp.c        |    4 ++--
>  compat/regex/regex_internal.c |    6 +++---
>  2 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/compat/regex/regcomp.c b/compat/regex/regcomp.c
> index 8c96ed9..aab8153 100644
> --- a/compat/regex/regcomp.c
> +++ b/compat/regex/regcomp.c
> @@ -2095,7 +2095,7 @@ peek_token_bracket (re_token_t *token, re_string_t *input, reg_syntax_t syntax)
>  
>  /* Entry point of the parser.
>     Parse the regular expression REGEXP and return the structure tree.
> -   If an error is occured, ERR is set by error code, and return NULL.
> +   If an error is occurred, ERR is set by error code, and return NULL.

Hi, you might want to fix the grammatical errors instead: "If an error occurs, ERR is set to the error code, and NULL is returned.", or something like that, I haven't actually read the code.

Best, Zbyszek

Show 43 quoted lines
>     This function build the following tree, from regular expression <reg_exp>:
>  	   CAT
>  	   / \
> @@ -3715,7 +3715,7 @@ build_charclass_op (re_dfa_t *dfa, RE_TRANSLATE_TYPE trans,
>  /* This is intended for the expressions like "a{1,3}".
>     Fetch a number from `input', and return the number.
>     Return -1, if the number field is empty like "{,1}".
> -   Return -2, If an error is occured.  */
> +   Return -2, If an error is occurred.  */
>  
>  static int
>  fetch_number (re_string_t *input, re_token_t *token, reg_syntax_t syntax)
> diff --git a/compat/regex/regex_internal.c b/compat/regex/regex_internal.c
> index 193854c..237642c 100644
> --- a/compat/regex/regex_internal.c
> +++ b/compat/regex/regex_internal.c
> @@ -1284,7 +1284,7 @@ re_node_set_merge (re_node_set *dest, const re_node_set *src)
>  
>  /* Insert the new element ELEM to the re_node_set* SET.
>     SET should not already have ELEM.
> -   return -1 if an error is occured, return 1 otherwise.  */
> +   return -1 if an error is occurred, return 1 otherwise.  */
>  
>  static int
>  internal_function
> @@ -1341,7 +1341,7 @@ re_node_set_insert (re_node_set *set, int elem)
>  
>  /* Insert the new element ELEM to the re_node_set* SET.
>     SET should not already have any element greater than or equal to ELEM.
> -   Return -1 if an error is occured, return 1 otherwise.  */
> +   Return -1 if an error is occurred, return 1 otherwise.  */
>  
>  static int
>  internal_function
> @@ -1416,7 +1416,7 @@ re_node_set_remove_at (re_node_set *set, int idx)
>  
>  
>  /* Add the token TOKEN to dfa->nodes, and return the index of the token.
> -   Or return -1, if an error will be occured.  */
> +   Or return -1, if an error will be occurred.  */
>  
>  static int
>  internal_function
Previous: Ævar Arnfjörð Bjarmason
Message 5 of 5 in “compat/regex: fix typos in comments.”
  1. compat/regex: fix typos in comments.Yi, EungJun, May 6, 2012
  2. Ævar Arnfjörð BjarmasonMay 6, 2012
  3. Yi, EungJunMay 6, 2012
  4. Ævar Arnfjörð BjarmasonMay 6, 2012
  5. Zbigniew Jędrzejewski-SzmekMay 6, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.