| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
Give the function one more argument to decide whether it should
print the flags like a C-expression or just space-delimited.
|
|
|
|
|
| |
Patch: 7.192
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
| |
|
|
|
|
| |
and simplify the printing functions.
|
|
|
|
|
|
| |
space after function names, remove spaces on emtpy lines.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
that get included just where they are needed. All headers include the
headers that they need to compile (just with an empty .c file). Sort
includes alphabetically where apropriate and fix some duplicate commenting
for struct Job, struct GNode and struct Shell by removing one version and
inlining the comments into the structure declaration (the comments have been
somewhat outdated).
This patch does not contain functional changes (checked with md5).
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
| |
just before exiting (especially given the number of memory leaks) -
it just costs time.
|
|
|
|
|
|
|
|
| |
now that their size is only two pointers. This eliminates a lot of calls
to Lst_Init and from there to malloc together with many calls to
Lst_Destroy (in places where the list is obviously empty). This also
reduces the chance to leave a list uninitilized so we can remove more
NULL pointer checks and probably eliminates a couple of memory leaks.
|
|
|
|
| |
to free them just before exiting.
|
|
|
|
|
|
|
|
| |
only in a couple of places and all of them except for one were easily
converted to use Lst_First/Lst_Succ. The one place is compatibility
mode in job.c where the it was used to advance to the next command on
each invocation of JobStart. For this case add a pointer to the node to
hold the currently executed command.
|
|
|
|
|
|
| |
the structs itself not of pointers to them. This will simplify constification.
Checked by: diff on the object files
|
| |
|
|
|
|
|
|
| |
pointer constant as NULL.
Checked by: diff -r on the object files before and after
|
|
|
|
|
|
|
|
| |
and the sizeof operator, missing empty lines, void casts, extra empty lines.
Checked by: diff on make *.o lst.lib/*.o
Submitted by: Max Okumoto <okumoto@soe.ucsd.edu> (partly)
|
|
|
|
|
|
| |
might help on the systems it could possibly be used as a bandaid for. In
fact, the only thing it's useful for is instrumenting free(3) calls, and in
that capacity, it's better served as a local patch, than a public wrapper.
|
|
|
|
|
|
|
|
|
|
|
| |
documentation already adequatedly existed in the description in most
cases. Where it did not, it was added. If no documentation existed
beforehand, then none was added. Some unused dummies for use in the
traversal functions were marked as __unused during the conversion.
Occasionally, local style fixes were applied to lines already being
modified or influenced.
Now make(1) should always build with WARNS=3.
|
|
|
|
|
|
|
|
|
|
|
| |
renaming variables to not shadow libc functions or greater scope locals. Kinda
makes one wonder if the extern ones weren't meant in some of these places :)
The only thing I'd still like to do WRT this is possibly combine rstat and
status in compat.c -- that should be fine, as I do not think the codepaths
will want both around at once.
Sponsored by: Bright Path Solutions
|
|
|
|
|
|
|
|
|
|
| |
in compat.c which doesn't even have preprocessor-conditional-hidden support
code, and add a debugging statement where we might end up with a nil list
somehow, but where I doubt it.
First confirmed userland kill for Flexelint.
Sponsored by: Bright Path Solutions
|
|
|
|
|
|
| |
the same, but uses a non-extension option.
Sponsored by: Rachel Hestilow <rachel@jerkcity.com>
|
|
|
|
|
| |
is merely printf() but to stderr. This takes care of the caveat which lead
to the use of a vararg macro -- getting everything to stderr.
|
|
|
|
|
|
|
|
|
| |
variable length arguments to a macro. Bump version as this makes DEBUG
statements *always* go to stderr rather than sometimes stdout. There are
a few stragglers, which I will take care of as soon as I can. Mostly these
relate to the need-for-death-of some of the remote job code.
Nearby stylistic nits and XXX added/fixed where appropriate.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
o remove badly bit-rotted compat file that likely won't work on the systems
it purports to support.
|
| |
|
| |
|
|
|
|
|
|
| |
rip out ClientData/Address pointers and use standard types.
Obtained from: OpenBSD
|
|
|
|
|
|
| |
of NIL, NILLST, NILLGNODE, etc. with NULL.
Obtained from: OpenBSD
|
| |
|
| |
|
| |
|
|
|
|
| |
Obtained from: OpenBSD, sometimes indirected from NetBSD; myself
|
| |
|
|
|
|
|
|
|
|
| |
This will make a number of things easier in the future, as well as (finally!)
avoiding the Id-smashing problem which has plagued developers for so long.
Boy, I'm glad we're not using sup anymore. This update would have been
insane otherwise.
|
|
|
|
|
|
| |
of Christos' version of make(1) and add Id's.
Set straight by: Bruce Evans and Peter Wemm :)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Add the .PHONY, .PARALLEL, and .WAIT directives
- Added the -B and -m commandline flags
- misc. man page cleanups
- numerous job-related enhancements
- removed unused header file (bit.h)
- add util.c for functions not found in other envs.
- and a few coordinated whitespace changes
Special thanks to Christos Zoulas <christos@netbsd.org>
for help in the merge. A 'diff -ur' between Net and
FreeBSD now only contains sccsid-related diffs. :)
Obtained from: NetBSD, christos@netbsd.org, and me
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
problems in the process:
1. Quoting should work properly now. In particular, Chet's reported bash
make problem has gone away.
2. A lot of memory that just wasn't being free'd after use is now freed.
This should cause make to take up a LOT less memory when dealing with
archive targets.
3. Give proper credit to Adam de Boor in a number of files.
Obtained from: NetBSD (and Adam de Boor)
|
|
|