cat(1): refactor cook_args()/raw_args() into single function, cat_file()
authorcheloha <cheloha@openbsd.org>
Wed, 9 Feb 2022 01:56:28 +0000 (01:56 +0000)
committercheloha <cheloha@openbsd.org>
Wed, 9 Feb 2022 01:56:28 +0000 (01:56 +0000)
commit0a1461c9aafe5b97e0544ea4d0c4ed564804880c
tree58c26c74dd5485b3a42237bc547ef59236d0c696
parent9ebd5a39871ba5b66c35b1c67d141890d11a2666
cat(1): refactor cook_args()/raw_args() into single function, cat_file()

- Combine the open/close portions of cook_args()/raw_args() into a single
  function, cat_file().

- Push the flag-checking conditional in main() down into cat_file().

- Pull the argv loop in cat_file() up into main().

These changes -- especially pulling the argv look up into main() --
will allow us to drop the "rpath" promise in a single spot in a
subsequent patch.

Tweaked by mestre@.  Descriptor leak in earlier version spotted by
Matthew Martin.

Thread: https://marc.info/?l=openbsd-tech&m=163941848104274&w=2

No objections on tech@ after several weeks.
bin/cat/cat.c