summaryrefslogtreecommitdiffstats
path: root/usr.bin/make
Commit message (Collapse)AuthorAgeFilesLines
* Scheduled mdoc(7) sweep.ru2005-01-111-3/+3
|
* NOSHARED -> NO_SHAREDru2004-12-211-1/+1
|
* Fix compilation for the USE_KQUEUE case.harti2004-12-201-0/+4
| | | | Submitted by: Emil Mikulic <emikulic@dmr.ath.cx>
* maxJobs is declared extern in job.h so it cannot be static in job.c.harti2004-12-201-1/+1
| | | | | PR: bin/75210 Submitted by: Andreas Jochens; Robert Millan <rmh@debian.org>
* Remove all the cleanup functions. There is no reason to free memoryharti2004-12-1710-185/+6
| | | | | just before exiting (especially given the number of memory leaks) - it just costs time.
* Fix a long-standing bug when make(1) is passed the -V andru2004-12-161-8/+5
| | | | | | | | -f options in MAKEFLAGS environment variable, and some of these options are also specified on the command line. Thanks to: marcel Reviewed by: harti
* Instead of dynamically allocating list heads allocated them staticallyharti2004-12-1620-651/+530
| | | | | | | | 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.
* Remove a list that was just used to free all variables just before exiting.harti2004-12-101-12/+1
|
* Remove a list that used to hold all the GNodes just to be ableharti2004-12-101-39/+0
| | | | to free them just before exiting.
* Remove a useless list where just all command lines are stuffed onto, neverharti2004-12-101-4/+0
| | | | | used and just freed at the end. The idea might have been to be able to free all the strings, but what's the point to free just before exiting?
* Nobody actually checked the return codes from Lst_Append and Lst_Insertharti2004-12-094-43/+9
| | | | so don't return anything.
* If a path element directory was never opened it is not on the list ofharti2004-12-091-2/+2
| | | | | open directories so there is no need to remove it from there. This fixes a core dump introduced by removing the run-time check from Lst_Remove.
* Now that circular lists are gone remove stuff for them. Simplifyharti2004-12-081-10/+6
| | | | somewhat so that we can remove a local variable.
* No caller checks the return code from Lst_Remove, so don't return one.harti2004-12-082-30/+12
| | | | Simplify the algorithm now that circular lists are gone.
* Don't check the return code from Lst_Remove. There is no wayharti2004-12-081-5/+2
| | | | that the list's first element is not on the list.
* Remove return value from Lst_Concat. None of the callers ever checkedharti2004-12-082-36/+20
| | | | it. Remove stuff that was needed for circular lists.
* Don't free the second list in Lst_Concat for LST_CONCLINK; free itharti2004-12-082-3/+3
| | | | in the caller instead.
* Get rid of the sequential access feature of the lists. This was usedharti2004-12-0815-551/+98
| | | | | | | | 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.
* Constify the arguments to the list compare function. This temporarilyharti2004-12-089-39/+50
| | | | | | | | requires to make a copy of the filename in ReadMakefile and to duplicate two small functions in suff.c. This hopefully will go away when everything is constified. Submitted by: Max Okumoto <okumoto@ucsd.edu> (partly)
* Constify some calls of Buf_AddBytes.harti2004-12-081-3/+3
| | | | Submitted by: Max Okumoto <okumoto@ucsd.edu>
* Consify the arguments to str_concat. Remove the STR_DOFREE flag for thatharti2004-12-084-14/+9
| | | | | | | purpose and explicitely free the input string in the one place that was calling str_concat with that flag. Submitted by: Max Okumoto <okumoto@ucsd.edu>
* Typedefs of pointers to structs are evil. Make Lst and LstNode typedef ofharti2004-12-0731-230/+230
| | | | | | the structs itself not of pointers to them. This will simplify constification. Checked by: diff on the object files
* Constification of arguments passed to functions; no change on theharti2004-12-071-15/+15
| | | | | | | | resulting executable. Checked by: diff on original and new objects Submitted by: Max Okumoto <okumoto@ucsd.edu>
* Make needs no circular lists so remove them from the list code.harti2004-12-0721-144/+88
|
* Some constification which doesn't require code rewrites.harti2004-12-064-17/+13
|
* Style: fix indentation.harti2004-12-062-643/+660
|
* Remove an unused macro.harti2004-12-061-5/+0
|
* Constify arguments to Hash_FindEntry and Hash_CreateEntry.harti2004-12-062-6/+6
|
* Remove extra empty lines.harti2004-12-061-2/+0
|
* Style: fix indentation, prototypes for functions even in comment.harti2004-12-061-25/+13
|
* Remove an extra space.harti2004-12-031-1/+1
| | | | Submitted by: ru
* Fix breakage introduced on 64-bit platforms with my last commit. Needharti2004-12-0310-52/+53
| | | | to change to size_t in a couple of other places too.
* Plug a memory leak.harti2004-12-031-0/+1
|
* Put macro arguments in paranthesis.harti2004-12-032-2/+2
| | | | Submitted by: johan
* Make sizes to be of type size_t and correct function arguments thatharti2004-12-033-32/+32
| | | | should be Byte (as the numerous casts to Byte in the function calls show).
* Style: fix indentation, protect macro with do { } while (0).harti2004-12-032-135/+134
| | | | Checked with: diff on object file.
* Describe the .SHELL target.harti2004-12-031-1/+109
| | | | Reviewed by: ru
* Change the algorithm that matches the builtin shells from the name keywordharti2004-12-031-32/+16
| | | | | | | | | | | | | | of the .SHELL target. Formerly it used to select the shell with the longest common trailing substring, so that bash would select sh, but pocsh would select csh. Now an exact match is required so that specifying bash without also giving a path and the other keywords will give an error. PR: Submitted by: Reviewed by: ru Approved by: Obtained from: MFC after:
* Remove a double test for the same ptr != NULL and use the officialharti2004-12-011-7/+6
| | | | macro for this; reorder an if.
* Style: remove a lot of unnecessary casts, add some and spell the nullharti2004-12-0117-508/+498
| | | | | | pointer constant as NULL. Checked by: diff -r on the object files before and after
* Stylification: missing spaces, extra space after function names, castsharti2004-11-3034-1401/+1452
| | | | | | | | 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)
* Constify the array of predefined shells. It turns out that we need twoharti2004-11-302-45/+63
| | | | | | | versions of the structure definition for this: one with const char pointers, because otherwise gcc won't let us initialize the fields with constant strings, and one without the const, because we need to work with the structure.
* Change a couple of the primitve list functions to be macros. This changesharti2004-11-3013-756/+23
| | | | | | the semantic of Lst_Datum which formerly returned NULL when the argument node was NULL. There was only one place in the source that relied on this so change that place.
* Fix a bug that would truncate the full name of an archive member ifharti2004-11-291-2/+5
| | | | | | | | the length of happens to be larger than MAXPATHLEN. PR: bin/74368 Submitted by: Max Okumoto <okumoto@ucsd.edu> Obtained from: DragonFlyBSD
* Use typedefs for the types of the functions that are passed as argumentsharti2004-11-2910-17/+22
| | | | | | to the list functions for better readability. Submitted by: Max Okumoto <okumoto@ucsd.edu>
* Merge the contents of lstInt.h into the public lst.h. This let's us getharti2004-11-2929-355/+227
| | | | | | | rid of a lot of uneccesary casts and temporary variables that have just obfuscated the code. This also let's us implement a couple of the one- liner list functions as macros (the first one is Lst_IsEmpty) and simplify life once we start to throw consts on the code.
* Remove unused file.harti2004-11-261-300/+0
|
* Style: prototypes, un-register and remove some empty lines.harti2004-11-2625-133/+100
|
* Include sys/sysctl.h on all architectures not only i386.harti2004-11-252-2/+1
|
* Fix a very long-standing error in handling .SHELL targets: this targetharti2004-11-251-46/+125
| | | | | | | | | | | uses the brk_string function to parse the line. That function uses static storage for both the expanded string and the returned argv[] vector. The JobParseShell function simply stored away pointers into this static storage. On the next use of something like ${FOO:O} this storage would get overwritten with fatal results. This also allows us to make the shells[] array const bringing us one step further in making make WARNS=4 ready.
OpenPOWER on IntegriCloud