Print

Print


Hi Andy, 

After looking over the code, I get you point related to the `Ignore` flag but this also means that this branch of the `switch` is dead code: https://github.com/xrootd/xrootd/blob/master/src/XrdCl/XrdClXRootDMsgHandler.cc#L560

Since such a response as you said in the comments is handled synchronously here: https://github.com/xrootd/xrootd/blob/master/src/XrdCl/XrdClStream.cc#L449

All this is true if a `kXR_attn` response can not contain an embedded `kXR_waitreps`. Can it? If not, what do you think about removing the dead case branch?

Elvin


---
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/issues/290#issuecomment-145855682

########################################################################
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