<div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Michael Van Canneyt <<a href="mailto:michael@freepascal.org">michael@freepascal.org</a>> schrieb am Mo., 7. Okt. 2019, 17:21:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On Mon, 7 Oct 2019, Ben Grasset wrote:<br>
<br>
> On Sat, Oct 5, 2019 at 1:37 PM Ryan Joseph <<a href="mailto:genericptr@gmail.com" target="_blank" rel="noreferrer">genericptr@gmail.com</a>> wrote:<br>
><br>
>> I can’t remember what Bens patch did using compiler directives but I like<br>
>> the Java idea of using the column of the opening string character as the<br>
>> reference point for indentation. Seems easier.<br>
>><br>
>> Did the FPC team even approve this idea yet? It’s such a simple patch but<br>
>> I fear it’s going to be waiting in limbo for up to 12 months.<br>
>><br>
><br>
> My patch does support exactly that, if you use {$MultiLineStringTrimLeft<br>
> Auto}.<br>
><br>
> As far as "approval", if I recall correctly Sven said he intended to review<br>
> it eventually.<br>
<br>
I remember the same.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">The code changes themselves are okay and that was all I promised to review. The problem is I personally don't care about the feature, so other core devs should give their okay for the feature itself as well. If I get that okay I'll integrate it, no problem... </div><div dir="auto"><br></div><div dir="auto">Regards, </div><div dir="auto">Sven </div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>