[Bug 277037] Typographical errors and grammar clarification in fclose(3) with a patch

2024-02-16 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277037

Maxim Konovalov  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|New |Closed

--- Comment #7 from Maxim Konovalov  ---
The fix committed.  Thanks for spotting this one!

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 277037] Typographical errors and grammar clarification in fclose(3) with a patch

2024-02-16 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277037

--- Comment #6 from commit-h...@freebsd.org ---
A commit in branch main references this bug:

URL:
https://cgit.FreeBSD.org/src/commit/?id=1250c9647959fab955fb8d9046cb41bc749e6855

commit 1250c9647959fab955fb8d9046cb41bc749e6855
Author: Maxim Konovalov 
AuthorDate: 2024-02-16 18:50:12 +
Commit: Maxim Konovalov 
CommitDate: 2024-02-16 18:50:12 +

fclose.3: remove a confusing sentence

PR: 277037
Reviewed by:oshogbo

 lib/libc/stdio/fclose.3 | 5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 277037] Typographical errors and grammar clarification in fclose(3) with a patch

2024-02-14 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277037

--- Comment #5 from Maxim Konovalov  ---
You are more than welcome!

Maxim

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 277037] Typographical errors and grammar clarification in fclose(3) with a patch

2024-02-14 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277037

--- Comment #4 from J Adams  ---
Glad to help; I'm trying to get more involved in the FBSD community. I hope to
continue to contribute! :-)
BTW, I took your lead and also read through libc/stdio/fclose.c. Surprisingly
simple.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 277037] Typographical errors and grammar clarification in fclose(3) with a patch

2024-02-14 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277037

Maxim Konovalov  changed:

   What|Removed |Added

   Assignee|b...@freebsd.org|ma...@freebsd.org

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 277037] Typographical errors and grammar clarification in fclose(3) with a patch

2024-02-14 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277037

--- Comment #3 from Maxim Konovalov  ---
You are not alone here!

Thanks for spotting this.  I'll try to reach the author and commit the patch
afterwards.

Maxim

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 277037] Typographical errors and grammar clarification in fclose(3) with a patch

2024-02-14 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277037

--- Comment #2 from J Adams  ---
(In reply to Maxim Konovalov from comment #1)
Can't really disagree on that. After all, the sentence made me scratch my head
a little; and it wasn't just the grammar. Your patch makes the paragraph more
clear and concise.

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 277037] Typographical errors and grammar clarification in fclose(3) with a patch

2024-02-13 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277037

Maxim Konovalov  changed:

   What|Removed |Added

 CC||ma...@freebsd.org

--- Comment #1 from Maxim Konovalov  ---
Created attachment 248452
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=248452=edit
fdclose man page diff

Hi,

After reading libc/stdio/fclose.c I tend to think that the whole sentence
should be just removed.  I.e., it should look like the following:

The fdclose() function is equivalent to fclose() except that it does not
close the underlying file descriptor.

If fdp is not NULL, the file descriptor will be written to it.

If the stream does not have an associated file descriptor, fdp will be set to
-1. 

%%%

I enclosed the patch for completeness.

Maxim

-- 
You are receiving this mail because:
You are on the CC list for the bug.


[Bug 277037] Typographical errors and grammar clarification in fclose(3) with a patch

2024-02-13 Thread bugzilla-noreply
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=277037

Bug ID: 277037
   Summary: Typographical errors and grammar clarification in
fclose(3) with a patch
   Product: Documentation
   Version: Latest
  Hardware: Any
OS: Any
Status: New
  Severity: Affects Some People
  Priority: ---
 Component: Manual Pages
  Assignee: b...@freebsd.org
  Reporter: jf...@duck.com
CC: d...@freebsd.org

I noticed some typos and potentially ambiguous grammar in flcose(3) and have a
patch to offer.

diff -u a/fclose.3 b/fclose.3 
--- a/fclose.3  2024-02-13 18:08:37.324978000 +
+++ b/fclose.3  2024-02-13 18:47:48.201409000 +
@@ -73,7 +73,7 @@
 the file descriptor will be written to it.
 If the
 .Fa fdp
-argument will be different then NULL the file descriptor will be returned in
it,
+argument is other than NULL, the file descriptor will be returned in it.
 If the stream does not have an associated file descriptor,
 .Fa fdp
 will be set to -1.

The sentence in question ended in a comma rather than a period. Also 'than' is
needed to indicate a comparison whereas 'then' indicates a continuation. I hope
my change in grammar accurately coveys the correct meaning of the sentence. If
not, I think it's because of the ambiguity.

-- 
You are receiving this mail because:
You are on the CC list for the bug.