Curly braces in coverpoints must close with the first character of the coverpoint name

[antmarzam]

This suggestion is followed in Opentitan

Good Example:

Covergroup cg;
  Bit x;
  Cp_name: coverpoint x {
    Bins zero = {0};
    Bins one  = {1};
  }
endgroup

Bad Examples:

Covergroup cg;
  Bit x;
  Cp_name: coverpoint x {
                         Bins zero = {0};
                         Bins one  = {1};
                        }
Endgroup

Covergroup cg;
  Bit x;
  Cp_name: coverpoint x {Bins zero = {0};
                         Bins one  = {1};
                        }
Endgroup

Covergroup cg;
  Bit x;
  Cp_name: coverpoint x {Bins zero = {0};
                         Bins one  = {1};}
Endgroup

[marnovandermaas]

What about?

Covergroup cg;
  Bit x;
  Cp_name: coverpoint x
  {
    Bins zero = {0};
    Bins one  = {1};
  }
endgroup

[antmarzam]

Is not what's followed in OpenTitan I think. But happy to include if others agree!

[rswarbrick]

I don't think I really mind. Do you have examples in mind that follow that style? I'm keen on the guidance: the question is probably just whether to add that as an option for how one might satisfy it.