Re: Fix more typos

2019-01-12 Thread Tim Rühsen
On 12.01.19 15:19, Bruno Haible wrote:
> Dmitry V. Levin wrote:
>> I don't think that use of "iff" here and in many other places touched
>> by this patch is a typo.
> 
> Agree. https://en.wikipedia.org/wiki/If_and_only_if
> 
> In gnulib-tool.texi, "fulfil" vs. "fulfill" is an English vs. American
> spelling difference. Better respect the preference of the author.
> 
> In mountlist.h, "filesystems" should be replaced by "file systems";
> that's the proper spelling.
> 
> I'm thus applying this in your name, Tim:

Thank you, Bruno !

Regards, Tim



signature.asc
Description: OpenPGP digital signature


Re: Fix more typos

2019-01-12 Thread Bruno Haible
Dmitry V. Levin wrote:
> I don't think that use of "iff" here and in many other places touched
> by this patch is a typo.

Agree. https://en.wikipedia.org/wiki/If_and_only_if

In gnulib-tool.texi, "fulfil" vs. "fulfill" is an English vs. American
spelling difference. Better respect the preference of the author.

In mountlist.h, "filesystems" should be replaced by "file systems";
that's the proper spelling.

I'm thus applying this in your name, Tim:


diff --git a/lib/af_alg.h b/lib/af_alg.h
index aabb52fab..877369a7f 100644
--- a/lib/af_alg.h
+++ b/lib/af_alg.h
@@ -66,7 +66,7 @@ afalg_buffer (const char *buffer, size_t len, const char *alg,
STREAM is an open file stream.  The last operation on STREAM should
not be 'ungetc', and if STREAM is also open for writing it should
have been fflushed since its last write.  Read from the current
-   position to the end of STREAM.  Handle regular files efficently.
+   position to the end of STREAM.  Handle regular files efficiently.
 
ALG is the message digest algorithm; see the file /proc/crypto.
 
diff --git a/lib/argp-help.c b/lib/argp-help.c
index 75abe84df..908ed91c7 100644
--- a/lib/argp-help.c
+++ b/lib/argp-help.c
@@ -1556,7 +1556,7 @@ argp_doc (const struct argp *argp, const struct 
argp_state *state,
 }
 
 /* Output a usage message for ARGP to STREAM.  If called from
-   argp_state_help, STATE is the relevent parsing state.  FLAGS are from the
+   argp_state_help, STATE is the relevant parsing state.  FLAGS are from the
set ARGP_HELP_*.  NAME is what to use wherever a 'program name' is
needed. */
 static void
diff --git a/lib/cdefs.h b/lib/cdefs.h
index 4b696590c..4f9217544 100644
--- a/lib/cdefs.h
+++ b/lib/cdefs.h
@@ -340,7 +340,7 @@
semantics.
 
clang++ identifies itself as gcc-4.2, but has support for GNU inlining
-   semantics, that can be checked fot by using the __GNUC_STDC_INLINE_ and
+   semantics, that can be checked for by using the __GNUC_STDC_INLINE_ and
__GNUC_GNU_INLINE__ macro definitions.  */
 #if (!defined __cplusplus || __GNUC_PREREQ (4,3) \
  || (defined __clang__ && (defined __GNUC_STDC_INLINE__ \
diff --git a/lib/file-set.c b/lib/file-set.c
index 5916a178e..5eb9fae3e 100644
--- a/lib/file-set.c
+++ b/lib/file-set.c
@@ -48,7 +48,7 @@ record_file (Hash_table *ht, char const *file, struct stat 
const *stats)
 
 if (ent_from_table != ent)
   {
-/* There was alread a matching entry in the table, so ENT was
+/* There was already a matching entry in the table, so ENT was
not inserted.  Free it.  */
 triple_free (ent);
   }
diff --git a/lib/get-rusage-as.c b/lib/get-rusage-as.c
index 0ace6d67c..73364c719 100644
--- a/lib/get-rusage-as.c
+++ b/lib/get-rusage-as.c
@@ -91,7 +91,7 @@
  b) pstat_getprocvm() can be used to find out about the virtual memory
 areas.
  Both methods agree, except that the value of get_rusage_as_via_iterator()
- is slighly larger higher than get_rusage_as_via_setrlimit(), by 4 KB in
+ is slightly larger higher than get_rusage_as_via_setrlimit(), by 4 KB in
  32-bit mode and by 40 KB in 64-bit mode.
 
IRIX:
diff --git a/lib/mountlist.h b/lib/mountlist.h
index d0cf6c1d6..e4ec024a3 100644
--- a/lib/mountlist.h
+++ b/lib/mountlist.h
@@ -32,7 +32,7 @@ struct mount_entry
   char *me_type;/* "nfs", "4.2", etc. */
   dev_t me_dev; /* Device number of me_mountdir. */
   unsigned int me_dummy : 1;/* Nonzero for dummy file systems. */
-  unsigned int me_remote : 1;   /* Nonzero for remote fileystems. */
+  unsigned int me_remote : 1;   /* Nonzero for remote file systems. */
   unsigned int me_type_malloced : 1; /* Nonzero if me_type was malloced. */
   struct mount_entry *me_next;
 };
diff --git a/lib/regexec.c b/lib/regexec.c
index 0a7a27b77..91d5a797b 100644
--- a/lib/regexec.c
+++ b/lib/regexec.c
@@ -2207,7 +2207,7 @@ sift_states_iter_mb (const re_match_context_t *mctx, 
re_sift_context_t *sctx,
dfa->nexts[node_idx]))
 /* The node can't accept the "multi byte", or the
destination was already thrown away, then the node
-   could't accept the current input "multi byte".   */
+   couldn't accept the current input "multi byte".   */
 naccepted = 0;
   /* Otherwise, it is sure that the node could accept
  'naccepted' bytes input.  */
diff --git a/lib/strptime.c b/lib/strptime.c
index bd7f87032..e6d405a44 100644
--- a/lib/strptime.c
+++ b/lib/strptime.c
@@ -277,7 +277,7 @@ __strptime_internal (rp, fmt, tm, decided, era_cnt 
LOCALE_PARAM)
 }
 
   /* Any character but '%' must be matched by the same character
- in the iput string.  */
+ in the input string.  */
   if (*fmt != '%')
 {
   match_char (*fmt++, *rp++);
diff --git a/lib/trigl.c b/lib/trigl.c
index 1f73bd92e..29cd2ba3e 100644
--- a/lib/trigl.c
+++ b/lib/trigl.c
@@ -325,7 +325,7 @@ static char rcsid[] =
  *

Re: Fix more typos

2019-01-12 Thread Tim Rühsen
On 12.01.19 14:40, Darshit Shah wrote:
> On January 12, 2019 12:21:36 PM UTC, "Dmitry V. Levin"  
> wrote:
>> On Sat, Jan 12, 2019 at 12:00:47PM +0100, Tim Rühsen wrote:
>>> Used a spell checker (codespell) to wipe out several typos.
>> [...]
>>> diff --git a/build-aux/texinfo.tex b/build-aux/texinfo.tex
>>> index 48d532bc6..74bea4134 100644
>>> --- a/build-aux/texinfo.tex
>>> +++ b/build-aux/texinfo.tex
>>> @@ -4688,7 +4688,7 @@ end
>>>\fi
>>>  }
>>>  
>>> -% @ifset VAR ... @end ifset reads the `...' iff VAR has been defined
>>> +% @ifset VAR ... @end ifset reads the `...' if VAR has been defined
>>
>> I don't think that use of "iff" here and in many other places touched
>> by this patch is a typo.
>>
>> Anyway, as texinfo.tex in gnulib comes verbatim from another project,
>> any changes to texinfo.tex should be made there first.
> 
> I agree. While iff is not a part of the English language, it's use in 
> technical writing to imply "if and only if" is standard practice.
> Replacing it with if changes the meaning of the sentence and the other 
> alternative is to use the entire phrase which gets too long and  unwieldy. 

OK, thanks. I didn't know that. Looked like a copy&paste thing. I'll
amend the patch accordingly.

Regards, Tim



signature.asc
Description: OpenPGP digital signature


Re: Fix more typos

2019-01-12 Thread Darshit Shah
On January 12, 2019 12:21:36 PM UTC, "Dmitry V. Levin"  
wrote:
>On Sat, Jan 12, 2019 at 12:00:47PM +0100, Tim Rühsen wrote:
>> Used a spell checker (codespell) to wipe out several typos.
>[...]
>> diff --git a/build-aux/texinfo.tex b/build-aux/texinfo.tex
>> index 48d532bc6..74bea4134 100644
>> --- a/build-aux/texinfo.tex
>> +++ b/build-aux/texinfo.tex
>> @@ -4688,7 +4688,7 @@ end
>>\fi
>>  }
>>  
>> -% @ifset VAR ... @end ifset reads the `...' iff VAR has been defined
>> +% @ifset VAR ... @end ifset reads the `...' if VAR has been defined
>
>I don't think that use of "iff" here and in many other places touched
>by this patch is a typo.
>
>Anyway, as texinfo.tex in gnulib comes verbatim from another project,
>any changes to texinfo.tex should be made there first.

I agree. While iff is not a part of the English language, it's use in technical 
writing to imply "if and only if" is standard practice.
Replacing it with if changes the meaning of the sentence and the other 
alternative is to use the entire phrase which gets too long and  unwieldy. 
-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.



Re: Fix more typos

2019-01-12 Thread Dmitry V. Levin
On Sat, Jan 12, 2019 at 12:00:47PM +0100, Tim Rühsen wrote:
> Used a spell checker (codespell) to wipe out several typos.
[...]
> diff --git a/build-aux/texinfo.tex b/build-aux/texinfo.tex
> index 48d532bc6..74bea4134 100644
> --- a/build-aux/texinfo.tex
> +++ b/build-aux/texinfo.tex
> @@ -4688,7 +4688,7 @@ end
>\fi
>  }
>  
> -% @ifset VAR ... @end ifset reads the `...' iff VAR has been defined
> +% @ifset VAR ... @end ifset reads the `...' if VAR has been defined

I don't think that use of "iff" here and in many other places touched
by this patch is a typo.

Anyway, as texinfo.tex in gnulib comes verbatim from another project,
any changes to texinfo.tex should be made there first.


-- 
ldv


signature.asc
Description: PGP signature