Skip to content

Commit

Permalink
pair as image index
Browse files Browse the repository at this point in the history
  • Loading branch information
hiiamboris authored Jun 7, 2021
1 parent 777d20d commit 3f59785
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions code-analysis/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -264,6 +264,7 @@ I understand that majority of case will be like `foreach [/i x] series` so all t
(4) a bit trickier: `i: 1 + offset? series p`\
(5) even more tricky: `i: 1 + to 1 (offset? series p) / width`\
(4) and (5) I find hard to reason about and explain, and I think for these cases an explicit computation should be used instead of an index you get "for free"\
(3) and (4) in case of iterating over image can be pairs, the rest cannot\
So, my choice would be (2) for it's simplicity of meaning and somewhat higher importance.

**Series** index considerations:
Expand Down

0 comments on commit 3f59785

Please sign in to comment.