fix: issue in generating :kw for anonymous functions

Author: MilesCranmer

src/utils.jl

@@ -271,7 +271,12 @@ function longdef1(ex)
       kw_args = []
       for a in arg.args
         if !(a isa LineNumberNode)
-          if isexpr(a, :...) || isexpr(a, :(=)) || isexpr(a, :kw)
+          if isexpr(a, :...)
+            push!(kw_args, a)
+          elseif isexpr(a, :(=))
+            # Transform = to :kw for keyword arguments
+            push!(kw_args, Expr(:kw, a.args[1], a.args[2]))
+          elseif isexpr(a, :kw)
             push!(kw_args, a)
           else
             push!(pos_args, a)

test/split.jl

@@ -115,6 +115,14 @@ let
     # Both splatted positional and keyword arguments
     @test (@splitcombine (a::Int, args::Int...; kws...) -> a + sum(args) + sum(values(kws)))(1, 2, 3; b=4, c=5) == 15
     @test (@splitcombine (a, ::Int...; b, kws...) -> a + sum(values(kws)))(1, 2, 3; b=4, c=5) == 1 + 5
+
+    # Issue with longdef
+    ex = longdef(:((a::Int; b=2) -> a + b))
+    any_kw(ex) = ex isa Expr ? (any_kw(ex.head) || any(any_kw, ex.args)) : ex == :kw
+    @test any_kw(ex)
+    ## ^Ensure we get a :kw expression in the output AST
+    @test eval(ex) isa Function
+    ## Shouldn't have issues evaluating
 end
 
 @testset "combinestructdef, splitstructdef" begin