Subj : feature request
To   : Andrew Leary
From : Michiel van der Vlist
Date : Fri Jul 27 2018 01:25:02

Hello Andrew,

On Thursday October 02 2014 14:10, you wrote to me:

 MvdV>> Perhaps it is better to let a human judge. But it definitely
 MvdV>> should be flagged as an error!

 AL> I just dug through the code.  Currently leading or trailing spaces in
 AL> the first 7 fields (keyword, number, name, location, sysop, telephone,
 AL> baudrate) of a nodelist line will be removed.

Leading and trailing spaces, but not spaces in the middle?

 AL> The issue here is an incorrect entry with a space embedded in the
 AL> flags field.  MakeNL has never done error checking on the flags,

An embedded space is a more general error than an unauthorized flag.

 AL>  which is why ERRFLAGS was developed in the first place.  I suppose I
 AL> could add basic error checking of the flags (probably limited to
 AL> flagging illegal characters such as spaces.)  Anything more extensive
 AL> would probably involve rewriting ERRFLAGS in C and merging it with
 AL> MakeNL.  That is a project that would require extensive amounts of
 AL> time to code, test, and debug.

It is doable. I have build errflags functionality in my pointlist assembler. It 
is open source...

 AL> Given that some of the current ZCs refuse to use ERRFLAGS, I don't
 AL> imagine they would want to have the equivalent added to MakeNL.

I agree.

Perhaps just flag any space in a line it as a warning. Or maybe just forget 
about it. It is a rare occasion that may never happen again once Joe is back 
home and corrects it.

It does not seem to break anything or it would have been noticed before...


Cheers, Michiel

--- GoldED+/W32-MSVC 1.1.5-b20130111
 * Origin: http://www.vlist.eu (2:280/5555)

.