5 ms·

if (submission.title.indexOf("lisp")!=-1) submission.voteUp(); surely...

• DavidSJ 18y ago
  I think you mean: (if (search "lisp" (submission-title submission)) (incf (submission-score submission)))
  • ken 18y ago
    Oh dear, I hope axod wouldn't have used IF with a single then-form. That's just asking for trouble. :-)
    • DavidSJ 18y ago
      Are you worried about the return value or something?
      • ken 18y ago
        No. I'm worried somebody will add another form, without realizing it has the opposite meaning (i.e., else) from the INCF call. Traditionally, if you have only one then-form, you'd use WHEN.
        • DavidSJ 18y ago
          Ah, fair point. I actually made that mistake a couple days ago, in fact.
  • Maascamp 18y ago
    I think you mean if(strpos(strtolower($title.$body), 'lisp') !== false) ++$votes; Oh wait, this is Hacker news. Apologies.