[m-rev.] for review: extend mutual tail rec optimization to functions

Zoltan Somogyi zoltan.somogyi at runbox.com
Wed Sep 13 18:06:03 AEST 2017



On Wed, 13 Sep 2017 00:09:25 -0400 (EDT), Julien Fischer <jfischer at opturion.com> wrote:
> > -                % The tscc variables for the output arguments will always
> > -                % be defined in the argument list of the MLDS function.
> > +                % The tscc variables for the byref output arguments will
> 
> Why has tscc suddenly become lowercase here?  It was uppercase above.

The sentence is referring not the the TSCC as a whole, but to the
lvn_tscc_{input,output}* MLDS variables. I have changed the comment
to spell them out.

I also followed your other suggestions.

> The diff looks fine otherwise.

Thanks for the review.

Zoltan.



More information about the reviews mailing list