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

2021-01-15 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 15 23:43:51 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: func.c lint1.h tree.c Log Message: lint: merge duplicate code for non-zero detection To generate a diff of this commit: cvs rdiff -u -r1.57 -r1.58

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

2021-01-13 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Jan 14 07:34:48 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: scan.l Log Message: lint: fix enum type in the lexical analysis part Detected by Clang, reported by riastradh: > scan.l:144:29: error: implicit conversion

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

2021-01-11 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Jan 11 20:04:01 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: split typeok into several functions To generate a diff of this commit: cvs rdiff -u -r1.149 -r1.150 src/usr.bin/xlint/lint1/tree.c

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

2021-01-11 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Jan 11 19:29:49 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c emit1.c externs1.h tree.c Log Message: lint: sprinkle a few const modifiers throughout the code To generate a diff of this commit: cvs rdiff -u

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

2021-01-10 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 10 18:22:52 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c Log Message: lint: split check_global_variable into several functions This reduces the indentation and allows the messages in the comments to contain

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

2021-01-10 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 10 14:12:48 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c main1.c Log Message: lint: remove redundant parentheses around return value To generate a diff of this commit: cvs rdiff -u -r1.117 -r1.118

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

2021-01-10 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 10 14:09:57 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c externs1.h Log Message: lint: change return type of eqptrtype to bool To generate a diff of this commit: cvs rdiff -u -r1.116 -r1.117

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

2021-01-10 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 10 14:07:34 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c externs1.h Log Message: lint: rename functions with short names To generate a diff of this commit: cvs rdiff -u -r1.137 -r1.138

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

2021-01-10 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 10 12:05:07 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: Makefile Log Message: lint: automate adding a test for lint1 To generate a diff of this commit: cvs rdiff -u -r1.57 -r1.58 src/usr.bin/xlint/lint1/Makefile

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

2021-01-10 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 10 11:17:53 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: externs1.h func.c tree.c Log Message: lint: rename cvtcon to convert_constant No functional change. To generate a diff of this commit: cvs rdiff -u -r1.50

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 10 00:12:50 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: remove redundant braces in is_confusing_precedence This nicely aligns the different branches, in which "l" and "r" are swapped. No

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 23:18:19 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: push down complexity from typeok to typeok_shr Contrary to the comment in typeok, the types of the expressions before promotions and

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 23:02:51 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: move pointer subtypes from typeok to the sub-functions The code in typeok is already complicated enough. Only few of the checks

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 22:19:11 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: oper.c ops.def Log Message: lint: make the table containing the operator properties more readable The C preprocessor does not require its arguments to be

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 21:37:44 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: Makefile Added Files: src/usr.bin/xlint/lint1: oper.c Removed Files: src/usr.bin/xlint/lint1: ops.c Log Message: lint: rename ops.c to oper.c

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 19:13:17 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: externs1.h tree.c Log Message: lint: change return type of typeok to bool No functional change. To generate a diff of this commit: cvs rdiff -u -r1.49

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 19:07:07 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: split typeok into several smaller functions This reduces the number of local variables from 16 to around 5, in most of the smaller

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 18:21:08 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: extract typeok_amper from typeok No functional change. To generate a diff of this commit: cvs rdiff -u -r1.138 -r1.139

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 18:15:14 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: extract typeok_incdec from typeok No functional change. To generate a diff of this commit: cvs rdiff -u -r1.137 -r1.138

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 17:36:10 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: op.h ops.def Log Message: lint: reorder table of operator properties The 4 "requires" properties are now listed together, in descending strictness. To

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

2021-01-09 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 13:12:13 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c externs1.h Log Message: lint: rename ename to enumeration_constant To generate a diff of this commit: cvs rdiff -u -r1.134 -r1.135

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

2021-01-08 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 03:28:47 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: align grammar rules with C99 To generate a diff of this commit: cvs rdiff -u -r1.133 -r1.134 src/usr.bin/xlint/lint1/cgram.y

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

2021-01-08 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 03:08:54 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y func.c Log Message: lint: expand abbreviations in grammar rules stmnt -> statement comp -> compound The abbreviation "stmnt" was non-standard, and

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

2021-01-08 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 9 02:38:27 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y scan.l Log Message: lint: rename T_MULT to T_ASTERISK In the early phase of lexical analysis, the '*' does not mean multiplication, therefore its name

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

2021-01-05 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 23:50:29 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: Makefile op.h ops.def Added Files: src/usr.bin/xlint/lint1: ops.c Removed Files: src/usr.bin/xlint/lint1: Makefile.ops-c Makefile.ops-h mkops

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

2021-01-05 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 21:40:42 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: mkops ops.def Log Message: lint: swap columns 1 and 2 in ops.def List the ID of a record first, followed by the describing fields. To generate a diff of

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

2021-01-05 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 21:32:47 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: mkops ops.def Log Message: lint: make generating the operators table simpler The generated files are exactly the same as before. To generate a diff of this

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

2021-01-05 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 17:37:57 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: make check_precedence_confusion simpler In C, only binary operators have possibly confusing precedence. All binary operators have

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

2021-01-05 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 17:13:44 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: scan.l Log Message: lint: in debug mode, log every newline This helps to quickly see where in the source file the parser currently is. Previously, the

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

2021-01-05 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 16:34:37 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: ops.def Log Message: lint: fix inconsistent whitespace in ops.def To generate a diff of this commit: cvs rdiff -u -r1.2 -r1.3 src/usr.bin/xlint/lint1/ops.def

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 07:37:41 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: print.c Log Message: lint: add missing strings for operators It's difficult to keep these lists in sync when they are spread over several files. The lists

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 00:22:05 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: func.c Log Message: lint: clean up comments To generate a diff of this commit: cvs rdiff -u -r1.50 -r1.51 src/usr.bin/xlint/lint1/func.c Please note that

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 00:17:21 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: extract code for determining possible precedence confusion The function check_precedence_confusion was pretty long, and right in the

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Jan 5 00:02:52 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: remove unused TBUILTIN_SIZEOF from the C grammar This token is not generated by the lexer, and it was misspelled anyway. All other

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Jan 4 23:58:19 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: op.h tree.c Log Message: lint: precedence confusion is only possible with binary operators No functional change. The operator table in ops.def states that

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Jan 4 23:17:04 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: add more rationale for removing effect-less code To generate a diff of this commit: cvs rdiff -u -r1.129 -r1.130

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Jan 4 22:33:47 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: replace LERROR with lint_assert in check_expr_misc This reduces the visual clutter. There is no reason for anyone to modify the

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Jan 4 22:29:01 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: mem1.c Log Message: lint: reduce comment for xgetblk to the actually interesting part To generate a diff of this commit: cvs rdiff -u -r1.21 -r1.22

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Jan 4 21:30:07 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: fix bugs in dprint_node in debug mode (since today) The node was dereferenced before the null check. GCC 5.5 didn't warn about this

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Jan 4 21:17:31 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: move dprint_node to the top of the file It now resides right below dumpnode, which implements the same idea but uses a fixed-size

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

2021-01-04 Thread Roland Illig
Module Name:src Committed By: rillig Date: Mon Jan 4 17:06:21 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: in debug mode, print node tree for precedence >From the code alone, it is too difficult to see how the various internal operators

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 21:33:50 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: use consistent indentation for C grammar To generate a diff of this commit: cvs rdiff -u -r1.129 -r1.130

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 20:44:55 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: Makefile.err-msgs-h Log Message: lint: fix code generation from err.c That file does not need to be in the current directory. To generate a diff of this

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 20:38:26 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y externs1.h tree.c Log Message: lint: rename funcarg and funccall to longer names >From the previous short names, it was no obvious that these

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 20:31:09 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y externs1.h init.c tree.c Log Message: lint: rename functions that create nodes To generate a diff of this commit: cvs rdiff -u -r1.127 -r1.128

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 20:14:38 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c emit1.c externs1.h func.c Log Message: lint: rename prflstrg to printflike_argnum, likewise for scflstrg To generate a diff of this commit: cvs rdiff

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 20:04:09 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: func.c lint1.h Log Message: lint: rename cstk to cstmt Most of the code that deals with control statements is only interested in the innermost control

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 19:15:36 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c lint1.h Log Message: lint: rename t_ispacked to t_packed and d_ispacked to d_packed It's shorter, and the other flags of the type or declaration also

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 19:10:47 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c init.c lint1.h tree.c Log Message: lint: rename type.t_isfield to t_bitfield To generate a diff of this commit: cvs rdiff -u -r1.107 -r1.108

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 18:48:37 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: Makefile decl.c err.c lint1.h Added Files: src/usr.bin/xlint/lint1: Makefile.err-msgs-h Log Message: lint: in DEBUG mode, verify printf parameters for

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 18:35:51 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: print.c tree.c Log Message: lint: remove redundant include of externs1.h It is already included by lint1.h. To generate a diff of this commit: cvs rdiff -u

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 17:42:45 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c err.c externs1.h init.c scan.l tree.c Log Message: lint: let gnuism and c99ism return void instead of int The return value was only used in a single

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

2021-01-03 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sun Jan 3 17:11:19 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c Log Message: lint: separate error and warning for 27 and 58 Even though this results in more lines of code, the benefit is that the message text in the

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

2021-01-02 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 2 18:44:58 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c externs1.h func.c Log Message: lint: convert bitfieldtype_ok from int to bool No functional change intended, except for the output in debug mode. To

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

2021-01-02 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 2 18:26:44 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c emit1.c lint1.h tree.c Log Message: lint: fix lint warning 161 "constant in conditional context" To generate a diff of this commit: cvs rdiff -u

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Sat Jan 2 01:12:39 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: check-msgs.lua Log Message: lint: fix typo in check-msgs tool To generate a diff of this commit: cvs rdiff -u -r1.4 -r1.5

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 20:02:56 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: add debug logging for initialization using named members To generate a diff of this commit: cvs rdiff -u -r1.54 -r1.55

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 19:28:51 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: document that C99-style initialization is necessarily buggy To generate a diff of this commit: cvs rdiff -u -r1.53 -r1.54

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 19:15:58 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: init.c lint1.h Log Message: lint: un-export struct istk To generate a diff of this commit: cvs rdiff -u -r1.52 -r1.53 src/usr.bin/xlint/lint1/init.c cvs

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 14:51:44 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: Makefile Log Message: lint: remove custom LINTFLAGS These flags are missing the options -g (GNUisms) and -S (C9X mode). Without these flags, NetBSD's system

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 14:11:20 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c Log Message: lint: combine duplicate case branches in popdecl This works around GCC 9, which didn't see that the following statement cannot terminate

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

2021-01-01 Thread Robert Elz
Module Name:src Committed By: kre Date: Fri Jan 1 13:43:34 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c Log Message: Build breakage fix (hopefully) - brain dead gcc. NFCI. To generate a diff of this commit: cvs rdiff -u -r1.95 -r1.96

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 11:41:01 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y check-msgs.lua decl.c func.c init.c tree.c Log Message: lint: add missing redundant messages in source code To generate a diff of this

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 11:14:06 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: func.c Log Message: lint: complete message 203 for case labels To generate a diff of this commit: cvs rdiff -u -r1.44 -r1.45 src/usr.bin/xlint/lint1/func.c

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 11:09:41 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y func.c scan.l tree.c Log Message: lint: rename tokens for left and right parentheses To generate a diff of this commit: cvs rdiff -u -r1.125 -r1.126

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 11:01:03 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: func.c Log Message: lint: extract main part of case_label into separate function To generate a diff of this commit: cvs rdiff -u -r1.42 -r1.43

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 10:55:28 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y externs1.h func.c Log Message: lint: split label handling into separate functions The only thing these cases have in common is the name "label" and

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 09:28:22 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c emit1.c scan.l Log Message: lint: replace simple LERROR with lint_assert To generate a diff of this commit: cvs rdiff -u -r1.123 -r1.124

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

2021-01-01 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 09:11:40 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c emit1.c func.c Log Message: lint: replace some more LERROR with lint_assert To generate a diff of this commit: cvs rdiff -u -r1.91 -r1.92

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

2020-12-31 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 01:38:14 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: clean up warn_incompatible_types Splitting the code arbitrarily in separate phases made the code harder to understand, both for

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

2020-12-31 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 01:29:30 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: decl.c Log Message: lint: remove redundant printf arguments To generate a diff of this commit: cvs rdiff -u -r1.89 -r1.90 src/usr.bin/xlint/lint1/decl.c

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

2020-12-31 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 01:26:02 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: cgram.y check-msgs.lua decl.c scan.l Log Message: lint: align comments with actual message, in the lexer and parser To generate a diff of this commit: cvs

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

2020-12-31 Thread Roland Illig
Module Name:src Committed By: rillig Date: Fri Jan 1 00:00:24 UTC 2021 Modified Files: src/usr.bin/xlint/lint1: check-msgs.lua decl.c func.c init.c tree.c Log Message: lint: align comments in code with actual messages Redundancy is bad. Especially in this case,

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

2020-12-31 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Dec 31 22:48:33 UTC 2020 Added Files: src/usr.bin/xlint/lint1: check-msgs.lua Log Message: lint: check messages and their IDs for consistency To generate a diff of this commit: cvs rdiff -u -r0 -r1.1

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

2020-12-31 Thread Roland Illig
Module Name:src Committed By: rillig Date: Thu Dec 31 16:19:05 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: mkops Log Message: lint: in mkops, replace printf with println One less backslash per line of code. To generate a diff of this commit: cvs rdiff -u -r1.10

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

2020-12-30 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 13:17:42 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c externs1.h func.c Log Message: lint: split cluparg into separate functions That function did more than its short name could express, and the

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

2020-12-30 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 12:22:52 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: replace LERROR with lint_assert This removes the redundancy of mentioning the function name in the error message. This redundancy

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

2020-12-30 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 11:56:10 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: decl.c func.c scan.l tree.c Log Message: lint: spell check comments To generate a diff of this commit: cvs rdiff -u -r1.85 -r1.86

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

2020-12-30 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 11:14:03 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: decl.c emit1.c func.c lint1.h scan.l Log Message: lint: un-abbreviate s_dpos, s_spos and s_upos To generate a diff of this commit: cvs rdiff -u -r1.83 -r1.84

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

2020-12-30 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 11:04:48 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: decl.c init.c lint1.h scan.l tree.c Log Message: lint: un-abbreviate s_field, s_keyw and s_xsym To generate a diff of this commit: cvs rdiff -u -r1.82 -r1.83

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

2020-12-30 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 10:56:51 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: cgram.y emit1.c init.c lint1.h print.c scan.l tree.c Log Message: lint: un-abbreviate parenthesized and _strg To generate a diff of this commit:

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

2020-12-30 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 10:49:10 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: decl.c func.c lint1.h scan.l tree.c Log Message: lint: rename remaining _nxt members to _next To generate a diff of this commit: cvs rdiff -u -r1.81 -r1.82

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

2020-12-30 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 10:35:38 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c lint1.h scan.l tree.c Log Message: lint: rename symt_t constants There's no need to abbreviate them, furthermore FMOS was imprecise. To

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 01:44:32 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: reduce nesting of function calls To generate a diff of this commit: cvs rdiff -u -r1.117 -r1.118 src/usr.bin/xlint/lint1/cgram.y

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 01:33:30 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: err.c externs1.h init.c lint1.h Log Message: lint: reduce verbosity of assertions Having 2 lines of source code per assertion is too much, especially since

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Wed Dec 30 01:02:38 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: cgram.y Log Message: lint: add debug logging to the parser Even with -DDEBUG and -DYYDEBUG, the debug output is not detailed enough to clearly see what

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 23:12:48 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: init.c lint1.h Log Message: lint: rename istk_t.i_cnt to i_remaining To generate a diff of this commit: cvs rdiff -u -r1.42 -r1.43

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 23:04:31 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: tree.c Log Message: lint: rename functions that had very short names To generate a diff of this commit: cvs rdiff -u -r1.99 -r1.100

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 20:56:28 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: untangle conditions in initstack_next_nobrace To generate a diff of this commit: cvs rdiff -u -r1.41 -r1.42

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 20:07:04 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: clean up debug logging for initializations To generate a diff of this commit: cvs rdiff -u -r1.40 -r1.41

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 19:57:44 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: make debug output for initializations more uniform To generate a diff of this commit: cvs rdiff -u -r1.39 -r1.40

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 19:09:53 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: remove redundant function prototypes To generate a diff of this commit: cvs rdiff -u -r1.38 -r1.39 src/usr.bin/xlint/lint1/init.c

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 19:02:16 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: improve debug output for initializing structs Still trying to find out where the wrong warning in d_struct_init_nested comes from.

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 17:29:31 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c externs1.h Log Message: lint: split complete_tag into separate functions To generate a diff of this commit: cvs rdiff -u -r1.115 -r1.116

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 16:59:12 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: split initstack_next into separate functions To generate a diff of this commit: cvs rdiff -u -r1.36 -r1.37

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 16:53:36 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: init.c Log Message: lint: split initstack_pop into separate functions To generate a diff of this commit: cvs rdiff -u -r1.35 -r1.36

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 16:48:53 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: cgram.y externs1.h init.c Log Message: lint: rename functions for handling the initialization stack To generate a diff of this commit: cvs rdiff -u -r1.114

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 12:29:03 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: scan.l Log Message: lint: remove redundant parentheses around return value To generate a diff of this commit: cvs rdiff -u -r1.96 -r1.97

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 11:54:56 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: lint1.h tree.c Log Message: lint: spell check To generate a diff of this commit: cvs rdiff -u -r1.32 -r1.33 src/usr.bin/xlint/lint1/lint1.h cvs rdiff -u

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

2020-12-29 Thread Roland Illig
Module Name:src Committed By: rillig Date: Tue Dec 29 10:24:22 UTC 2020 Modified Files: src/usr.bin/xlint/lint1: cgram.y decl.c externs1.h func.c main1.c tree.c Log Message: lint: rename functions that had very short names C99 guarantees that the first 31

<    1   2   3   4   5   6   7   8   >