
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

</head>

<body>

<style type="text/css" style="display:none;"><!-- P {margin-top:0;margin-bottom:0;} --></style>

<div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;font-family:Calibri,Helvetica,sans-serif;" dir="ltr">

<p>Hi Marcel,</p>

<p><br>

</p>

<p>this is indeed not valid Smalltalk code. In the test, I expected the last keyword (#not) to be parsed as #<span>excessCode. At the moment, the Shout treats this syntax as valid syntax which has confused me already too often.</span></p>

<p><span><br>

</span></p>

<p><span>Given the confusion, I should probably rename this test or add a proper comment.</span></p>

<p><span><br>

</span></p>

<p><span>Best,</span></p>

<p><span>Christoph</span></p>

</div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>Von:</b> Squeak-dev <squeak-dev-bounces@lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel<br>

<b>Gesendet:</b> Donnerstag, 25. November 2021 15:04:57<br>

<b>An:</b> squeak-dev<br>

<b>Betreff:</b> Re: [squeak-dev] The Inbox: ShoutTests-ct.34.mcz</font>

<div> </div>

</div>

<div>

<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000;text-align: left" dir="ltr">

Hi Christoph --

<div class="mb_sig"></div>

<div><br>

</div>

<div>This also addresses ShoutCore-ct.85 sitting in the inbox.</div>

<div><br>

</div>

<div>The code snippet you want Shout to style is</div>

<div><br>

</div>

<div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">x 1 positive; negative not</span><br>

</div>

<div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br>

</span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">I don't see how this is valid Smalltalk code. Even like this</span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br>

</span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">x := 1 positive; negative not</span><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br>

</span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br>

</span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">the trailing "not" cannot be compiled. Adding parentheses, it becomes valid:</span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br>

</span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif"><span style="font-size: 13px">x := (1 positive; negative) not.</span></span><br>

</div>

<div><span style="font-family: Arial, Helvetica, sans-serif"><span style="font-size: 13px"><br>

</span></span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif"><span style="font-size: 13px">...so can you elaborate on what you wanted to test here? I would expect that Shout's results for invalid Smalltalk code is ... undefined? </span></span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif"><span style="font-size: 13px"><br>

</span></span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif"><span style="font-size: 13px">Best,</span></span></div>

<div><span style="font-family: Arial, Helvetica, sans-serif"><span style="font-size: 13px">Marcel</span></span></div>

<blockquote class="history_container" type="cite" style="border-left-style: solid;border-width: 1px;margin-top: 20px;margin-left: 0px;padding-left: 10px;min-width: 500px">

<p style="color: #AAAAAA; margin-top: 10px;">Am 07.04.2021 17:20:24 schrieb commits@source.squeak.org <commits@source.squeak.org>:</p>

<div style="font-family:Arial,Helvetica,sans-serif">A new version of ShoutTests was added to project The Inbox:<br>

http://source.squeak.org/inbox/ShoutTests-ct.34.mcz<br>

<br>

==================== Summary ====================<br>

<br>

Name: ShoutTests-ct.34<br>

Author: ct<br>

Time: 7 April 2021, 5:20:15.847598 pm<br>

UUID: a7f39f26-a6fb-f44e-aabb-931714b56ed8<br>

Ancestors: ShoutTests-mt.33<br>

<br>

Document a present bug that occurs when styling a method that contains multiple unary selectors after a cascade.<br>

<br>

=============== Diff against ShoutTests-mt.33 ===============<br>

<br>

Item was added:<br>

+ ----- Method: SHParserST80Test>>testChainAfterCascade (in category 'tests') -----<br>

+ testChainAfterCascade<br>

+ <br>

+ | types s |<br>

+ s := 'x 1 positive; negative not'.<br>

+ <br>

+ types := self tokenTypesIn: s.<br>

+ <br>

+ self<br>

+ assert: #(patternUnary number unary cascadeSeparator unary excessCode)<br>

+ equals: types.!<br>

<br>

<br>

</div>

</blockquote>

</div>

</div>

</body>

</html>