<div dir="auto"><div dir="auto"><div data-smartmail="gmail_signature" dir="auto"><br></div></div><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Tue, Apr 7, 2020, 18:05 Jonathan Wilkes <<a href="mailto:jon.w.wilkes@gmail.com">jon.w.wilkes@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Apr 7, 2020 at 4:57 PM Ivica Ico Bukvic <<a href="mailto:ico@vt.edu" target="_blank" rel="noreferrer">ico@vt.edu</a>> wrote:<br>
><br>
> On 4/7/2020 4:09 PM, Jonathan Wilkes wrote:<br>
><br>
> > On Tue, Apr 7, 2020 at 1:45 PM Ivica Ico Bukvic <<a href="mailto:ico@vt.edu" target="_blank" rel="noreferrer">ico@vt.edu</a>> wrote:<br>
> >> I am not so concerned with A_SEMI since the objects that preset_node can<br>
> >> connect to AFAIR do not support that character.<br>
> > The thing is you've got two cases of "argv++" in the middle of that<br>
> > enormous function body.<br>
> ><br>
> > It's difficult to tell at a glance whether that would jump off the end<br>
> > of the array in some cases.<br>
><br>
> This is necessary due to logic of how it parses the linear file<br>
> containing both node and preset info (with potentially multiple presets<br>
> per node).<br>
<br>
It's not necessary. The function is too large and does too many different tasks<br>
that make it difficult to read and reason about.<br>
<br>
Don't let your code spiral so out of control that it swerves into the<br>
right side of the screen<br>
and causes accidents. :)<br></blockquote></div><div dir="auto"><br></div><div dir="auto"><span style="font-family:sans-serif">I love how you succinctly described most of the core Pd codebase :-)</span><div dir="auto" style="font-family:sans-serif"><br></div><div dir="auto" style="font-family:sans-serif">Joking aside, in this case I prefer the monolithic nature of the function because it contains all possible bugs inside a single flow and I would argue it is as a result easier to debug. That said, if anyone wishes to refactor this, I am fine with that as long as it maintains the same or better level of bugginess :-)<br><br><div data-smartmail="gmail_signature" dir="auto">Best,<br><br>Ico</div></div></div><div dir="auto"><br></div><div class="gmail_quote" dir="auto"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
-Jonathan<br>
</blockquote></div></div>