Re: [PATCH] fixup! docs: spec: add variables and constants specification

["Carlos Maniero" <carlos@maniero.me>]

> Thanks a lot for your contribution, I very impressed you managed to send
> a really nice PATCH.
>
> Sorry for that, but there is another Patchset version which was applied
> to the main branch.  I think Carlos forgot to mention this Patchset was
> SUPERSEDED[v2] by version 2.
>
> v2: http://lists.johnnyrichard.com/olang/5fzsolce5h42aa6udppiwezgbzeqerkde3xvyilidnkcjaho2j@ygxd7ugzck4m/T/#m0f5cc6f2d49835ec83d4fd5c24d97d2597cb5363

Sorry Ricardo. My bad! I forgot to update the mailing list saying that
this patch was superseded.

> > ---
> > Discussions:
> > 
> > - Are you planning to hoist declarations, or are declarations required
> >   to be placed before their use?
>
> We are planning to have declarations being placed before their use.

Just an observation here, this is only valid for external declarations
and just inside a function scope.

valid:
  
  1| fn main(): u8 {
  2|   return exit_success
  3| }
  4| 
  4| const exit_success: u8 = 0

invalid:

  1| const a = b + 1
  2| const b = 0

also invalid:
  
  1| fn main(): u8 {
  2|   exit_success = 0
  3|
  4|   var exit_success
  4|   return exit_success
  5| }

> Could you please prepare a new patch over the _main_ branch with this
> change (if Carlos agree of course)?

LGTM! And thank you so much for your contributions Ricardo!