Glow-Lang / glow

The Glow language for Blockchain Decentralized Applications.
https://glow-lang.org
Apache License 2.0
58 stars 8 forks source link

Clearer syntax (potential typo fix) in future/crowdfunding.glow contract. - [opened] #335

Open kwannoel opened 3 years ago

kwannoel commented 3 years ago

In GitLab by @flexabyte on Mar 30, 2021, 19:35

Merges master -> master

In an attempt to consolidate some quality examples for potential future contracts I went through the crowdfunding contract and discovered some syntactical inconsistencies, specifically two curly brackets that weren't closed, as well as the fact in the first choice statement you have:

choice {
  | ForAllParticipant (NewPledger) {

with curly bracket and for the last one you have

| ForAllParticipant (Pledger) 

My changes remove the curly brackets, though it could be that those brackets are required. (Not sure how the parser determines this, my intuition tells me we will want them, but I leave that to the MuKn team to establish.

kwannoel commented 3 years ago

In GitLab by @flexabyte on Mar 30, 2021, 21:32

added 1 commit

Compare with previous version

kwannoel commented 3 years ago

In GitLab by @flexabyte on Mar 30, 2021, 21:45

added 1 commit

Compare with previous version

kwannoel commented 3 years ago

Original PR: https://gitlab.com/mukn/glow/-/merge_requests/125