-
Notifications
You must be signed in to change notification settings - Fork 24
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
Duplicate key/value pairs #3
Comments
Good catch! I have a patch for both JavelinJS and this branch, in my mythical spare time. |
I was working on a fixed hash but then ran into the duplicates that have different values and wasn't sure what to do with them. |
Hmm. I won't have time to finish my patch for a while. Maybe send / post the dupes? We can probably work out which values are "right". |
Here's what I found: dupe keys (removed):
dupe keys with different values:
Here's the updated hash with above changes: https://gist.github.com/2772409 |
Also noticed some missing ones that are relatively common: Ææ => should be replaced with "ae" And some inaccurate ones: I'll go ahead and add the missing ones. What do you think about the inaccurate ones? |
Here's an updated hash with the missing ones added and inaccurate ones fixed: https://gist.github.com/2772435 |
There appears to be a grip of duplicate key/value pairs in this hash.. For example Ł is in there as a key twice. Some of the duplicates have the same key with different values. I noticed this cause ExecJS complains about duplicate keys in hashes when it compiles Coffee to JS (I ported the hash to Coffee real quick).
The text was updated successfully, but these errors were encountered: