Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some performance related changes #14

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Commits on Sep 11, 2013

  1. Some performance related changes

    Performance related:
    1. Caching of Stringbuilder on RecordOperations.cs for "RecordToString"
    and "RecordValuesToString" methods to lessen a little the burden on the
    GC (I actually didn't measure anything...)
    2. Change to use "CompareOptions.OrdinalIgnoreCase" and
    "CompareOptions.Ordinal" whenever possible (prety good performance gain
    whenever we have some heavy string working to do...)
    3. Change to use "ToLowerInvariant" instead of "ToLower" for a small
    performance gain (10%)
    
    Access Visibility
    1. some access visibility level changes to useful Field properties on
    "FieldBase", "FixedLengthField" and "DelimitedField" (similar to the
    previous change on atributes!)
    
    Other changes
    1. FileHelperAsyncEngine.EOF, it is userful if using this engine using
    ReadNextRecord () or ReadNexts () (usage in a while loop for example)
    2. FileHelperAsyncEngine.Close, add a dispose to the TextWriter just as
    a precaution...
    khalidsalomao committed Sep 11, 2013
    Configuration menu
    Copy the full SHA
    3305605 View commit details
    Browse the repository at this point in the history

Commits on Sep 13, 2013

  1. Add to ErrorManager an recording limit

    Add to ErrorManager an recording limit to avoid possible out-of-memory
    errors in case of layout errors when dealing with large files.
    
    After the threshold is reached, successive errors are ignored.
    khalidsalomao committed Sep 13, 2013
    Configuration menu
    Copy the full SHA
    00483b5 View commit details
    Browse the repository at this point in the history