Changed warning to notice for acces on non-object error

Summary: Fixes: #1905

This is similar to #1949. I don't get bad tests for current master.
Closes #2746

Reviewed By: @fredemmott

Differential Revision: D1369945

Pulled By: @JoelMarcey
Fix behavior of Redis client

Summary: Because HHVM forces auto_detect_line_ending to be enabled (which is a separate
bug), if a CRLF is split across two TCP packets, fgets() is liable to get
confused and assume that the stream uses CR line ending. As a result, the
return value of Redis::sockReadLine() is liable to have either a trailing CR or
a leading LF, both of which are corruptions. This causes the client to throw an
exception, complaining of a protocol error.

The behavior of fgets() is documented here:
http://www.php.net/manual/en/function.fgets.php#101963

The Redis protocol spec affirms that responses terminate with CRLF and are
guaranteed not to contain either CR or LF in the response body:
http://redis.io/topics/protocol
Closes #2874

Reviewed By: @ptarjan

Differential Revision: D1370933

Pulled By: @JoelMarcey