Print

Print


@amadio commented on this pull request.


In src/XrdCl/XrdClAsyncMsgReader.hh:

> @@ -168,6 +169,23 @@ namespace XrdCl
               continue;
             }
             //------------------------------------------------------------------
+            // kXR_status is special as it can have both body and raw data,
+            // handle it separately
+            //------------------------------------------------------------------
+            case ReadMore:
+            {
+              XRootDStatus st = xrdTransport.GetMore( *inmsg, &socket );
+              if( !st.IsOK() || st.code == suRetry )
+                return st;
+              inmsgsize = inmsg->GetCursor();
+
+              //----------------------------------------------------------------
+              // The next step is to finalize the read
+              //----------------------------------------------------------------
+              readstage = ReadDone;

What I wonder is if setting readstage = ReadDone will not prevent ReadMore from being called twice in a row if needed.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <xrootd/xrootd/pull/1854/review/1296324170@github.com>

[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/xrootd/xrootd/pull/1854#discussion_r1104939274", "url": "https://github.com/xrootd/xrootd/pull/1854#discussion_r1104939274", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1