Skip to content

Commit f84996f

Browse files
Add comments suggested by @cstjean
Co-authored-by: Cédric St-Jean <[email protected]>
1 parent 8da820c commit f84996f

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/utils.jl

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -434,6 +434,8 @@ See also: [`combinearg`](@ref)
434434
"""
435435
function splitarg(arg_expr)
436436
if @capture(arg_expr, arg_expr2_ = default_)
437+
# This assert will only be triggered if a `nothing` literal was somehow spliced into the Expr.
438+
# A regular `nothing` default value is a `Symbol` when it gets here. See #178
437439
@assert default !== nothing "splitarg cannot handle `nothing` as a default. Use a quoted `nothing` if possible. (MacroTools#35)"
438440
else
439441
arg_expr2 = arg_expr

0 commit comments

Comments
 (0)