Slides/add some idioms #416
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
adacore-bot
force-pushed
the
slides/add-some-idioms
branch
from
8a300e0 to
4c46d7c
Compare
adacore-bot
force-pushed
the
slides/add-some-idioms
branch
from
4c46d7c to
001668c
Compare
Co-authored-by: Dana Binkley <[email protected]>
Overview is very large, and it was getting into lots of details (4 slides) on the type system *before* we get to actually talk about it. As a result, we had to repeat some of that info where it made more sense: in the type system slides. This commit splits that up, by moving the information from overview to declarations and basic type chapters.
We don't even talk about some of these in the slides (e.g. C variadics support) so there's no point in introducing them on this already quite long overview
Use the same "spec /= impl" as for the subprogram slide above. Remove the slide on views, this is far too advanced at this point and the slide just above covers encapsulation. Move the ADT definition to basic type slide where it makes tons of sense to be (basically we would re-intro them at this point). Maybe the two resulting slides could end up being merged? OOP was referencing ADT, that's too advanced at this point too (people haven't written a single line of code yet, it's an overview!)
Tasking first slide was a bit abrupt, and in overview there was a specific slide (of the 2 that were there) that was going into the details too much, so this moves and merges with the tasking introduction where it fits nicely as a warm-up and intro to terminology.
adacore-bot
force-pushed
the
slides/add-some-idioms
branch
from
201f22f to
5c2405e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.