Skip to content

Conversation

@toivoh
Copy link
Contributor

@toivoh toivoh commented Oct 24, 2012

Per discussion in #1430. Changes usage from e g

for (x,i) in enumerate(xs) ...

to

for (i,x) in enumerate(xs) ...

RFC: This change will break any code outside the julia repo that uses enumerate,
but will make it consistent with Python. Any objections to merging this now?

@nolta
Copy link
Member

nolta commented Oct 24, 2012

+1

@ViralBShah
Copy link
Member

Seems like a good idea to me to make it more consistent with python.

@nolta
Copy link
Member

nolta commented Oct 25, 2012

3a0e570 added some enumerate tests; could you also add a commit updating them?

@carlobaldassi
Copy link
Member

Actually no, the tests I added are for the Iterators module in extras/iterators.jl only (because I fixed a couple of bugs); I didn't add anything related to base/iterator.jl. There should probably be tests though.

BTW I also support this change.

@nolta
Copy link
Member

nolta commented Oct 25, 2012

Oops. Anyway, these are the tests i meant.

@nolta
Copy link
Member

nolta commented Oct 25, 2012

Yikes, i clearly need more coffee. The test changes are already part of the commit. Merged.

nolta added a commit that referenced this pull request Oct 25, 2012
RFC: Reverse order of outputs from enumerate, to emulate python. (fix #1430)
@nolta nolta merged commit 9acd270 into JuliaLang:master Oct 25, 2012
ViralBShah added a commit that referenced this pull request Jan 10, 2026
Stdlib: LinearAlgebra
URL: https://github.com/JuliaLang/LinearAlgebra.jl.git
Stdlib branch: master
Julia branch: master
Old commit: b599095
New commit: 0eaade8
Julia version: 1.14.0-DEV
LinearAlgebra version: 1.13.0 (Does not match)
Bump invoked by: @ViralBShah
Powered by:
[BumpStdlibs.jl](https://github.com/JuliaLang/BumpStdlibs.jl)

Diff:
JuliaLang/LinearAlgebra.jl@b599095...0eaade8

```
$ git log --oneline b599095..0eaade8
0eaade8 In-place multiplication of `Diagonal` with OffsetMatrix should throw (#1526)
417e914 Reduce method definitions for cat of `AbstractQ`s (#1500)
653ead1 Make sure inputs to the hemm etc tests are Hermitian  (#1522)
4004db9 fix typos (#1521)
eb007bb Fix GEMM dispatch for complex-real matmul (#1520)
f41bf65 Fix unitful 3-arg `*` (#1499)
24ac8e9 Bump actions/checkout from 5 to 6 (#1518)
2a6f3b7 make eigvals(::Diagonal) accept sortby (#1477)
189d49d Make `dot` consistently return `zero` on empty arrays (#1494)
1c0b673 Ensure inverse of Symmetric{<:,Diagonal} returns same type (#1439)
fa00b3c Fixed typo in matmul.jl (#1492)
21af0af Fix 3-arg `dot` for empty arrays (#1485)
6beb32c Make matmul work with zero-less eltypes (#1488)
```

Co-authored-by: ViralBShah <[email protected]>
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.

4 participants