Nested enumeration container

[guimarqu]

Structure to create a tree of duty using UInt instead of types. It'll allow us to have VarData and ConstrData bitstype.

Syntax to create a tree :

Coluna.@nestedenum VarConstrDuty,
    A ← VarConstrDuty,
        A1 ← A,
        A2 ← A,
        A3 ← A,
    B ← VarConstrDuty,
        B1 ← B,
        B2 ← B,
        B3 ← B,
            B3A ← B3,
            B3B ← B3,
            B3C ← B3,

Syntax to know if duty inherits from another :

A1 ← A # Return true
A1 ← B # Return false
!(A1 ← B) # Return true

@vitornesello your opinion about the syntax ?

[vitornesello]

I don't like too much the arrows. Can we change it to something easy to find in the keyboard? like <= or =>. Also to keep some sort of consistency with dicts

begin and end are missing

[guimarqu]

I don't like too much the arrows. Can we change it to something easy to find in the keyboard? like <= or =>. Also to keep some sort of consistency with dicts

begin and end are missing

Ok for <=. My first idea was to use <: but it's not possible to overload this operator.
It works without begin and end.

[vitornesello]

so begin and end are optional?

[guimarqu]

so begin and end are optional?

Yes because I use commas. Maybe it's better without commas and using a block expression ?

[vitornesello]

yeah, I prefer the block. It is also more common I think

[codecov-io]

Codecov Report

Merging #213 into master will increase coverage by 0.22%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #213      +/-   ##
==========================================
+ Coverage      85%   85.22%   +0.22%     
==========================================
  Files          43       44       +1     
  Lines        2620     2659      +39     
==========================================
+ Hits         2227     2266      +39     
  Misses        393      393

Impacted Files	Coverage Δ
src/Coluna.jl	100% <ø> (ø)	⬆️
src/containers/nestedenum.jl	100% <100%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7105b1f...a55c9cb. Read the comment docs.

[vitornesello]

Can you add a small comment to this function?

[vitornesello]

Why is it not an Int?

[vitornesello]

Would be nice to have a quick explanation saying that internally the macro builds a tree for which the nodes are indexed using prime numbers. And the interest of using prime numbers is to be able to recover all the parents of a given node.
Also, it would be nice to have a comment explaining what is what here : i, names, parent_pos, leaves, primes

     O
   /   \
  O     O
 / \   / \
O   O  O  O

[vitornesello]

why do you need leaves?