In article <20210418085205.10b75f...@cvs.netbsd.org>, Roland Illig <source-changes-d@NetBSD.org> wrote: >-=-=-=-=-=- > >Module Name: src >Committed By: rillig >Date: Sun Apr 18 08:52:04 UTC 2021 > >Modified Files: > src/usr.bin/xlint/lint1: err.c externs1.h lint1.h > >Log Message: >lint: add error_at, warning_at, message_at > >Right now there are several places in the code that use the global >variable curr_pos for passing a parameter to the diagnostic functions. >That's not what global variables are for. > >Make it easy for the code to migrate to the parameter-passing style. >
Isn't it better to pass "const pos_t *", instead of pos_t which needs to copy the struct? christos