Skip to content
This repository has been archived by the owner on May 29, 2020. It is now read-only.

Some cosmetic changes #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Some cosmetic changes #27

wants to merge 1 commit into from

Conversation

lambdaterm
Copy link

Some cosmetic changes. Removed semicolons and return operator.

@@ -77,10 +77,10 @@ class PorterStemmer {
def vowelInStem(s: String): Boolean = {
for (i <- 0 to b.length - 1 - s.length) {
if (!cons(i)) {
return true
true
Copy link
Member

Choose a reason for hiding this comment

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

Sorry, this has to be return.

Copy link
Member

Choose a reason for hiding this comment

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

alternatively: !s.slice(0, b.length - s.length).forall(cons)

@dlwh
Copy link
Member

dlwh commented Dec 18, 2014

also, we're deprecating Chalk in favor of Epic.

@dlwh
Copy link
Member

dlwh commented Dec 18, 2014

github.com/dlwh/epic

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants