[cpp-sp] 04/04: Remove declaration of unused exceptions

Rod Widdowson rdw at steadingsoftware.com
Mon Jun 23 13:40:55 UTC 2025


This is an automated email from the git hooks/post-receive script.

rdw pushed a commit to branch main
in repository cpp-sp.

View the commit online:
http://git.shibboleth.net/view/?p=cpp-sp.git;a=commit;h=aee9e12d4f85945fdcba7749d64f999caefe09c9

commit aee9e12d4f85945fdcba7749d64f999caefe09c9
Author: Rod Widdowson <rdw at steadingsoftware.com>
AuthorDate: Mon Jun 23 14:36:16 2025 +0100

    Remove declaration of unused exceptions
---
 shibsp/session/impl/AbstractSessionCache.cpp | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/shibsp/session/impl/AbstractSessionCache.cpp b/shibsp/session/impl/AbstractSessionCache.cpp
index a7479ef3..61b26567 100644
--- a/shibsp/session/impl/AbstractSessionCache.cpp
+++ b/shibsp/session/impl/AbstractSessionCache.cpp
@@ -271,7 +271,7 @@ string AbstractSessionCache::create(SPRequest& request, DDF& session)
         m_log.debug("writing new session to persistent store");
         key = cache_create(&request, session);
     }
-    catch (const exception& ex) {
+    catch (const exception&) {
         // Should be logged by the SPI.
         session.destroy();
         throw;
@@ -401,7 +401,7 @@ unique_lock<Session> AbstractSessionCache::_find(
         // Note this performs the relevant enforcement for us.
         obj = cache_read(request, applicationId, key, lifetime, timeout, client_addr);
     }
-    catch (const exception& ex) {
+    catch (const exception&) {
         // Should be logged by the SPI.
         return unique_lock<Session>();
     }
@@ -460,7 +460,7 @@ void AbstractSessionCache::remove(SPRequest& request)
     try {
         cache_remove(&request, key);
     }
-    catch (const exception& ex) {
+    catch (const exception&) {
         // Should be logged by the SPI.
     }
     m_cookieManager->unsetCookie(request);
@@ -657,7 +657,7 @@ bool BasicSession::isValid(SPRequest* request, unsigned int lifetime, unsigned i
                     return false;
                 }
             }
-            catch (const exception& ex) {
+            catch (const exception&) {
                 // Should be logged by the SPI.
                 return false;
             }
@@ -674,7 +674,7 @@ bool BasicSession::isValid(SPRequest* request, unsigned int lifetime, unsigned i
                 return false;
             }
         }
-        catch (const exception& ex) {
+        catch (const exception&) {
             // Should be logged by the SPI.
             return false;
         }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list