Closed 23Skidoo closed 8 years ago
The former is useful for testing, the latter is because we need this patch: https://github.com/informatikr/aeson-pretty/commit/d7a6c6e49d8c9e7a0de6a73f5fe2135d87568e8a.
You might want to check if it's not too lazy wrt. modifyIORef (if you didn't do this already :)
You're right, there's no reason not to use modifyIORef'. Fixed.
modifyIORef'
LGTM, please merge.
The former is useful for testing, the latter is because we need this patch: https://github.com/informatikr/aeson-pretty/commit/d7a6c6e49d8c9e7a0de6a73f5fe2135d87568e8a.