<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<font size="4"><font face="monospace">If "block" means multi-line,
and non-block means single-line, I'm confused as to why we have<br>
<br>
public record Mapping(String key, List<Fragment>
fragments) implements Printable {}<br>
but<br>
public record BlockMapping(String key, List<Printable>
printables) implements Printable {}<br>
<br>
where one has fragments and the other only has Printables.
Similarly, we have<br>
<br>
public record ValueList(String key, List<ConstantDesc>
values) implements Fragment {}<br>
public record BlockList(String key,
List<BlockMapping> blockMappings) implements Printable {}<br>
</font></font><br>
where again the List element differs. Is there no way to make the
block/non-block orthogonal to the payload type? That would allow us
to replace the two BlockXxx(...) with different payloads, with a
Block<T> wrapper as a formatting hint. <br>
<br>
<br>
<div class="moz-cite-prefix">On 7/25/2022 1:51 PM, Adam Sotona
wrote:<br>
</div>
<blockquote type="cite" cite="mid:CY4PR1001MB2150642A76D52BAAF1108DCF8C959@CY4PR1001MB2150.namprd10.prod.outlook.com">
<meta name="Generator" content="Microsoft Word 15 (filtered
medium)">
<style>@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}@font-face
{font-family:"Apple Color Emoji";
panose-1:0 0 0 0 0 0 0 0 0 0;}@font-face
{font-family:"Courier New \;color\:black";
panose-1:2 7 3 9 2 2 5 2 4 4;}p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
font-size:10.0pt;
font-family:"Calibri",sans-serif;}pre
{mso-style-priority:99;
mso-style-link:"HTML Preformatted Char";
margin:0cm;
margin-bottom:.0001pt;
font-size:10.0pt;
font-family:"Courier New",serif;}span.HTMLPreformattedChar
{mso-style-name:"HTML Preformatted Char";
mso-style-priority:99;
mso-style-link:"HTML Preformatted";
font-family:"Courier New",serif;}span.EmailStyle20
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}div.WordSection1
{page:WordSection1;}</style>
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US">Further
reduction will require additional info about formatting.
Actual list has very simple printers implementations:<o:p></o:p></span></p>
<p class="MsoNormal" style="background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="font-family:"Courier
New",serif;color:#336BDD">switch</span><span style="font-family:"Courier
New",serif;color:black"> (node) {<o:p></o:p></span></p>
<p class="MsoNormal" style="background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="font-family:"Courier
New",serif;color:#336BDD">case</span><span style="font-family:"Courier
New",serif;color:black">
<i>Value</i> v -><o:p></o:p></span></p>
<p class="MsoNormal" style="background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="font-family:"Courier
New",serif;color:#336BDD">case</span><span style="font-family:"Courier
New",serif;color:black">
<i>ValueList</i> vl -><o:p></o:p></span></p>
<p class="MsoNormal" style="background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="font-family:"Courier
New",serif;color:#336BDD">case</span><span style="font-family:"Courier
New",serif;color:black">
<i>Mapping</i> m <o:p></o:p></span></p>
<p class="MsoNormal" style="background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><span style="font-family:"Courier
New",serif;color:#336BDD">case</span><span style="font-family:"Courier
New",serif;color:black">
<i>BlockList</i> bl -><o:p></o:p></span></p>
<p class="MsoNormal" style="background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="font-family:"Courier
New",serif;color:#336BDD">case</span><span style="font-family:"Courier
New",serif;color:black">
<i>BlockMapping</i> bm -><o:p></o:p></span></p>
<p class="MsoNormal" style="background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="font-family:"Courier
New",serif;color:#336BDD">case</span><span style="font-family:"Courier
New",serif;color:black">
<i>Comment</i> c -><o:p></o:p></span></p>
<p class="MsoNormal" style="background:white"><span style="font-family:"Courier
New",serif;color:black"> }<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US">Replacing
all simple values with list of values will wrap everything
into square brackets and significantly reduce user
readability.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US">Or
dynamic detection of single value in a list will destroy any
schema for parsing.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US">Visual
difference of rendered Mapping and BlockMapping is
significant in all three formats.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US">The
key here means a mapping key in YAML, object key in JSON and
element or attribute name in XML, so it defines a key in the
schemas.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US">BlockMapping
renders as multi-line mapping in YAML, as multi-line object
in JSON and as multi-line nested elements in XML.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US">While
Mapping renders as single-line flow in YAML, as single-line
object in JSON and as single element with attributes only
in XML.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US">Dynamic
detection by content will make the print very unstable and
unfriendly for user reading (for example one method will be
collapsed, while the other will expand).<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US">I
don’t see much space for reduction without losing visual
features.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt" lang="EN-US"><o:p></o:p></span></p>
<div>
<p class="MsoNormal" style="margin-left:36.0pt"><span style="font-size:11.0pt">On 25.07.2022 19:28, "Brian
Goetz" <a class="moz-txt-link-rfc2396E" href="mailto:brian.goetz@oracle.com"><brian.goetz@oracle.com></a> wrote:<o:p></o:p></span></p>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:0cm;margin-right:0cm;margin-bottom:12.0pt;margin-left:36.0pt"><span style="font-size:11.0pt"><br>
</span><span style="font-size:13.5pt;font-family:"Courier
New",serif">That's a good move. Can we push further?
We have types here for mapping a key to a:<br>
<br>
- list of simple values<br>
- list of arbitrary fragments <br>
- list of printables<br>
- list of list of printable (blocklist)<br>
<br>
<br>
</span><span style="font-size:11.0pt"><o:p></o:p></span></p>
<div>
<p class="MsoNormal" style="margin-left:36.0pt"><span style="font-size:11.0pt">On 7/25/2022 1:20 PM, Adam Sotona
wrote:<o:p></o:p></span></p>
</div>
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
<p class="MsoNormal" style="margin-left:36.0pt"><span style="font-size:11.0pt" lang="EN-US">Or maybe we could
let the printer to auto-quote as needed by the actual
value in the context of the actual format
</span><span style="font-size:11.0pt;font-family:"Apple
Color Emoji"" lang="EN-US">😊</span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:36.0pt"><span style="font-size:11.0pt" lang="EN-US">So yes, it can be
reduced:</span><o:p></o:p></p>
<pre style="margin-left:108.0pt;background:white"><span style="color:#336BDD">public</span><span style="color:black"> </span><span style="color:#336BDD">sealed</span><span style="color:black"> </span><span style="color:#336BDD">interface</span><span style="color:black"> <b>Printable</b> {</span><o:p></o:p></pre>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">public</span><span style="font-family:"Courier
New",serif;color:black"> String
<b>key</b>();</span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">public</span><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">sealed</span><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">interface</span><span style="font-family:"Courier
New",serif;color:black">
<b><i>Fragment</i></b> </span><span style="font-family:"Courier
New",serif;color:#336BDD">extends</span><span style="font-family:"Courier
New",serif;color:black"> Printable {}</span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">public</span><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">record</span><span style="font-family:"Courier
New",serif;color:black">
<i>Value</i>(String </span><span style="font-family:"Courier
New",serif;color:#CE54B8">key</span><span style="font-family:"Courier
New",serif;color:black">,
<s>String</s> ConstantDesc </span><span style="color:black">value</span><span style="font-family:"Courier
New",serif;color:black">)
</span><span style="color:black">implements</span><span style="font-family:"Courier
New",serif;color:black">
<i>Fragment</i> {}</span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><s><span style="color:black">public</span></s><s><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">record</span></s><s><span style="font-family:"Courier
New",serif;color:black">
<i>Quoted</i>(String </span></s><s><span style="font-family:"Courier
New",serif;color:#CE54B8">key</span></s><s><span style="font-family:"Courier
New",serif;color:black">, String
</span><span style="color:black">value</span></s><s><span style="font-family:"Courier
New",serif;color:black">)
</span><span style="color:black">implements</span></s><s><span style="font-family:"Courier
New",serif;color:black">
<i>Fragment</i> {}</span></s><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><s><span style="color:black">public</span></s><s><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">record</span></s><s><span style="font-family:"Courier
New",serif;color:black">
<i>Decimal</i>(String </span></s><s><span style="font-family:"Courier
New",serif;color:#CE54B8">key</span></s><s><span style="font-family:"Courier
New",serif;color:black">,
</span><span style="color:black">int</span></s><s><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">value</span></s><s><span style="font-family:"Courier
New",serif;color:black">)
</span><span style="color:black">implements</span></s><s><span style="font-family:"Courier
New",serif;color:black">
<i>Fragment</i> {}</span></s><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">public</span><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">record</span><span style="font-family:"Courier
New",serif;color:black">
<i>ValueList</i>(String </span><span style="font-family:"Courier
New",serif;color:#CE54B8">key</span><span style="font-family:"Courier
New",serif;color:black">, List<<s>String</s></span><span style="color:black">
</span><span style="font-family:"Courier New
\;color\:black";color:black">ConstantDesc>
</span><span style="color:black">values</span><span style="font-family:"Courier
New",serif;color:black">)
</span><span style="color:black">implements</span><span style="font-family:"Courier
New",serif;color:black">
<i>Fragment</i> {}</span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><s><span style="color:black">public</span></s><s><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">record</span></s><s><span style="font-family:"Courier
New",serif;color:black">
<i>QuotedList</i>(String </span></s><s><span style="font-family:"Courier
New",serif;color:#CE54B8">key</span></s><s><span style="font-family:"Courier
New",serif;color:black">, List<String>
</span><span style="color:black">values</span></s><s><span style="font-family:"Courier
New",serif;color:black">)
</span><span style="color:black">implements</span></s><s><span style="font-family:"Courier
New",serif;color:black">
<i>Fragment</i> {}</span></s><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">public</span><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">record</span><span style="font-family:"Courier
New",serif;color:black">
<i>Mapping</i>(String </span><span style="font-family:"Courier
New",serif;color:#CE54B8">key</span><span style="font-family:"Courier
New",serif;color:black">, List<<i>Fragment</i>>
</span><span style="color:black">fragments</span><span style="font-family:"Courier
New",serif;color:black">)
</span><span style="color:black">implements</span><span style="font-family:"Courier
New",serif;color:black"> Printable {}</span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">public</span><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">record</span><span style="font-family:"Courier
New",serif;color:black">
<i>BlockMapping</i>(String </span><span style="font-family:"Courier
New",serif;color:#CE54B8">key</span><span style="font-family:"Courier
New",serif;color:black">, List<Printable>
</span><span style="color:black">printables</span><span style="font-family:"Courier
New",serif;color:black">)
</span><span style="color:black">implements</span><span style="font-family:"Courier
New",serif;color:black"> Printable {}</span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">public</span><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">record</span><span style="font-family:"Courier
New",serif;color:black">
<i>BlockList</i>(String </span><span style="font-family:"Courier
New",serif;color:#CE54B8">key</span><span style="font-family:"Courier
New",serif;color:black">, List<<i>BlockMapping</i>>
</span><span style="color:black">blockMappings</span><span style="font-family:"Courier
New",serif;color:black">)
</span><span style="color:black">implements</span><span style="font-family:"Courier
New",serif;color:black"> Printable {}</span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black"> </span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:108.0pt;background:white"><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">public</span><span style="font-family:"Courier
New",serif;color:black">
</span><span style="color:black">record</span><span style="font-family:"Courier
New",serif;color:black">
<i>Comment</i>(String </span><span style="font-family:"Courier
New",serif;color:#CE54B8">key</span><span style="font-family:"Courier
New",serif;color:black">)
</span><span style="color:black">implements</span><span style="font-family:"Courier
New",serif;color:black"> Printable {}</span><o:p></o:p></p>
<p class="MsoNormal" style="margin-left:36.0pt"><span style="font-size:11.0pt" lang="EN-US"> </span><o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:0cm;margin-right:36.0pt;margin-bottom:5.0pt;margin-left:0cm"><o:p> </o:p></p>
</blockquote>
</div>
</blockquote>
<br>
</body>
</html>