duplicate named enum member in different enum type cause error

[jidibinlin]

enum FSBossStrongholdEvent {
  ready = 0;
  start = 1;
  finish = 2;
}

(pi:define-enum fs-boss-stronghold-event
    (:name "FSBossStrongholdEvent")
  (ready :index 0)
  (start :index 1)
  (finish :index 2))

 enum SAStage {
  READY = 0; 
  SIGN_UP = 1; 
  GROUP = 2; 
  TOP32 = 3; 
  FINAL = 4; 
  FINISH = 5; 
  CANCEL = 6; 
}

(pi:define-enum sa-stage
    (:name "SAStage")
  (ready :index 0)
  (sign-up :index 1)
  (group :index 2)
  (top32 :index 3)
  (final :index 4)
  (finish :index 5)
  (cancel :index 6))

fs-boss-stronghold-event`s finish conflict with sa-stags`s finish

and got this error

The constant +FINISH+ is being redefined (from 5 to 2)
   [Condition of type SB-EXT:DEFCONSTANT-UNEQL]

[Slids]

This is WAI, if you macroexpand the define enum youll see

(defconstant +finish+ 5)

and

(defconstant +finish+ 2)

easiest fix is to
a) Give them the same value
b) but them inside a message

For b:
define message SAS {
enum SAStage {
READY = 0;
SIGN_UP = 1;
GROUP = 2;
TOP32 = 3;
FINAL = 4;
FINISH = 5;
CANCEL = 6;
}
}
so you should get
+sas.finish+

See:

cl-protobufs/define-proto.lisp

Line 281 in 628d713

(defun make-enum-constant-forms (type enum-values)

[Slids]

Also in the readme:
Each numeric enum value is also bound to a constant by the same name but with "+" on each side:

[jidibinlin]

Also in the readme: Each numeric enum value is also bound to a constant by the same name but with "+" on each side:

I think prefix enum constants with the enum name is a batter choice.

[cgay]

I think you're right. Precluding having two enum values with the same name at top-level isn't good.