Merge pull request #17307 from stevengj/broadcast-splat

fix f.(args...) with splatting

Author: JeffBezanson

src/julia-syntax.scm

@@ -1562,13 +1562,13 @@
 
    '|.|
    (lambda (e) ; e = (|.| f x)
-     (let ((f (expand-forms (cadr e)))
-           (x (expand-forms (caddr e))))
+     (let ((f (cadr e))
+           (x (caddr e)))
        (if (or (eq? (car x) 'quote) (eq? (car x) 'inert) (eq? (car x) '$))
-         `(call (core getfield) ,f ,x)
+         `(call (core getfield) ,(expand-forms f) ,(expand-forms x))
          ; otherwise, came from f.(args...) --> broadcast(f, args...),
-         ; where x = (call (top tuple) args...) at this point:
-         `(call broadcast ,f ,@(cddr x)))))
+         ; where x = (tuple args...) at this point:
+         (expand-forms `(call broadcast ,f ,@(cdr x))))))
 
    '|<:| syntactic-op-to-call
    '|>:| syntactic-op-to-call

test/broadcast.jl

@@ -213,3 +213,8 @@ end
 @test Base.promote_op(+, Bool) === Int
 @test isa(broadcast(+, true), Array{Int,0})
 @test Base.promote_op(Float64, Bool) === Float64
+
+# issue #17304
+let foo = [[1,2,3],[4,5,6],[7,8,9]]
+    @test max.(foo...) == broadcast(max, foo...) == [7,8,9]
+end