atlas-engineer / nclasses

A `define-class` macro for less boilerplate
Other
7 stars 2 forks source link

defclass-star(define-generic): Add :export as short export keyword. #25

Closed aartaka closed 12 months ago

aartaka commented 12 months ago

This adds a new keyword to define-generic. :export is an alias for :export-generic-name-p and provides the same functionality in a shorter form. It's also consistent with define-class slot :export option. So we can have our cake and eat it:

That'll also save us some typing in Nyxt, as part of the move to nclasses that I'm working on.

aadcg commented 12 months ago

Do we have any other alias in the library? What justifies its addition?

aartaka commented 12 months ago

Do we have any other alias in the library? What justifies its addition?

We have alias macros for convenience:

So yes, providing aliases for user's convenience is consistent with existing practices. I mean, nclasses is a convenience-first library, always was.

aadcg commented 12 months ago

Those aliases are of a different nature, as those symbols are exported from the library.

aartaka commented 12 months ago

Those aliases are of a different nature, as those symbols are exported from the library.

Fair, closing then.