LayoutBuilder template using either ForthMachine32 or ForthMachine64 and a tutorial

[ianna]

No description provided.

[agoose77]

Absolutely, I'll take a look tomorrow!

…

On Tue, 24 Aug 2021, 17:01 Ianna Osborne, ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In docs-src/how-to-create-layoutbuilder.md <#1063 (comment)> : > +builder.tag(1) +builder.boolean(True) +builder.tag(0) +builder.float64(4.4) +builder.tag(1) +builder.boolean(False) +builder.tag(1) +builder.boolean(True) +builder.tag(0) +builder.float64(-2.2) +``` + +Snapshot +-------- + +To turn a #ak::LauoutBuilder into a layout, call `snapshot`. This is an inexpensive operation (may be done multiple times; the builder is unaffacted). @agoose77 <https://github.com/agoose77> - Can I take you up on your kind offer of code review? Thanks! — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1063 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJQZHIWEM6EMCB4O5KJWIDT6O66VANCNFSM5CLWECQA> .

[ianna]

@jpivarski - I think, I'm done with this PR. It introduces LayoutBuilder32 and LayoutBuilder64, plus the error handling is improved, I think.

[jpivarski]

The documentation looks good to me, and I'm glad that writing it inspired some tweaks to the code. The references to class names could be links, as I've said in a comment above, and that would be an improvement.

Thanks @agoose77 for the review! You brought up some good points.

[jpivarski]

Oh, and I'm holding off on merging until you decide what you want to do about @agoose77's comments, @ianna. Although when you've addressed everything, you can do the merge. That's what my sign-off above means.