Github user britter commented on the pull request:

    https://github.com/apache/commons-lang/pull/81#issuecomment-99921440
  
    Hello @eduardomartins-luizalabs,
    
    the following will fail:
    
    ```java
    assertFalse(WordUtils.containsAllWords("lorem ipsum null dolor sit amet", 
"ipsum", null, "lorem", "dolor"));
    ```
    
    This is because you only check for nulls at index 0 of the input array. 
IMHO null is not contained in any string. Have a look at for example 
```StringUtils.containsAny(CharSequence, CharSequence...)``` and 
adc6bc4104514063b3e8ebe7a301d5dd4bce36d3. Further more I wonder whether we 
should let this method operate on CharSequences. WDYT?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to