Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Switch to `IntX` encodings provided by the new bytestring builder #10

Open
meiersi opened this Issue · 1 comment

2 participants

@meiersi

The decimal function in Data.Csv.Conversion.Internal seems to just format a number in decimal format. I was not sure if this really is the case. However, if it is, then one could just replace the calls to this functions with the appropriate intXDec functions from the new bytestring builder. Note that the integerDec function is not the newest version. It will be fast, once I port my newest patches to the darcs repo. Somehow, the state of mine and Duncan's bytestring repository diverged :-/

@tibbe
Owner

Thanks for the tip. I will take a look when I find some spare time.

@meiersi meiersi referenced this issue from a commit in meiersi/cassava
@meiersi meiersi minor performance improvements
Avoid constructing intermediate lists. This mitigates the slowdown from the
switch to the new bytestring builder. More improvements are achievable from
issues #8, #9, and #10.
acc8d9a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.