| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
to a fallthrough.
Patch: 7.82
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
|
| |
an else clause. Move the assignment to the lengthPtr down to
just before the return statements.
Patch: 7.81
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
replace FALSE by NULL.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
|
| |
necessary to NUL-terminated it, because we know the lengths
from the pointers.
Patch: 7.80
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
|
| |
functions: one for the single letter variables, one for the others
and one that does the recursive expansion.
Patches: 7.68-7.79
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
| |
of a char *.
Patch: 7.49
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
|
|
|
|
| |
after a return. Move assignments to {freePtr, dynamic, start} closer to the
return statements to clarify which variables are actually used for
communication between the losely coupled blocks of the code. Clear up
an if-expression to make common structures of the conditions clearer.
Use strchr instead of switch statements to check for a character beeing
a member of a set.
Patches: 7-62.2, 7-62.3, 7-64, 7-65.1, 7-65.2
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
| |
get it clearer what variables are actually needed.
Patches: 7.62.2, 7.62.3
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
| |
fix the prototype for Compat_RunCommand to take the actual argument
types instead of void *.
|
|
|
|
|
| |
and so get rid of the ForExec helper function by inlining it
into For_Run.
|
|
|
|
| |
Add a couple of comments.
|
|
|
|
| |
Noted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
committed together with the commit to dir.c:1.48.
|
|
|
|
| |
Lst_ForEach and Lst_Find.
|
|
|
|
|
|
|
| |
to the Lst_ForEach function this macro reduces the number of function
calls per invocation by N + 1 (where N is the number of list elements)
and increases code locality thereby increasing readability and
(maybe) performance.
|
|
|
|
|
| |
80 characters and slightly reorder functions to get rid of static
prototypes.
|
|
|
|
| |
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
| |
when looking into an already hashed archive, the code tried to use
the name shortened to the maximum length allowed for the archive.
Unfortunately it passed a buffer of junk to the hashing routine when
the name actually wasn't too long. Theoretically this could lead to
a false positive.
|
|
|
|
|
|
|
| |
it is actually needed. This makes clear in which subblocks the variables
are not needed and which can easier be split out.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
liveness clear for splitting up this monster function.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
whitespace and line up some variable definitions.
|
|
|
|
|
|
|
| |
startc, but only at the place where the initialisation is needed. Remove
a needless else.
Submitted by: Max Okumoto <okumoto@ucsd.edu> (partly)
|
|
|
|
| |
Submitted by: ru & Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
earlier instead of mixing its handling with other cases.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
condition positive and fix long lines.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
| |
to prepare for function splitting and slightly reorganise the code
in anticipation of Var_Subst returning a Buffer.
Submitted by: Max Okumoto <okumoto@ucsd.edu> (with slight changes)
|
| |
|
|
|
|
| |
Pointy hat to: harti
|
|
|
|
|
|
|
|
| |
to be split out into a function soon). Also there is no need to write
back the colon that we have NUL-ed - the string is going to be freed
anyway.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
a NULL.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
| |
use a more consistent style with regard to *str and str[0];
simplify code by introducing a temporary variable;
shift a break around and add braces where appropriate.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
instead of specifying the default size explicitely.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
with a comment.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
|
|
| |
Move some assignments nearer to where they actually used. Convert a loop
from a for() to a while() to make it clearer and add braces to the long
body of it. Split assignment from variable declaration.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
|
|
| |
of empty lines in appropriate places.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
Submitted by: stefanf
|
|
|
|
|
|
| |
should also be void.
Submitted by: stefanf
|
|
|
|
|
|
|
| |
parantheses. This helps editors to find its way through the horrible
mess of Var_Parse. Rewrite a for() loop into a while() to make it clearer.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
twisted.
|
|
|
|
| |
intendation. (A large part of the function was already at intendation 8).
|
|
|
|
|
|
| |
appropriate.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
| |
|
|
|
|
|
|
| |
where appropriate.
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
Submitted by: Max Okumoto <okumoto@ucsd.edu>
|
|
|
|
| |
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>
|