Experimental Physics and
Industrial Control System

Martin Konrad has proposed merging ~info-martin-konrad/epics-base:remove-epicsthrows into epics-base:7.0.

Requested reviews:
  EPICS Core Developers (epics-core)

For more details, see:
https://code.launchpad.net/~info-martin-konrad/epics-base/+git/epics-base/+merge/375445

Potential impact on users, let's merge this into 7.0.
-- 
Your team EPICS Core Developers is requested to review the proposed merge of ~info-martin-konrad/epics-base:remove-epicsthrows into epics-base:7.0.

diff --git a/modules/libcom/src/osi/compiler/clang/compilerSpecific.h b/modules/libcom/src/osi/compiler/clang/compilerSpecific.h
index 2053227..0498f6e 100644
--- a/modules/libcom/src/osi/compiler/clang/compilerSpecific.h
+++ b/modules/libcom/src/osi/compiler/clang/compilerSpecific.h
@@ -33,10 +33,8 @@
 
 /*
  * CXX_PLACEMENT_DELETE - defined if compiler supports placement delete
- * CXX_THROW_SPECIFICATION - defined if compiler supports throw specification
  */
 #define CXX_PLACEMENT_DELETE
-#define CXX_THROW_SPECIFICATION
 
 #endif /* __cplusplus */
 
diff --git a/modules/libcom/src/osi/compiler/gcc/compilerSpecific.h b/modules/libcom/src/osi/compiler/gcc/compilerSpecific.h
index 13d9119..f053495 100644
--- a/modules/libcom/src/osi/compiler/gcc/compilerSpecific.h
+++ b/modules/libcom/src/osi/compiler/gcc/compilerSpecific.h
@@ -44,13 +44,8 @@
 
 /*
  * CXX_PLACEMENT_DELETE - defined if compiler supports placement delete
- * CXX_THROW_SPECIFICATION - defined if compiler supports throw specification
  */
 
-#if __GNUC__ > 2 || ( __GNUC__ == 2 && __GNUC_MINOR__ >= 95 )
-#   define CXX_THROW_SPECIFICATION
-#endif
-
 #if __GNUC__ > 2 || ( __GNUC__ == 2 && __GNUC_MINOR__ >= 96 )
 #   define CXX_PLACEMENT_DELETE
 #endif
diff --git a/modules/libcom/src/osi/compilerDependencies.h b/modules/libcom/src/osi/compilerDependencies.h
index 3bd835c..0b333d7 100644
--- a/modules/libcom/src/osi/compilerDependencies.h
+++ b/modules/libcom/src/osi/compilerDependencies.h
@@ -21,18 +21,6 @@
 #ifdef __cplusplus
 
 /*
- * usage: void func () epicsThrows (( std::bad_alloc, std::logic_error ))
- *
- * Note: now a widely accepted concensus (ref Meyers and C++ faq) is that 
- * one should avoid using throw specifications in C++ code
- */
-#if defined ( CXX_THROW_SPECIFICATION )
-#   define epicsThrows(X) throw X
-#else
-#   define epicsThrows(X)
-#endif
-
-/*
  * usage: epicsPlacementDeleteOperator (( void *, myMemoryManager & ))
  */
 #if defined ( CXX_PLACEMENT_DELETE )

Replies:

Re: [Merge] ~info-martin-konrad/epics-base:remove-epicsthrows into epics-base:7.0 mdavidsaver via Core-talk

Re: [Merge] ~info-martin-konrad/epics-base:remove-epicsthrows into epics-base:7.0 mdavidsaver via Core-talk

[Merge] ~info-martin-konrad/epics-base:remove-epicsthrows into epics-base:7.0 mdavidsaver via Core-talk

Navigate by Date:

Prev: New day/time for teleconference? Konrad, Martin via Core-talk

Next: [Merge] ~info-martin-konrad/epics-base:retire-epicsexcept into epics-base:3.15 Martin Konrad via Core-talk

Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  <2019>  2020  2021  2022  2023  2024 

Navigate by Thread:

Prev: Re: New day/time for teleconference? Ralph Lange via Core-talk

Next: Re: [Merge] ~info-martin-konrad/epics-base:remove-epicsthrows into epics-base:7.0 mdavidsaver via Core-talk

Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  <2019>  2020  2021  2022  2023  2024