Skip to content

Commit

Permalink
Fix CI part 2
Browse files Browse the repository at this point in the history
This time I hopefully used the correct invocation in local testing
  • Loading branch information
Seelengrab committed Dec 6, 2024
1 parent 37336f2 commit b8978ea
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/RequiredInterfaces.jl
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ macro required(T::Symbol, expr::Expr)
msg = error_msg(funcpart, sig)
escFunc = esc(funcpart)
e.args[1] = esc(e.args[1])
push!(e.args[2].args, :(throw(NotImplementedError(string($escT), $msg))))
e.args[2] = :(throw(NotImplementedError(string($escT), $msg)))
res = ntuple(length(sig)) do i
s = sig[i]
if s isa Symbol
Expand Down Expand Up @@ -375,9 +375,9 @@ function check_interface_implemented(interface::Type, implementor::Type)
ct, rettype = only(mt)
rettype !== Union{} && continue # if it infers, we can't throw our error
isempty(ct.code) && continue # empty function
offset = ct.code[1] isa Expr && ct.code[1].head === :code_coverage_effect
length(ct.code) < 2 && continue # function with only one expr - not our code
offset += ct.code[2] isa Expr && ct.code[2].head === :code_coverage_effect
offset::Int = ct.code[2] isa Expr && ct.code[2].head === :code_coverage_effect
length(ct.code) < 3 && continue # function with only one expr - not our code
offset += ct.code[3] isa Expr && ct.code[3].head === :code_coverage_effect
length(ct.code) < (offset + 2) && continue # function with 2 expr not from us
errorExpr = ct.code[offset + 2]
errorExpr isa Expr || continue # not our Error? could be a change in IR
Expand Down

0 comments on commit b8978ea

Please sign in to comment.