[L2Ork-dev] How to debug Windows and OSX

Jonathan Wilkes jon.w.wilkes at gmail.com
Tue Apr 7 18:05:17 EDT 2020


On Tue, Apr 7, 2020 at 4:57 PM Ivica Ico Bukvic <ico at vt.edu> wrote:
>
> On 4/7/2020 4:09 PM, Jonathan Wilkes wrote:
>
> > On Tue, Apr 7, 2020 at 1:45 PM Ivica Ico Bukvic <ico at vt.edu> wrote:
> >> I am not so concerned with A_SEMI since the objects that preset_node can
> >> connect to AFAIR do not support that character.
> > The thing is you've got two cases of "argv++" in the middle of that
> > enormous function body.
> >
> > It's difficult to tell at a glance whether that would jump off the end
> > of the array in some cases.
>
> This is necessary due to logic of how it parses the linear file
> containing both node and preset info (with potentially multiple presets
> per node).

It's not necessary. The function is too large and does too many different tasks
that make it difficult to read and reason about.

Don't let your code spiral so out of control that it swerves into the
right side of the screen
and causes accidents. :)

-Jonathan


More information about the L2Ork-dev mailing list