eclipse-vertx / eclipse-vertx/vertx-codegen

Should throw error on VertxGen declared inside VertxGen/DataObject

Aperta
#319 1 commento 1 reazione 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

bug
Lingua principale
Java
Stelle
111
Fork
88
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Currently, I've noticed a few generation errors regarding how classes and interfaces are annotated.

For example, if a @VertxGen interface has a enum declared inside, this will be valid java code and will compile and no warning is given by codegen. However, you will not be able to process such element.

@VertxGen
interface A {
  @VertxGen
  enum E { }
}

In the example above E will never be processed and no warning or error is raised. I think we should raise an error in this situation.

The same applies for @DataObject. During the processing we should guarantee that this isn't allowed. This pattern is a problem to es4x but may not be for other generators, so we may agree in raising a warning instead.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Non vengono indicati file o test. Inizia individuando l'elaborazione codegen per le dichiarazioni @VertxGen e @DataObject annidate, quindi riproduci l'esempio di enum annidato dell'issue. Il lavoro è completato quando il processore segnala in modo coerente un errore o un avviso per le dichiarazioni che non possono essere elaborate e il comportamento scelto è confermato per entrambi i tipi di annotazione.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
java
Ambito
compilers
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.