Spec comments
Closed
Spec comments
Goals and Outcomes
- Bit shifts and bitwise operators are no longer in the lab
Image Rotation
- This is a weird way of saying that they should reflect along the diagonal. I worked this out by looking at the tests, but this is a reflection, not a rotation (just semantics).
ARGB Values as Bits
- "Because computers ultimately" --> "Because computers are ultimately"
Putting it Together
- Mention that each char should be stored with least significant bit coming first in the standard iteration of a 2D array.
Time: ~30 min, but I spent a while forgetting that fixLowestBitOfR returned a new pixel instead of modifying it.
err...it's definitely a rotation; i'm not sure what you're talking about.
all the others are totally valid. @ddimino can you fix those?
Re: rotation - to be more clear, I'll attach what I mean.
This is the original image that is to be transformed.
This is the desired output image.
If we were to rotate the original image 90 degrees counterclockwise, it would appear like this.
Actually, I was incorrect to call it a reflection. The transformation we want - swapping the row and column coordinates - is called a transpose.
yeah, that's a bug then; it's supposed to be a rotation. @ddimino can we fix this please?
actually, on second thought--transpose is easier-- @ddimino let's fix all the names and spec to say transpose