Skip to content

remove stale broadcasted_indices#1012

Merged
thchr merged 1 commit intoJuliaArrays:masterfrom
thchr:jet
Mar 14, 2022
Merged

remove stale broadcasted_indices#1012
thchr merged 1 commit intoJuliaArrays:masterfrom
thchr:jet

Conversation

@thchr
Copy link
Copy Markdown
Collaborator

@thchr thchr commented Mar 10, 2022

This is a minor nit found while playing around with JET.jl's report_package (the other highlights from that appeared to be false positives as far as I could see, so that's nice).

Justification is in the commit message.

@thchr
Copy link
Copy Markdown
Collaborator Author

thchr commented Mar 10, 2022

CI is having a hard time with some internal out-of-memory errors. Shouldn't be related to this though (but somewhat annoying; same sporadic failures for CI restarted 3 times).

- referenced non-extant `indices`
- `broadcasted_indices` was deprecated to `broadcasted_axes` in JuliaLang/julia#25377
- `broadcasted_axes` was then replaced by plain `axes` JuliaLang/julia@a2feccf
- StaticArrays already has `axes` methods
@thchr thchr merged commit e35134c into JuliaArrays:master Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants