| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
Reviewed by: rodrigc
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
spaces in values. Without this change, the following valid
call broke due to parsing of .MAKEFLAGS in bsd.symver.mk:
cd /usr/src/lib/libc && make -n DEBUG_FLAGS="-DFOO -DBAR"
Spotted by: Igor Sysoev
Submitted by: Maxim Dounin, ru
MFC after: 1 week
|
|
|
|
|
|
|
|
| |
context (and only in one place to substitute the .for variable). Therefor
there is no need to pass the context as a parameter.
Patch: 7.197
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
|
|
| |
only one variable and Var_Subst() which substitutes all. Split out the
test whether a variable should not be expanded into match_var().
Make access to the input string consistently using str[]. Remove two
unused functions: Var_GetTail() and Var_GetHead().
Patches: 7.184-7.189
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
|
|
|
|
| |
on every line that starts with a dot use a minimal perfect hash
function and a single strcmp() on the first word after the dot
to find out whether it is really a directive call and, if yes, which
one. Then directly dispatch to a handler function for that directive
(or fall through to the dependency handling code). This makes the
directive parse a little bit more strict about the syntax: the directive
word must be followed by a character that is not alphanumerical and not
an underline (making .undefFOO illegal); .endif and .else can only be
followed by comments.
|
|
|
|
|
|
|
|
| |
appropriate.
Patch: 7.147-7.151
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
Remove an unused struct definition.
|
|
|
|
|
|
|
|
| |
of a char *.
Patch: 7.49
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
| |
and so get rid of the ForExec helper function by inlining it
into For_Run.
|
|
|
|
| |
Add a couple of comments.
|
|
|
|
|
|
| |
of empty lines in appropriate places.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
| |
given by a pointer to the start of the string and a pointer one behind
the end.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
and use it thoughout the code.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
a NULL to the function. Delete the now unused local variables.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
obfuscates the code.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
| |
itself. This will ease constification (think of what 'const Ptr foo'
means if Ptr is a pointer to a struct).
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>
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
|
| |
the structs itself not of pointers to them. This will simplify constification.
Checked by: diff on the object files
|
| |
|
|
|
|
| |
to change to size_t in a couple of other places too.
|
|
|
|
|
|
| |
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)
|
|
|
|
|
|
| |
to the list functions for better readability.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
|
|
|
| |
the .for loop:
- Replaced four global variables in parse.c with one.
- Made Parse_FromString() accept the "lineno" as an argument.
- Fixed line numbering when there are escaped newlines in the
body of the .for loop.
Adopted from: NetBSD
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make this true in the .for loops too. The following fragment,
FOO= foo bar
all:
.for f in ${FOO}
@echo ${f}
.endfor
when run as "make FOO=xxx" should print "xxx". (OpenBSD had
this bug fixed for some time.)
|
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
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.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
rip out ClientData/Address pointers and use standard types.
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 :)
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
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)
|
|
|