simon-git: putty (main): Simon Tatham

Commits to Tartarus hosted VCS tartarus-commits at lists.tartarus.org
Sun Oct 30 10:38:49 GMT 2022


TL;DR:
  79b1cb22 testcrypt: remove some pointless macros.

Repository:     https://git.tartarus.org/simon/putty.git
On the web:     https://git.tartarus.org/?p=simon/putty.git
Branch updated: main
Committer:      Simon Tatham <anakin at pobox.com>
Date:           2022-10-30 10:38:49

commit 79b1cb226777783a06de481b703b47abff861bef
web diff https://git.tartarus.org/?p=simon/putty.git;a=commitdiff;h=79b1cb226777783a06de481b703b47abff861bef;hp=f9a8213d956fe4276a60a691f5ce5e224021834c
Author: Simon Tatham <anakin at pobox.com>
Date:   Sun Oct 30 08:54:16 2022 +0000

    testcrypt: remove some pointless macros.
    
    I just happened to notice ARG1 and ARGN in the code that builds the
    dispatch table in process_line(), which aren't used at all, because
    they date from a previous version of the testcrypt-func.h macro
    system. They were supposed to be replaced everywhere with the unified
    ARG.
    
    So why didn't the missing definition of ARG break anything? Because
    ARG only ever appears in the variadic part of a FUNC_INNER call - and
    for this particular trawl of testcrypt-func.h, the variadic part isn't
    ever used in the macro expansion in the first place. So there's no
    need to define ARG and VOID to anything at all, not even the empty
    string.

 test/testcrypt.c | 5 -----
 1 file changed, 5 deletions(-)



More information about the tartarus-commits mailing list