Age | Commit message (Collapse) | Author |
|
Since we don't really know how to change Var_Subst and dependent functions,
we cheat, and copy the string to a buffer instead.
Clean-up the parser a bit: reorganize the include files lookup function:
introduce a resolve_include_filename function that figures out the full name
of the file, introduce a handle_include_file that does all the include file
handling, and rename the functions into handle_xxx_include, so that we
can remove most comments.
Rename ParseIsCond into handle_bsd_command, cut most of its code into
constituent functions.
Semantic changes:
- follow the rules that, if an optional construct does not parse correctly,
then it's not a real extension, and give the normal parser a chance to figure
it out. If the syntax checks out, semantic errors ARE real errors.
- allow variables to not be defined in include file names, no reason not to.
okay miod@
|
|
VAR_GLOBAL context, so make it the common case: rename the basic functions
to Var_Set_with_ctxt/Var_Append_with_ctxt, define Var_Set and Var_Append
as macros that specify VAR_GLOBAL, and use these.
okay miod@
|
|
okay miod@
|
|
Var_Parse and Var_ParseSkip (the code that figures out the variable name
up to eventual modifiers).
okay miod@
|
|
- remove globals curdir and objdir, stuff them into a structure, and pass
it around to the relevant functions.
- cut up most of main code into functions:
figure_out_MACHINE/ARCH, figure_out_CURDIR, setup_CURDIR_OBJDIR, setup_VPATH,
no_fd_limits, read_makefile_list, read_all_make_rules...
- simplify the code used to figure out curdir/objdir.
No semantic changes.
okay miod@
|
|
okay miod@
|
|
state and using it, leading to a simpler loop and less code.
okay miod@
|
|
|
|
|
|
|
|
|
|
ok jmc@
|
|
|
|
|
|
|
|
|
|
inspired by zinovik@cs.karelia.ru
|
|
list one of them as a reference in SEE ALSO;
|
|
|
|
turned off. This is consistent with historic UNIX behavior.
|
|
the list of scripting windows. Appears to be a cut and paste error.
OK deraadt@
|
|
1.6.9 has changed relative to older versions. Sudo now starts
commands with a minimal environment containing the variables in the
env_keep and env_check lists. This behavior is configurable in the
sudoers file. Please see the "SECURITY NOTES" section in the sudo
manual.
|
|
|
|
comes from route(8) which already had the -T option. OK henning@
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
and keeping track of a length, we just advance the char *, and ditch
the length. We can still get the length at the end of the top-level
functions to satisfy existing interfaces.
Much simpler code, less error-prone.
Okay millert@
|
|
get_expanded_value.
Extend the code a bit to be much more thorough in case of a recursive
expansion: shows exactly the cycle of variable names involved.
okay millert@
|
|
length, simplifies code.
(warns a bit, symptom of some further issues to fix).
okay millert@
|
|
ok jmc@
|
|
(causes .for loops to give weird results very infrequently, memory handling
was slightly wrong, but not enough to break things thoroughly).
noticed by naddy@
|
|
Check the rtm_version before trying to print an entry. If the rtmsg has a
different version skip it and don't try to print it. Solves a SIGSEGV I have
triggered with one of my scarier diffs. OK henning@
|
|
stays defined.
|
|
|
|
- rename a few functions in var.c to names that make more sense.
- introduce Var_Deletei because it makes more sense.
- rewrite .for loops to use a local LoopVar construct to avoid looking up more
stuff.
- reformat var.c to near K&R
- rewrite most comments in var.c
shown to a few people. millert@ gave me his okay since it passes through
ports correctly.
|
|
|
|
- document the difference between "play" and "cdplay" more clearly
ok mjc
|
|
> xcalloc is unneeded here since i is always 0 and we always use the
> memory after initializing it.
>
> Initial diff from Igor Zinovik.
>
> OK niallo and xsa.
|
|
memory after initializing it.
Initial diff from Igor Zinovik.
OK niallo and xsa.
|
|
|
|
|
|
|
|
Input and OK ray@.
|
|
OK joris@.
|