eclipse-vertx / eclipse-vertx/vertx-codegen
GenIgnore with more fine-grained language control through "features"
- Vorherrschende Sprache
- Java
- Sterne
- 111
- Forks
- 89
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Having used vertx with the [Ceylon programming language](https://ceylon-lang.org/) for a while, I have noticed that `@GenIgnore` [source](https://github.com/vert-x3/vertx-codegen/blob/master/src/main/java/io/vertx/codegen/annotations/GenIgnore.java) has blocked access to interesting features for reasons that don't seem to apply to the Ceylon language.
For example methods returning `Map` or `X509Certificate` are out of reach while I believe they could be used perfectly well in Ceylon. And #40 lists `byte[]` as another case.
At the most coarse level one could simply choose to ignore `@GenIgnore` when generating bindings for specific languages. Alternatively we could perhaps add a (defaulted) parameter to the annotation indicating a feature required from the target language for the declaration to be included. This would give more fine-grained control over which constructs are exposed in which languages. For example the config file for the Ceylon language could indicate that it supports "MAP" and "CERTIFICATE" features (just to give the idea). Perhaps a better-named annotation could be introduced for this and deprecate the original one.
Beitragsleitfaden
Rechercherichtung
Beginnen Sie mit der Quelle der GenIgnore-Annotation in src/main/java/io/vertx/codegen/annotations/GenIgnore.java und prüfen Sie Issue #40 zum verwandten byte[]-Fall. Der Vorschlag stellt mehrere mögliche Designs vor; klären Sie daher zunächst, welchen Ansatz die Maintainer möchten; abgeschlossen wäre dies mit einem vereinbarten, implementierten Mechanismus zur sprachspezifischen Feature-Steuerung.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- devtools
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100