build: Add warning for non-virtual destructor and fix discovered issues
Change-Id: I1d67b9b63d84d8524580e80ac06c3efef860c56a
diff --git a/rib/readvertise/readvertise-policy.hpp b/rib/readvertise/readvertise-policy.hpp
index ebcc6c0..dce10b1 100644
--- a/rib/readvertise/readvertise-policy.hpp
+++ b/rib/readvertise/readvertise-policy.hpp
@@ -1,5 +1,5 @@
/* -*- Mode:C++; c-file-style:"gnu"; indent-tabs-mode:nil; -*- */
-/**
+/*
* Copyright (c) 2014-2017, Regents of the University of California,
* Arizona Board of Regents,
* Colorado State University,
@@ -45,6 +45,9 @@
class ReadvertisePolicy : noncopyable
{
public:
+ virtual
+ ~ReadvertisePolicy() = default;
+
/** \brief decide whether to readvertise a route, and what prefix to readvertise
*/
virtual ndn::optional<ReadvertiseAction>