--- - branch: MAIN date: Sat Jan 30 20:53:29 UTC 2021 files: - new: '1.48' old: '1.47' path: src/usr.bin/make/buf.c pathrev: src/usr.bin/make/buf.c@1.48 type: modified - new: '1.39' old: '1.38' path: src/usr.bin/make/buf.h pathrev: src/usr.bin/make/buf.h@1.39 type: modified - new: '1.254' old: '1.253' path: src/usr.bin/make/cond.c pathrev: src/usr.bin/make/cond.c@1.254 type: modified - new: '1.265' old: '1.264' path: src/usr.bin/make/dir.c pathrev: src/usr.bin/make/dir.c@1.265 type: modified - new: '1.139' old: '1.138' path: src/usr.bin/make/for.c pathrev: src/usr.bin/make/for.c@1.139 type: modified - new: '1.518' old: '1.517' path: src/usr.bin/make/main.c pathrev: src/usr.bin/make/main.c@1.518 type: modified - new: '1.534' old: '1.533' path: src/usr.bin/make/parse.c pathrev: src/usr.bin/make/parse.c@1.534 type: modified - new: '1.784' old: '1.783' path: src/usr.bin/make/var.c pathrev: src/usr.bin/make/var.c@1.784 type: modified id: 20210130T205329Z.7ced51b6e0eaa9d16e553c84021e75c077dc4607 log: | make(1): split Buf_Destroy into Buf_Done and Buf_DoneData In all cases except one, the boolean argument to Buf_Destroy was constant. Removing that argument by splitting the function into two separate functions makes the intention clearer on the call site. It also removes the possibility for using the return value of Buf_Done, which would have made no sense. The function Buf_Done now pairs with Buf_Init, just as in HashTable and Lst. Even though Buf_Done is essentially a no-op, it is kept as a function, both for symmetry with Buf_Init and for clearing the Buffer members after use (this will be done only in CLEANUP mode, in a follow-up commit). module: src subject: 'CVS commit: src/usr.bin/make' unixtime: '1612040009' user: rillig