Date: Thu, 17 Dec 2020 13:01:01 +0000 From: Jessica Clarke <jrtc27@freebsd.org> To: Konstantin Belousov <kostikbel@gmail.com> Cc: Mateusz Piotrowski <0mp@freebsd.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r368714 - head/lib/libc/string Message-ID: <686CF2E6-1D3C-4A83-A323-02CD9F536675@freebsd.org> In-Reply-To: <X9tU0kTm9V6KoCRr@kib.kiev.ua> References: <202012171241.0BHCfl1r008452@repo.freebsd.org> <X9tU0kTm9V6KoCRr@kib.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
On 17 Dec 2020, at 12:53, Konstantin Belousov <kostikbel@gmail.com> = wrote: >=20 > On Thu, Dec 17, 2020 at 12:41:47PM +0000, Mateusz Piotrowski wrote: >> Author: 0mp (doc,ports committer) >> Date: Thu Dec 17 12:41:47 2020 >> New Revision: 368714 >> URL: https://svnweb.freebsd.org/changeset/base/368714 >>=20 >> Log: >> strerror.3: Add an example for perror() >>=20 >> This is a nice and quick reference. >>=20 >> Reviewed by: jilles, yuripv >> MFC after: 2 weeks >> Differential Revision: https://reviews.freebsd.org/D27623 >>=20 >> Modified: >> head/lib/libc/string/strerror.3 >>=20 >> Modified: head/lib/libc/string/strerror.3 >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/lib/libc/string/strerror.3 Thu Dec 17 03:42:54 2020 = (r368713) >> +++ head/lib/libc/string/strerror.3 Thu Dec 17 12:41:47 2020 = (r368714) >> @@ -32,7 +32,7 @@ >> .\" @(#)strerror.3 8.1 (Berkeley) 6/9/93 >> .\" $FreeBSD$ >> .\" >> -.Dd December 7, 2020 >> +.Dd December 17, 2020 >> .Dt STRERROR 3 >> .Os >> .Sh NAME >> @@ -170,6 +170,31 @@ The use of these variables is deprecated; >> or >> .Fn strerror_r >> should be used instead. >> +.Sh EXAMPLES >> +The following example shows how to use >> +.Fn perror >> +to report an error. >> +.Bd -literal -offset 2n >> +#include <fcntl.h> >> +#include <stdio.h> >> +#include <stdlib.h> >> + >> +int >> +main(void) >> +{ >> + int fd; >> + >> + if ((fd =3D open("/nonexistent", O_RDONLY)) =3D=3D -1) { >> + perror("open()"); >> + exit(1); >> + } >> + printf("File descriptor: %d\en", fd); > This lines is indented with spaces, while other lines have tabs. >=20 >> + return (0); > return (0) is redundand. It's not required as per the standard, but omitting it is needlessly obfuscating it and bad practice. C lets you do a whole load of things that are a bad idea, and whilst this one is harmless, it is nonetheless confusing to anyone who is not intimately acquainted quirks like this special case in the standard. Jess
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?686CF2E6-1D3C-4A83-A323-02CD9F536675>