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

Add vilebotmoji #92

Merged
merged 1 commit into from
Aug 1, 2018
Merged

Add vilebotmoji #92

merged 1 commit into from
Aug 1, 2018

Conversation

onyiny-ang
Copy link
Contributor

Add support for a specific kaomoji for vilebot: ( ͡° ͜ʖ ͡° )

Copy link
Contributor

@Christopher-Chianelli Christopher-Chianelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR LGTM, but I noticed the code used bitwise or | above the addition. This cause a potential additional evaluation of equals that is not needed. It should be changed to the short-circuit or, ||. But that is for another PR.

@@ -55,6 +55,10 @@ private String kaomojiify(String message) throws Exception {
else if (words[0].equals("nsfw") | words[0].equals("wtf")){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think circuit-break or || is intended here instead of bitwise or |. Since both operands are booleans, the only different is that this might do an additional equals call that is not needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You’re right, that change would be preferable. I might as well change it in this PR.

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