--- - branch: MAIN date: Sun Jan 3 17:42:45 UTC 2021 files: - new: '1.106' old: '1.105' path: src/usr.bin/xlint/lint1/decl.c pathrev: src/usr.bin/xlint/lint1/decl.c@1.106 type: modified - new: '1.63' old: '1.62' path: src/usr.bin/xlint/lint1/err.c pathrev: src/usr.bin/xlint/lint1/err.c@1.63 type: modified - new: '1.45' old: '1.44' path: src/usr.bin/xlint/lint1/externs1.h pathrev: src/usr.bin/xlint/lint1/externs1.h@1.45 type: modified - new: '1.58' old: '1.57' path: src/usr.bin/xlint/lint1/init.c pathrev: src/usr.bin/xlint/lint1/init.c@1.58 type: modified - new: '1.111' old: '1.110' path: src/usr.bin/xlint/lint1/scan.l pathrev: src/usr.bin/xlint/lint1/scan.l@1.111 type: modified - new: '1.118' old: '1.117' path: src/usr.bin/xlint/lint1/tree.c pathrev: src/usr.bin/xlint/lint1/tree.c@1.118 type: modified id: 20210103T174245Z.dd4c9094a5fab94eeee489a6c4b21e716ad32c8f log: | lint: let gnuism and c99ism return void instead of int The return value was only used in a single case. Duplicating the condition for printing a message is ok in that case, since it makes all other places in the code simpler. The occasional "(void)" or "msg = " before the function call had hidden the calls from check-msgs.lua, which didn't check the message texts in such cases. module: src subject: 'CVS commit: src/usr.bin/xlint/lint1' unixtime: '1609695765' user: rillig