Documentation error in Array class


#1

documentation for Array::removeAllInstancesOf() is identical to Array::removeFirstMatchingValue() http://www.juce.com/api/classArray.html#adb629ebfbcba98f1f5e1ca47ccca594e

 

PR sent --> https://github.com/julianstorer/JUCE/pull/24


#2

Though I would agree that PRs would be nice, I recall Jules saying he doesn't accept them.

Edit: read here - http://www.juce.com/forum/topic/juce-now-available-github


#3

Thanks for the heads-up on that, Bill - I've fixed the comment now!

Yeah, sorry, there are practical and legal reasons why I can't accept pull requests directly, but feel free to use a pull request as a way of asking me to change something, if that's an easy way for you to do so.


#4

sure that sounds good - seems better than cluttering this forum with issues that do not require an answer


ive just found another actually in XmlElement doc - PR -> https://github.com/julianstorer/JUCE/pull/25

 


#5

Nice catch!