ArrayUtils::inArray always call strict version of in_array

[adamlundrigan]

Proposed fix for #41.

[Maks3w]

I disagree this is a bug. For me this is an PoC attached to the discussion #41.

[Ocramius]

Yep, I was going to merge, then realized that it needed further discussion.
On Oct 27, 2015 7:04 PM, "Maks3w" notifications@github.com wrote:

I disagree this is a bug. For me this is an PoC attached to the discussion
#41 #41.

—
Reply to this email directly or view it on GitHub
#42 (comment)
.

[Maks3w]

I'll close this just for to prevent accidental merge.

[adamlundrigan]

Why close? It's a valid possible fix regardless of the fact that it's not been fully discussed yet. I opened it to facilitate discussion on the fix. Was that bad form?

[Maks3w]

I'm not discarding your PR. As documentation still been useful (the travis report still accesible)

Just I'm preventing accidental merge while still under discussion.

[adamlundrigan]

No problem, it just caught me off-guard as I've never seen that before (closing a PR because an accidental merge might cause problems).

I should have let that response bounce around in my noggin a bit first...re-reading it now it sounds abrupt, which wasn't my intention.