src/bin/ed/buf.c, branch stable/14

Remove $FreeBSD$: one-line .c pattern

2023-08-16T17:54:42Z

Remove /^[\s*]*__FBSDID$"\$FreeBSD\$"$;?\s*\n/

ed(1): drop redundant parenthesis.

2016-12-14T21:14:43Z

These were Left-over from the cleanup in r309936.

ed(1): Simplify some checks.

2016-12-12T20:25:59Z

The return type for both fread(3) and fwrite(3) cannot be negative, this renders some checks invalid and variable 'ct' unnecessary. Also bump 'len' to size_t to avoid signed/unsigned comparison warnings.

More -Wmissing-variable-declarations fixes.

2012-10-19T14:49:42Z

In addition to adding `static' where possible: - bin/date: Move `retval' into extern.h to make it visible to date.c. - bin/ed: Move globally used variables into ed.h. - sbin/camcontrol: Move `verbose' into camcontrol.h and fix shadow warnings. - usr.bin/calendar: Remove unneeded variables. - usr.bin/chat: Make `line' local instead of global. - usr.bin/elfdump: Comment out unneeded function. - usr.bin/rlogin: Use _Noreturn instead of __dead2. - usr.bin/tset: Pull `Ospeed' into extern.h. - usr.sbin/mfiutil: Put global variables in mfiutil.h. - usr.sbin/pkg: Remove unused `os_corres'. - usr.sbin/quotaon, usr.sbin/repquota: Remove unused `qfname'.

Fix warnings found by -Wmising-variable-declarations.

2012-10-19T05:43:38Z

This self-written compiler warning, which is hopefully going to be committed into LLVM sources soon, warns about potentially missing `static' keywords, similar to -Wmissing-prototypes. - bin/pax: Move external declaration of chdname and s_mask into extern.h. - bin/setfacl: Move setfacl.c-specific stuff out of setfacl.h. - sbin/mount_fusefs: Remove char *progname; use getprogname(). - others: add `static' where possible.

Fix memory leaks in error cases.

2011-08-27T15:28:57Z

PR: 159011 Submitted by: Henning Petersen Approved by: re (kib) MFC after: 1 week

Consistently use __FBSDID

2002-06-30T05:13:54Z

Bogusness may have happened using a variable assignment here before, but

2002-06-20T07:15:55Z

proper parens mean that fd is always set by open(2) [in any part of C], and so we can accurately check for it returning -1, without feeling like we need to initialise fd to -1 in its declaration. In other words, fix a stylistic/bogus nit.

o __P has been reoved

2002-02-02T06:36:49Z

o Old-style K&R declarations have been converted to new C89 style o register has been removed o prototype for main() has been removed (gcc3 makes it an error) o int main(int argc, char *argv[]) is the preferred main definition. o Attempt to not break style(9) conformance for declarations more than they already are. o gc some #ifdef sun ... #endif code Approved by: arch@, new style(9)

File positions are off_t nowdays, not long, so:

2001-09-01T23:26:40Z

fseek -> fseeko ftell -> ftello NOTE: that fseek/ftell not works for >long offsets per POSIX: [EOVERFLOW] For fseek( ), the resulting file offset would be a value which cannot be represented correctly in an object of type long. [EOVERFLOW] For ftell ( ), the current file offset cannot be represented correctly in an object of type long.