Print

Print


@bbockelm commented on this pull request.



> +    return mapper->IsValid() ? mapper.get() : nullptr;
+}
+
+
+void *
+XrdVomsMapfile::MaintenanceThread(void *myself_raw)
+{
+    auto myself = static_cast<XrdVomsMapfile*>(myself_raw);
+
+   auto now = monotonic_time_s();
+   auto next_update = now + m_update_interval;
+   while (true) {
+       now = monotonic_time_s();
+       auto remaining = next_update - now;
+       struct pollfd fds;
+       fds.fd = myself->m_maintenance_pipe_r;

It is a bit overkill - I only triggered it during the destruction of the object when the startup configuration failed.

I figured I'd err toward meticulous cleanup (the inspiration is in the temp CA handler code) instead of just assuming nothing needs to be torn down.

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

Message ID: <[log in to unmask]>

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