Skip to content

Implemented a HashMap in the EmojiManager and Related Features#9

Open
mezcalhead wants to merge 1 commit intokcthota:masterfrom
mezcalhead:feature/extras
Open

Implemented a HashMap in the EmojiManager and Related Features#9
mezcalhead wants to merge 1 commit intokcthota:masterfrom
mezcalhead:feature/extras

Conversation

@mezcalhead
Copy link
Copy Markdown

Implemented a HashMap in the EmojiManager to allow access to the Emoji resource data quickly. Added a new function to return the Emojis found in a String populated with data from the Emoji resource data.

resource data quickly.  Added a new function to return the Emojis found
in a String populated with data from the Emoji resource data.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-6.4%) to 91.371% when pulling fed0d7f on mezcalhead:feature/extras into 62ac1f9 on kcthota:master.

@mezcalhead
Copy link
Copy Markdown
Author

mezcalhead commented Nov 2, 2017

The features I added allow for the following critical use case:

List<Emoji> emojis = EmojiUtils.parseEmojis(SOME_TEXT); for (Emoji e : emojis) { System.out.println(e.getEmoji() + ": " + e.getDescription()); if (e.getAliases() != null) { for (int j=0; j<e.getAliases().size(); j++) { System.out.println("Alias #" + (j+1) + ": " + e.getAliases().get(j)); } }

I am not sure why GitHub diff'ed the way it did. I didn't change many lines at all. New line issue?

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.

2 participants