Review feedback

Author: drewnoakes

src/NetMQ/Core/Patterns/Server.cs

@@ -67,8 +67,6 @@ public Server(Ctx parent, int threadId, int socketId)
         /// <param name="icanhasall">not used</param>
         protected override void XAttachPipe(Pipe pipe, bool icanhasall)
         {
-            Assumes.NotNull(pipe);
-
             uint routingId = m_nextRoutingId++;
             if (routingId == 0)
                 routingId = m_nextRoutingId++; //  Never use Routing ID zero

src/NetMQ/Core/YPipe.cs

@@ -178,7 +178,7 @@ public bool CheckRead()
         /// Attempts to read an item from the pipe.
         /// </summary>
         /// <returns><c>true</c> if the read succeeded, otherwise <c>false</c>.</returns>
-        public bool TryRead([MaybeNull] out T value)
+        public bool TryRead([MaybeNullWhen(returnValue: false)] out T value)
         {
             // Try to prefetch a value.
             if (!CheckRead())