tlaplus / Examples

A collection of TLA⁺ specifications of varying complexities
Other
1.29k stars 200 forks source link

Add small improvements to the Disruptor spec. #151

Closed nicholassm closed 1 month ago

nicholassm commented 1 month ago
nicholassm commented 1 month ago

Alright, added the same improvements to the Multi Producer Multi Consumer specification. Thanks.

nicholassm commented 1 month ago

I'm considering moving the "spec" section (Init, Next and Liveliness) to the top (i.e. below constants and variables) to make it easier to understand the models for users. Kind of like an article where you read the highlights in the subheading.

What is generally recommended?

lemmy commented 1 month ago

The definition of a formula has to come before its use. In other words, the parser won't accept it.

nicholassm commented 1 month ago

Makes sense.

Is there anything else you can see that I can do to improve this PR? It's mostly related to (excellent) suggestions from @muenchnerkindl.

Kind regards, Nicholas

nicholassm commented 1 month ago

Hi guys,

Is there anything else I should do to increase the quality further? Otherwise, I think it's ready for merging. :-) Many thanks in advance.

Kind regards Nicholas