attachment
<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class="">
<div><br class=""><blockquote type="cite" class=""><div class="">On Dec 18, 2020, at 2:26 PM, Michael Sweet <<a href="mailto:msweet@msweet.org" class="">msweet@msweet.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Smith,<br class=""><br class=""><blockquote type="cite" class="">On Dec 18, 2020, at 4:16 PM, Kennedy, Smith (Wireless & IPP Standards) via ipp <<a href="mailto:ipp@pwg.org" class="">ipp@pwg.org</a>> wrote:<br class="">...<br class="">I think this ABNF definition should be fixed to be more like this to limit the set of values:<br class="">finisher-presentonoff = "presentonoff" "=" "other" / "on" / "off" / "notPresent"<br class=""> ; string value as an alpha string mapped directly from ; PresentOnOff in [RFC3805]<br class=""></blockquote><br class="">That would be fine with me - these mappings often get pretty obtuse and including the list of values in the ABNF would be a welcome improvement for me...<br class=""><br class=""><blockquote type="cite" class="">Perhaps we need to do it with two rules:<br class=""><br class="">finisher-presentonoff = "presentonoff" "=" 1*presentonoffvalue<br class="">finisher-presentonoff-value = "other" / "on" / "off" / "notPresent"<br class=""> ; string value as an alpha string mapped directly from ; PresentOnOff in [RFC3805]<br class=""></blockquote><br class="">Doesn't 1*SOMETHING allow one or more values? If you really want to split the values out (not necessary IMHO) you just need "finisher-presentonoff-value".</div></div></blockquote><br class=""></div><div>You are right about the "1*" part. I'll use the first proposal with some added hard wraps and a clarified origin:</div><div><br class=""></div><div><div><font face="Courier New" class="">finisher-presentonoff = "presentonoff" "=" "other" / "on" /</font></div><div><font face="Courier New" class=""> "off" / "notPresent"</font></div><div><font face="Courier New" class=""> ; string value as an alpha string mapped directly from</font></div><div><font face="Courier New" class=""> ; finDevicePresentOnOff in [RFC3806] which imports the</font></div><div><span style="font-family: "Courier New";" class=""> ; definition of PresentOnOff from [RFC3805]</span></div><div class=""><br class=""></div></div></body></html>