Replaces the IO argument with a Union{IOBuffer,IOContext} which
should be the only types that are actually reachable since the code
paths pass through sprint, which only uses either of those.
It's still not a perfect fix due to the type piracy, as noted in the
comments, but avoids the method overwrite warning at least.
pfitzseb
commented
3 years ago
Replaces the
IOargument with aUnion{IOBuffer,IOContext}which should be the only types that are actually reachable since the code paths pass throughsprint, which only uses either of those.It's still not a perfect fix due to the type piracy, as noted in the comments, but avoids the method overwrite warning at least.