ClojureScript

Add invoke* helper macro

Details

  • Type: Enhancement Enhancement
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 1.9.562
  • Fix Version/s: 1.9.671
  • Component/s: None
  • Labels:
  • Patch:
    Code
  • Approval:
    Screened

Description

This is a simple refactor around {IFn} protocol around core.cljc and core.cljs. We would like to hide the details of the invocation naming convention to avoid simple errors as well as to support changes more simply.

Activity

Hide
David Nolen added a comment -

The scope of this ticket needs to be narrowed to make it simpler for me to review. For the time being the only thing I would like to see is `invoke*` which hides the naming convention for direct invokes. No other higher level macro helpers should be provided in the resolution of this sissue.

Show
David Nolen added a comment - The scope of this ticket needs to be narrowed to make it simpler for me to review. For the time being the only thing I would like to see is `invoke*` which hides the naming convention for direct invokes. No other higher level macro helpers should be provided in the resolution of this sissue.
Hide
A. R added a comment -

Patch updated. Much fewer changes to keep it simple for now.

Show
A. R added a comment - Patch updated. Much fewer changes to keep it simple for now.
Hide
David Nolen added a comment -

Looking better but lets have one helper for constructing the name, should just take number or :variadic.

Show
David Nolen added a comment - Looking better but lets have one helper for constructing the name, should just take number or :variadic.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated: