define IOContext(io, pairs...) and remove IOContext(io; kw...) #23271
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
IOContext(io, :k=>v)
),but if another pair has to be pushed, one had to either add an
IOContext call (
IOContext(IOContext(io, :k=>v), :x=>y)
) or toswitch to kwargs syntax (
IOContext(io, k=v, x=y)
), noneof which is satisfactory.
method using keyword arguments is obsoleted (and hence deleted).
IOContext(io, key, value)
is deleted infavor of
IOContext(io, key=>value)
.I originally only wanted to allow
IOContext(io, pairs...)
, but in the process saw opportunities for simplification of the API, asIOContext(io, :key, value)
andIOContext(io, key=value)
were almost not used. I have no problem to restore them if they are deemed necessary.