remove an empty destructor to please g++-9 and CRAN / BDR

Author: eddelbuettel

ChangeLog

@@ -1,3 +1,8 @@
+2018-09-27  Dirk Eddelbuettel  <[email protected]>
+
+	* inst/include/Rcpp/date_datetime/Date.h: Remove the empty destructor
+	to make g++-9 (prerelease) happy [CRAN request]
+
 2018-09-26  Dirk Eddelbuettel  <[email protected]>
 
 	* docker/ci/Dockerfile: Set env var RunAllRcppTests=yes

inst/NEWS.Rd

@@ -3,6 +3,21 @@
 \newcommand{\ghpr}{\href{https://github.com/RcppCore/Rcpp/pull/#1}{##1}}
 \newcommand{\ghit}{\href{https://github.com/RcppCore/Rcpp/issues/#1}{##1}}
 
+\section{Changes in Rcpp version 0.12.20 (2018-11-xx)}{
+  \itemize{
+    \item Changes in Rcpp API:
+    \itemize{
+      \item The empty destructor for the \code{Date} class was removed
+      to please g++-9 (prerelease) and \code{-Wdeprecated-copy} (Dirk).
+    }
+    \item Changes in Rcpp Deployment:
+    \itemize{
+      \item The Dockerfile for Continuous Integration sets the required
+      test flag (for release versions) inside the container (Dirk).
+    }
+  }
+}
+
 \section{Changes in Rcpp version 0.12.19 (2018-09-20)}{
   \itemize{
     \item Changes in Rcpp API:

inst/include/Rcpp/date_datetime/Date.h

@@ -67,8 +67,6 @@ namespace Rcpp {
             m_d = tmp/(24*60*60);
         }
 
-        ~Date() {};
-
         double getDate(void) const {
             return m_d;
         }