CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 16:16:32 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y externs1.h func.c Log Message: lint: rename ftflg to seen_fallthrough One less abbreviation to remember when reading the code. No functional change.

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 16:11:12 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: make the calls to clear_warning_flags stand out less Even though clear_warning_flags and its companions are implemented as macros,

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 15:30:58 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: clean up grammar for declaration after statement No functional change. To generate a diff of this commit: cvs rdiff -u -r1.185

CVS commit: src/usr.bin/cmp

2021-03-20 Thread Aleksey Cheusov
Module Name:src Committed By: cheusov Date: Sat Mar 20 14:27:47 UTC 2021 Modified Files: src/usr.bin/cmp: cmp.c Log Message: cmp.c: use C99 strtoll(3) instead of legacy strtoq(3) To generate a diff of this commit: cvs rdiff -u -r1.20 -r1.21 src/usr.bin/cmp/cmp.c Please

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 13:53:28 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: use proper boolean literals instead of 0/1 The code in the C grammar is generated by yacc and is not checked by lint's strict bool

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 13:25:31 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c Log Message: lint: update comment on setasm for C99 No functional change. To generate a diff of this commit: cvs rdiff -u -r1.151 -r1.152

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 13:22:06 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c Log Message: lint: reduce indentation and braces in merge_type_specifiers No functional change. To generate a diff of this commit: cvs rdiff -u

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 13:06:05 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c Log Message: lint: extract adjusting of the storage class into separate function No functional change. To generate a diff of this commit: cvs rdiff

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 13:00:43 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: lint1.h Log Message: lint: document the struct for declarations more precisely No functional change. To generate a diff of this commit: cvs rdiff -u -r1.75

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 11:33:50 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: remove unnecessary '%prec T_COMMA' from grammar My previous commit message was wrong in saying that the '%prec' was necessary. It

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 11:05:16 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: fix grammar for initialization Previously, the grammar syntactically accepted the following code: int var = .member =

CVS commit: src/usr.bin/xlint/lint1

2021-03-20 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Mar 20 10:32:43 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: align rule names in grammar with C99 No functional change. To generate a diff of this commit: cvs rdiff -u -r1.178 -r1.179

CVS commit: src/usr.bin/xlint/lint1

2021-03-19 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 18:17:46 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: improve debug loggin for initialization To generate a diff of this commit: cvs rdiff -u -r1.105 -r1.106

CVS commit: src/usr.bin/xlint/lint1

2021-03-19 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 17:37:57 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: rename designator_pop_name to designator_shift_name The entries are removed from the beginning, not from the end. No functional

CVS commit: src/usr.bin/xlint/lint1

2021-03-19 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 08:23:39 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: externs1.h Log Message: lint: use standard form of statement macro for debug_node No functional change. To generate a diff of this commit: cvs rdiff -u

CVS commit: src/usr.bin/xlint/lint1

2021-03-19 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 08:21:26 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c lint1.h tree.c Log Message: lint: rename in_bit to in_bits No functional change. To generate a diff of this commit: cvs rdiff -u -r1.148 -r1.149

CVS commit: src/usr.bin/xlint/lint1

2021-03-19 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 08:19:25 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c externs1.h tree.c Log Message: lint: rename tsize to type_size_in_bits The shorter name size_in_bits was already taken by the function-like macro with

CVS commit: src/usr.bin/xlint/lint1

2021-03-19 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 07:54:13 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: add reminder that C99 does not allow '{}' in initializer No functional change. To generate a diff of this commit: cvs rdiff -u

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 00:55:02 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y externs1.h init.c Log Message: lint: rename push_member and pop_member These two functions are supposed to model the designator that is used for

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 00:39:17 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: extend documentation about initialization No functional change. To generate a diff of this commit: cvs rdiff -u -r1.101 -r1.102

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 00:19:32 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: split initstack_pop_item into separate functions No functional change. To generate a diff of this commit: cvs rdiff -u -r1.100

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Mar 19 00:08:13 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: improve debug logging in initstack_push No functional change outside debug mode. To generate a diff of this commit: cvs rdiff -u

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Mar 18 23:45:20 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: split initstack_push into smaller functions No functional change. To generate a diff of this commit: cvs rdiff -u -r1.98 -r1.99

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Mar 18 23:37:31 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: clean up control flow in initstack_push No functional change. To generate a diff of this commit: cvs rdiff -u -r1.97 -r1.98

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Mar 18 23:23:40 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: extract extend_if_array_of_unknown_size from initstack_push No functional change. To generate a diff of this commit: cvs rdiff -u

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Mar 18 22:08:05 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: improve debug logging during initialization No functional change outside debug mode. To generate a diff of this commit: cvs rdiff

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Mar 18 22:05:33 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: make the debug log for nodes more readable The operator NAME has the name 'name', therefore no special case is needed. Having the

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Mar 18 21:56:35 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: ops.def Log Message: lint: rename operator 'const' to 'constant' The previous name could be too easily confused with the type qualifier 'const'. The operator

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Mar 18 20:55:58 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: reduce debug logging for initialization, update documentation No functional change outside debug mode. To generate a diff of this

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Mar 18 20:22:50 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: document how initialization works, improve debug logging No functional change outside debug mode. To generate a diff of this

CVS commit: src/usr.bin/join

2021-03-18 Thread Aleksey Cheusov
Module Name:src Committed By: cheusov Date: Thu Mar 18 19:47:41 UTC 2021 Modified Files: src/usr.bin/join: join.c Log Message: join.c: print usage after warning "illegal option..." as it was originally intended To generate a diff of this commit: cvs rdiff -u -r1.32

CVS commit: src/usr.bin/join

2021-03-18 Thread Aleksey Cheusov
Module Name:src Committed By: cheusov Date: Thu Mar 18 19:41:54 UTC 2021 Modified Files: src/usr.bin/join: join.c Log Message: join.c: explicitly convert -1 to u_long in order to fix compiler warnings To generate a diff of this commit: cvs rdiff -u -r1.31 -r1.32

CVS commit: src/usr.bin/ipcs

2021-03-18 Thread Aleksey Cheusov
Module Name:src Committed By: cheusov Date: Thu Mar 18 19:34:05 UTC 2021 Modified Files: src/usr.bin/ipcs: ipcs.c Log Message: ipcs.c: do not #include sys/inttypes.h header which is not necessary To generate a diff of this commit: cvs rdiff -u -r1.43 -r1.44

CVS commit: src/usr.bin/find

2021-03-18 Thread Aleksey Cheusov
Module Name:src Committed By: cheusov Date: Thu Mar 18 18:24:14 UTC 2021 Modified Files: src/usr.bin/find: function.c Log Message: find: use POSIX strtoll(3) instead of legacy strtoq(3) To generate a diff of this commit: cvs rdiff -u -r1.78 -r1.79

CVS commit: src/usr.bin/find

2021-03-18 Thread Aleksey Cheusov
Module Name:src Committed By: cheusov Date: Thu Mar 18 18:21:18 UTC 2021 Modified Files: src/usr.bin/find: find.h function.c Log Message: find: use POSIX type uint32_t instead of u_int32_t To generate a diff of this commit: cvs rdiff -u -r1.26 -r1.27

CVS commit: src/usr.bin/cksum

2021-03-18 Thread Aleksey Cheusov
Module Name:src Committed By: cheusov Date: Thu Mar 18 18:12:35 UTC 2021 Modified Files: src/usr.bin/cksum: cksum.c crc.c crc_extern.h extern.h print.c sum1.c sum2.c Log Message: cksum: use POSIX type uint32_t instead of u_int32_t To generate a diff of this

CVS commit: src/usr.bin/xlint/lint1

2021-03-18 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Mar 18 14:58:44 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c lint1.h Log Message: lint: document the initialization of an object in more detail This will help fixing the bugs that are currently demonstrated in

CVS commit: src/usr.bin/xlint/lint1

2021-03-17 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 15:45:31 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y externs1.h init.c Log Message: lint: add debug logging for C99-style initialization of arrays No functional change outside debug mode. To generate a

CVS commit: src/usr.bin/xlint/lint1

2021-03-17 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 15:37:42 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: move debug logging for the designator to the top of the code The debugging code is needed by the soon-to-be-added proper handling of

CVS commit: src/usr.bin/xlint/lint1

2021-03-16 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 02:24:06 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: lint1.h Log Message: lint: clean up documentation of dinfo_t No functional change. To generate a diff of this commit: cvs rdiff -u -r1.72 -r1.73

CVS commit: src/usr.bin/xlint/lint1

2021-03-16 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 02:18:03 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c lint1.h Log Message: lint: rename dinfo_t members to be more expressive While here, sync redundant but diverging comments, split multi-assignments and

CVS commit: src/usr.bin/xlint/lint1

2021-03-16 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 01:53:21 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c externs1.h Log Message: lint: move to_int_constant from the grammar to decl.c This way, the code is covered by running 'make lint'. The code

CVS commit: src/usr.bin/xlint/lint1

2021-03-16 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 01:38:31 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c externs1.h Log Message: lint: move main part of idecl over to decl.c and rename it No functional change. To generate a diff of this commit:

CVS commit: src/usr.bin/xlint/lint1

2021-03-16 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 01:22:55 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: replace a call to LERROR with lint_assert No functional change. To generate a diff of this commit: cvs rdiff -u -r1.172 -r1.173

CVS commit: src/usr.bin/xlint/lint1

2021-03-16 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 01:19:50 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: rename 'toicon' to 'to_int_constant' No functional change. To generate a diff of this commit: cvs rdiff -u -r1.171 -r1.172

CVS commit: src/usr.bin/xlint/lint1

2021-03-16 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 01:15:31 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c externs1.h func.c lex.c lint1.h mem1.c tree.c Log Message: lint: rename 'blklev' to 'block_level' No functional change. To

CVS commit: src/usr.bin/xlint/lint1

2021-03-16 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Mar 17 01:07:33 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: rename 'parn' to 'paren' in the grammar No functional change. To generate a diff of this commit: cvs rdiff -u -r1.169 -r1.170

CVS commit: src/usr.bin/make

2021-03-16 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Mar 16 16:21:27 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: fix documentation for ModChain The outer ModChain can be interrupted by an inner ModChain, but it continues to exist. No functional change.

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 20:00:51 UTC 2021 Modified Files: src/usr.bin/make: var.c src/usr.bin/make/unit-tests: varmod-indirect.mk Log Message: make: rename ApplyModifiersState to ModChain The new name accurately describes the

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 19:15:05 UTC 2021 Modified Files: src/usr.bin/make: var.c src/usr.bin/make/unit-tests: var-op-assign.mk Log Message: make: improve documentation of ApplyModifiersState No functional change. To generate a

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 19:02:57 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: document an example for a 'chain of modifiers' No functional change. To generate a diff of this commit: cvs rdiff -u -r1.888 -r1.889

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 18:56:38 UTC 2021 Modified Files: src/usr.bin/make: var.c src/usr.bin/make/unit-tests: varmod-assign.exp varmod-assign.mk Log Message: make: fix double varname expansion in the variable modifier '::=' This is

CVS commit: src/usr.bin/make/unit-tests

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 18:46:05 UTC 2021 Modified Files: src/usr.bin/make/unit-tests: varmod-assign.exp varmod-assign.mk Log Message: tests/make: demonstrate that the modifier '::=' expands the varname To generate a diff of this commit:

CVS commit: src/usr.bin/make/unit-tests

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 17:54:50 UTC 2021 Modified Files: src/usr.bin/make/unit-tests: varmod-loop.exp varmod-loop.mk Log Message: tests/make: convert varmod-loop to parse-time In case of unexpected failures, this provides the line number of

CVS commit: src/usr.bin/make/unit-tests

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 17:11:08 UTC 2021 Modified Files: src/usr.bin/make/unit-tests: varmod-loop.exp varmod-loop.mk Log Message: tests/make: add position marker in test 'varmod-loop' To generate a diff of this commit: cvs rdiff -u -r1.6

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 16:51:14 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: clean up documentation of ApplyModifiersState No functional change. To generate a diff of this commit: cvs rdiff -u -r1.886 -r1.887

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 16:45:30 UTC 2021 Modified Files: src/usr.bin/make: lst.c Log Message: make: fix documentation of Lst_MoveAll In CLEANUP mode, was originally meant to track memory allocations but is useful during debugging as well,

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 16:06:05 UTC 2021 Modified Files: src/usr.bin/make: lst.h Log Message: make: indent inline functions for lists No functional change. To generate a diff of this commit: cvs rdiff -u -r1.96 -r1.97

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 16:00:05 UTC 2021 Modified Files: src/usr.bin/make: enum.h Log Message: make: clean up header for runtime type information for enums An enum with 32 bits would lead to signed integer overflow anyway, so that

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 15:39:13 UTC 2021 Modified Files: src/usr.bin/make: nonints.h var.c src/usr.bin/make/unit-tests: deptgt-makeflags.exp directive-export-impl.exp directive-unexport-env.exp

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 12:15:03 UTC 2021 Modified Files: src/usr.bin/make: arch.c cond.c meta.c nonints.h parse.c suff.c var.c src/usr.bin/make/unit-tests: cond-func-empty.mk recursive.mk varmod-defined.mk varmod-ifelse.mk

CVS commit: src/usr.bin/make

2021-03-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Mar 15 11:41:07 UTC 2021 Modified Files: src/usr.bin/make: cond.c nonints.h parse.c suff.c var.c Log Message: make: rename VARE_NONE to VARE_PARSE_ONLY The name 'NONE' described the bit pattern, which was not useful to

CVS commit: src/usr.bin/make/unit-tests

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 20:41:39 UTC 2021 Modified Files: src/usr.bin/make/unit-tests: var-eval-short.exp var-eval-short.mk Log Message: tests/make: document today's bug fixes in the test To generate a diff of this commit: cvs rdiff -u

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 20:23:29 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: fix documentation of VarFreeEnv No functional change. To generate a diff of this commit: cvs rdiff -u -r1.882 -r1.883

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 20:18:33 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: skip variable lookup for '::=' modifiers in parse-only mode This is just to keep the code consistent among the various variable modifiers.

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 20:12:16 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: separate parsing and evaluating for modifier '::=' No functional change. To generate a diff of this commit: cvs rdiff -u -r1.880 -r1.881

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 20:09:26 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: in parse-only mode, do not update the expression value in ':sh' No functional change outside debug mode. The other variable modifiers

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 20:03:56 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: do not evaluate the ':O' modifiers in parse-only mode No functional change in practical usage. Theoretically this change can be observed by

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 20:00:48 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: separate parsing and evaluating in the ':O' modifiers No functional change. To generate a diff of this commit: cvs rdiff -u -r1.877

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 19:29:37 UTC 2021 Modified Files: src/usr.bin/make: var.c src/usr.bin/make/unit-tests: var-eval-short.exp Log Message: make: do not evaluate modifier ':[...]' in parse-only mode In parse-only mode, variable

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 19:25:05 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: do not evaluate modifiers ':M' and ':N' in parse-only mode No functional change outside debug mode (-dv for ModifyWord_Match). To generate

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 19:21:29 UTC 2021 Modified Files: src/usr.bin/make: var.c src/usr.bin/make/unit-tests: var-eval-short.exp var-eval-short.mk Log Message: make: do not return unevaluated 'else' part from the ':?' modifier No

CVS commit: src/usr.bin/make/unit-tests

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 19:16:41 UTC 2021 Modified Files: src/usr.bin/make/unit-tests: var-eval-short.exp var-eval-short.mk Log Message: tests/make: add test for the ':?' modifier in parse-only mode The debug output for this scenario will

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 18:30:24 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: do not evaluate ':gmtime' and ':localtime' in parse-only mode No functional change. To generate a diff of this commit: cvs rdiff -u

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 18:23:44 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: don't evaluate several simple modifiers in parse-only mode This affects the modifiers ':E', ':H', ':P', ':Q', ':R', ':T', ':hash', ':q',

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 18:10:57 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: skip strdup when parsing an irrelevant ':L' modifier No functional change. To generate a diff of this commit: cvs rdiff -u -r1.871 -r1.872

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 18:08:25 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: skip memcpy when parsing but not evaluating ':D' and ':U' No functional change, just a tiny bit of performance improvement, probably not

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 18:02:44 UTC 2021 Modified Files: src/usr.bin/make: var.c src/usr.bin/make/unit-tests: var-eval-short.exp Log Message: make: only evaluate the ':@' modifier if the result is actually used The test

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 17:38:24 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: eliminate common subexpression in ApplyModifier_Remember No functional change. To generate a diff of this commit: cvs rdiff -u -r1.868

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 17:34:50 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: merge duplicate code in ApplyModifier_Remember This way, parsing and evaluating the modifier is only written once in the code. The downside

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 17:27:28 UTC 2021 Modified Files: src/usr.bin/make: var.c src/usr.bin/make/unit-tests: varmod-remember.mk Log Message: make: do not expand the variable name in the ':_' modifier This edge case had been so

CVS commit: src/usr.bin/make/unit-tests

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 17:14:16 UTC 2021 Modified Files: src/usr.bin/make/unit-tests: varmod-remember.mk Log Message: tests/make: demonstrate unintended edge case for the ':_' modifier To generate a diff of this commit: cvs rdiff -u -r1.4

CVS commit: src/usr.bin/make/unit-tests

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 17:07:11 UTC 2021 Modified Files: src/usr.bin/make/unit-tests: varmod-remember.exp varmod-remember.mk Log Message: tests/make: convert test for ':_' modifier to parse time To generate a diff of this commit: cvs rdiff

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 16:43:31 UTC 2021 Modified Files: src/usr.bin/make: var.c src/usr.bin/make/unit-tests: var-eval-short.exp Log Message: make: only evaluate the ':_' modifier if the expression is needed See var-eval-short.mk:46

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 16:03:04 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: return failure in TryParseIntBase0 for empty string No functional change since the only caller of TryParseIntBase0 already handles all

CVS commit: src/usr.bin/make/unit-tests

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 16:00:07 UTC 2021 Modified Files: src/usr.bin/make/unit-tests: modword.exp modword.mk Log Message: make: add test for edge case in modifier ':[...]' TryParseIntBase0 wrongly returns successful for a string that does

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 15:43:31 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: separate parsing from evaluating for several modifiers This aligns the implementation of these modifiers with the requirements in the long

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 15:24:37 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: reduce indentation in ApplyModifier_SunShell No functional change. To generate a diff of this commit: cvs rdiff -u -r1.862 -r1.863

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 15:22:21 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: separate parsing and evaluating for modifier ':u' No functional change. To generate a diff of this commit: cvs rdiff -u -r1.861 -r1.862

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 15:19:15 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: separate parsing and evaluating for modifiers ':Q' and ':q' No functional change. To generate a diff of this commit: cvs rdiff -u -r1.860

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 15:15:29 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: extract duplicate code for parsing ':S' and ':C' No functional change. To generate a diff of this commit: cvs rdiff -u -r1.859 -r1.860

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 15:06:19 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: extract duplicate code for detecting a modifier's end No functional change. To generate a diff of this commit: cvs rdiff -u -r1.858

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 15:04:13 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: document how error handling should be done correctly Right now, when a variable expression cannot be parsed, the result of calling Var_Subst

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 11:15:37 UTC 2021 Modified Files: src/usr.bin/make: var.c Log Message: make: sort modifiers in ApplyModifier alphabetically No functional change. To generate a diff of this commit: cvs rdiff -u -r1.856 -r1.857

CVS commit: src/usr.bin/make

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 10:57:13 UTC 2021 Modified Files: src/usr.bin/make: var.c src/usr.bin/make/unit-tests: opt-debug-lint.exp opt-debug-lint.mk Log Message: make: fix wrong expression evaluation in -dL mode The modifier ':C' now

CVS commit: src/usr.bin/make/unit-tests

2021-03-14 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 10:45:51 UTC 2021 Modified Files: src/usr.bin/make/unit-tests: Makefile opt-debug-lint.exp opt-debug-lint.mk Log Message: make: demonstrate wrong expression evaluation in -dL mode To generate a diff of

CVS commit: src/usr.bin/indent

2021-03-13 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 05:26:42 UTC 2021 Modified Files: src/usr.bin/indent: pr_comment.c Log Message: indent: clean up check_size_comment The additional parameter last_bl_ptr was only necessary because the last blank was stored as a

CVS commit: src/usr.bin/indent

2021-03-13 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 04:52:10 UTC 2021 Modified Files: src/usr.bin/indent: pr_comment.c Log Message: indent: remove trailing whitespace To generate a diff of this commit: cvs rdiff -u -r1.33 -r1.34 src/usr.bin/indent/pr_comment.c Please

CVS commit: src/usr.bin/indent

2021-03-13 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 04:42:17 UTC 2021 Modified Files: src/usr.bin/indent: pr_comment.c Log Message: indent: clean up target column computation in process_comment No functional change. To generate a diff of this commit: cvs rdiff -u

CVS commit: src/usr.bin/indent

2021-03-13 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Mar 14 01:44:37 UTC 2021 Modified Files: src/usr.bin/indent: io.c Log Message: indent: make compute_code_indent more readable The '?:' operator computing the factor was too hard to read. When quickly scanning the code, the

<    2   3   4   5   6   7   8   9   10   11   >