On 1/20/21 5:01 PM, Gerd Hoffmann wrote: > Hi, > >>> +/* Illegal request, Invalid value in parameter list */ >>> +extern const struct SCSISense sense_code_INVALID_PARAM_VALUE; > >> Pre-existing: the term 'illegal' is suspect in computer science (the >> code isn't breaking any laws); > > Indeed. It's named that way in the scsi specs though, and being > consistent with that is a reasonable thing too ...
In v2 I renamed 'illegal' in the description comments but not in the #definitions (which match the specs). Regards, Phil.