/bin/mail doesn't handle piped in data

[AGMS]

/bin/mail is now included in the Haiku image (thanks!) but doesn't work when piping the message text via standard input (the BeOS one does work that way too).

Needs an end-of-file detection as well as the manual period on a line test. If you want to get fancy, only use the "." end of message marker when reading from a terminal. gets() returns NULL at end of file, should test for that and break out of the line reading loop at that time. Example user level usage to test it (make sure the last line of the listing gets through too):

ls | mail -s Testing me@…

[AGMS]

Patch to fix this ticket, and quite a few other bugs.

[axeld]

Thanks! There are a number of minor (coding style) issues that you might want to fix to simplify accepting that patch:

• We use sentence casing, ie. "No subject" over "No Subject"
• No space after isatty
• The printf in the following if-clause is a multi-line term, and must therefore be enclosed in curly braces
• BString body automatically creates an empty string, so BString body = "" is superfluous
• The comment after break should be on its own line like this:

  if (fgets(line, sizeof(line), stdin) == NULL) {
      // End of file or an error happened.
      break;
  }
  

  Extra points for actually checking for errors there, and reporting them :-)

[AGMS]

Fixes several bugs in /bin/mail, now with coding style fixes.

[korli]

Looks good. It still uses stdout to print error messages, wouldn't stderr be better?

[AGMS]

Good idea, I'll update the patch today to use stderr for error messages and user prompts. Though I think verbose mode log messages should stay in stdout, since the user may want to capture them separately.

[AGMS]

Now with stderr for certain messages (mostly stuff you need to see when stdout is sent to a file, stdout used for things you might want to save in a file), and style changes as suggested by Axel.

[waddlesplash]

Applied in hrev50117. Thanks!