Print

Print


Actually, the fix we will be using is to issue an error message is the 
status is not clean. That will be posted tomorrow.

Andy

On Mon, 16 Jan 2017, Brian Bockelman wrote:

> bbockelm commented on this pull request.
>
>
>
>> @@ -243,7 +243,7 @@ class ZipArchiveReaderImpl
>     {
>       delete pBuffer;
>       ClearRecords();
> -      pArchive.Close();
> +      XRootDStatus st = pArchive.Close();
>
> There is an attribute for this situation added in C++17; is there a pre-existing `gnu:` attribute that we could use?
>
> -- 
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub:
> https://github.com/xrootd/xrootd/pull/448


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/pull/448#issuecomment-272922750

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