gadget-framework / gadget3

TMB-based gadget implemtation
GNU General Public License v2.0
8 stars 6 forks source link

Fix splitting g3a_spawn() offspring over multiple stocks #146

Closed lentinj closed 2 months ago

lentinj commented 3 months ago

@bthe @vbartolino this patch should get rid of the warning you're seeing.

My explanation wasn't right. The warning was actually coming from within g3a_spawn(), as when we have multiple output_stocks we redefine sum_all_outputs_f. The following should silence it, by using a different name.

Ideally this needs some tidying up before merging, most importantly a test for multiple output_stocks, since it's obvious it can't be tested, but you can if it helps.

For the problem at hand this is irrelevant noise. Silencing the warning doesn't change the code generated.

Fixes #147