[pfbtops]: Fix code style and diagnostic nits.

* src/utils/pfbtops/pfbtops.c (error): Exit with `EXIT_FAILURE` status
  (from standard C library) instead of status 2.

  (main): Exit with `EXIT_SUCCESS` status when writing version or help
  information.  Exit with status 2 when dying due to usage error.  Use
  `fprintf()` and `strerror()` to construct error message when dying due
  to inability to open input file instead of using `perror()`, which
  anonymizes its caller and thus should never be used in serious work.
  Avoid it like `gets()`.

* NEWS: Add item for exit status changes.

Continues the long process of fixing Savannah #52463.

Also update editor aid comments.

1 files changed, 4 insertions, 0 deletions

diff --git a/NEWS b/NEWS
index c777ea1ae..ed9b4d9df 100644
--- a/NEWS
+++ b/NEWS
@@ -601,6 +601,10 @@ o The "afmtodit" utility no longer writes file names with directory
   it generates.  (The `fp` request no longer accepts such names; see
   "troff" above.)
 
+o pfbtops now exits with status 2 upon usage errors and the standard C
+  library's `EXIT_FAILURE` status (usually 1) on operational failures
+  instead of vice versa.
+
 o groffer has been deleted from the distribution.
 
 o grog no longer supports the "--warnings" option; the one diagnostic