This update addresses a couple of minor, offline comments from David Marker.
Original webrevs: http://cr.opensolaris.org/~mjnelson/webrev.shortcddl/ http://cr.opensolaris.org/~mjnelson/webrev.shortcddl_tests/ Tests changes remain as originally provided. I think jmcp reviewed them already. Updated and incremental code webrevs: http://cr.opensolaris.org/~mjnelson/webrev.shortcddl.2/ http://cr.opensolaris.org/~mjnelson/webrev.shortcddl.incr/ Mostly I'm looking for signoff from Dave that his concerns have been addressed. The old tests pass for both old and new code. The new tests pass for the new code, and fail appropriately for the old code. --Mark > Mark J. Nelson wrote: >> This is for >> >>> 6833294 hg nits should accept shortened CDDL block >>> http://bugs.opensolaris.org/bugdatabase/view_bug.do?bug_id=6833294 >> >> Design notes: I extended the generic comment block checker to take one >> or more valid text blocks. The mapfile comment checker only uses one >> block, and the cddl checker now uses two. When passing multiple, >> valid text blocks, the canonical form should be listed first. >> >> The mapfile checker did not previously warrant its own test, because >> the block checking logic that had been refactored from the original >> cddl checker was thoroughly exercised by the cddl check tests. That's >> no longer true, so I wrote a separate mapfile test script. >> >> The webrevs, one each for code and tests: >> >>> http://cr.opensolaris.org/~mjnelson/webrev.shortcddl/ >>> http://cr.opensolaris.org/~mjnelson/webrev.shortcddl_tests/ >> >> ON folks: I plan to make Cadmium accept either form of the CDDL, and >> not make this configurable. If you disagree, speak up now (or even >> better, speak up a day or two ago when I sent such a question to you >> internally...) >> >> --Mark >> > >
