Skip to content

This adds the possibility to use 'ignorecase' access, to headers mapping#9

Closed
yamilmedina wants to merge 1 commit into
apache:trunkfrom
yamilmedina:trunk
Closed

This adds the possibility to use 'ignorecase' access, to headers mapping#9
yamilmedina wants to merge 1 commit into
apache:trunkfrom
yamilmedina:trunk

Conversation

@yamilmedina

Copy link
Copy Markdown

Hello, please review this contribution.
I added a 'minor' feature, the possibility to use "ignorecase" access in HeaderMapping.
The change is basically the use of an instance of TreeMap + String.CASE_INSENSITIVE_ORDER comparator:

new TreeMap<String, Integer>(String.CASE_INSENSITIVE_ORDER)

when filling headerMap, this based on a new flag in CSVFormat class, that handles this optional format.

This was originally issued by this SO question -> http://stackoverflow.com/q/29911366/2619091 .
The feature was encouraged by @britter response, plus "opencsv" already has this feature I decided to make this small contribution.

PS: I didn't know if I had to open a JIRA issue for this, so I'll stay tuned for replies in case I have to open one.

@yamilmedina

Copy link
Copy Markdown
Author

JIRA ticket is: https://issues.apache.org/jira/browse/CSV-159

@garydgregory

Copy link
Copy Markdown
Member

Patch applied.
Thank you Yamil.
Committed SVN revision 1708685.
Please verify trunk and close this issue if satisfied.

DarrenJAN added a commit to DarrenJAN/commons-csv that referenced this pull request Nov 7, 2024
…apache#9)

Add support in Commons CSV for tracking byte positions during parsing
DarrenJAN added a commit to DarrenJAN/commons-csv that referenced this pull request Nov 13, 2024
…apache#9)

Add support in Commons CSV for tracking byte positions during parsing
DarrenJAN added a commit to DarrenJAN/commons-csv that referenced this pull request Nov 18, 2024
…apache#9)

Add support in Commons CSV for tracking byte positions during parsing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants