Timeline for pset4 resize.c - general struggles
Current License: CC BY-SA 3.0
11 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Sep 19, 2015 at 19:51 | vote | accept | Andrew Smith | ||
| Sep 19, 2015 at 19:50 | vote | accept | Andrew Smith | ||
| Sep 19, 2015 at 19:51 | |||||
| Sep 19, 2015 at 19:50 | comment | added | Andrew Smith | int resizedPadding = (4 - (bytesWide % 4)) % 4; ---- BLOODY MODULO!! | |
| Sep 19, 2015 at 19:45 | comment | added | ChrisG | Yes your resize_pading is wrong. Check your scans padding and adjust it accordingly. | |
| Sep 19, 2015 at 19:31 | comment | added | Andrew Smith | Padding arithmetic is wrong. Check50 fails when no padding is required but passes when there is padding. I need an extra 4 or modulo somewhere. | |
| Sep 19, 2015 at 18:47 | comment | added | Andrew Smith | gist.github.com/AndrewSmithWasTaken/3c4c3a44897cb1e75a5b | |
| Sep 19, 2015 at 18:37 | comment | added | ChrisG | Could you paste the rest of the code too so I can execute exactly what you have? | |
| Sep 19, 2015 at 18:12 | comment | added | Andrew Smith | Definitely getting there, op updated. Visually everything looks good but check50 is failing thus: :( resizes 1x1-pixel BMP to 4x4 correctly when n is 4 :( resizes 2x2-pixel BMP to 4x4 correctly when n is 2 | |
| Sep 19, 2015 at 17:38 | comment | added | ChrisG | Yeap cool! You fixed the first one. Now go for the second one too! | |
| Sep 19, 2015 at 17:27 | comment | added | Andrew Smith | Hey, thanks for the response. I have updated my op with the code I've come up with while I was away, I think I've addressed one of the issues you mention. I'll take a look at your suggestions now, thanks! | |
| Sep 19, 2015 at 17:22 | history | answered | ChrisG | CC BY-SA 3.0 |