bug-datamash
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Patches for --vnlog support


From: Dima Kogan
Subject: Re: Patches for --vnlog support
Date: Sun, 07 Aug 2022 00:05:14 -0700
User-agent: mu4e 1.8.6; emacs 29.0.50

Alright. Tell me what you want.

Another way you can clarify the logic, and to make it amenable to
supporting future formats is to be more clear about why any given
if(vnlog) is there. Currently --vnlog does several things:

- -W
- -C
- -H
- trailing comments ignored
- numerical column references disabled
- column labels come from the first # comment

I think that's it? That's most of it, at least. The first 3 features set
variables in the code, and there are if() statements for those features.
We can do that for the other features too. For instance we can have a
variable "ignore_trailing_comments", and the relevant if() statement can
act on that, and not the "vnlog" variable. Then future formats that want
that behavior can set that variable, instead of introducing any more
if() statements.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]