Commit 28974b4
authored
Do not suggest convert in Rosalind example (#13)
It was probably an API mistake to allow `convert`ing between mutable, heap-
allocated sequences. This is because `convert` is called implicit in Julia,
and converting sequences:
* Is slow compared to converting integers, causing unexpected allocations
* Creates a new, mutable struct with a separate object identity. This can lead
to nasty bugs when mutating the original sequence suddenly seems to have
no effect, because the observed sequence is actually a distinct copy.1 parent 427d649 commit 28974b4
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
58 | | - | |
| 58 | + | |
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
73 | | - | |
| 73 | + | |
74 | 74 | | |
75 | 75 | | |
76 | 76 | | |
| |||
0 commit comments