Merge pull request #256 from robotpy/minor-updates

Minor updates

Author: virtuald

robotpy_build/autowrap/render_cls_rpy_include.py

@@ -40,14 +40,15 @@ def render_cls_rpy_include_hpp(ctx: HeaderContext, cls: ClassContext) -> str:
     )
 
     if ctx.extra_includes_first:
-        r.writeln()
+        r.writeln("\n// from extra_includes_first")
         for inc in ctx.extra_includes_first:
             r.writeln(f"#include <{inc}>")
 
+    r.writeln("\n// wrapped header")
     r.writeln(f"\n#include <{ctx.rel_fname}>")
 
     if ctx.extra_includes:
-        r.writeln()
+        r.writeln("\n// from extra_includes")
         for inc in ctx.extra_includes:
             r.writeln(f"#include <{inc}>")
 

tests/cpp/gen/ft/tbasic.yml

@@ -7,7 +7,7 @@ classes:
       t:
     template_inline_code: |
       cls_TBasic
-        .def("__repr__", [=](const TBasic<T> &self){
+        .def("__repr__", [this](const TBasic<T> &self){
           // checking to see if clsName is available
           return "<" + clsName + ">";
         });

tests/cpp/rpytest/ft/include/abstract.h

@@ -8,6 +8,7 @@ struct Abstract
 struct PrivateAbstract
 {
     PrivateAbstract() {}
+    virtual ~PrivateAbstract() = default;
 
     static int getPrivateOverride(PrivateAbstract *p) {
         return p->mustOverrideMe();

tests/cpp/rpytest/ft/include/buffers.h

@@ -6,6 +6,8 @@
 class Buffers {
 public:
 
+    virtual ~Buffers() = default;
+
     // in
     void set_buffer(const uint8_t *data, size_t len) {
         m_buf.resize(len);

tests/cpp/rpytest/ft/include/lifetime.h

@@ -3,6 +3,8 @@
 #include <memory>
 
 struct LTWithVirtual {
+    virtual ~LTWithVirtual() = default;
+
     virtual bool get_bool() {
         return false;
     }

tests/cpp/rpytest/ft/include/ns_hidden.h

@@ -12,7 +12,7 @@ namespace o {
         virtual ~O() = default;
     };
 
-    class AnotherC;
+    struct AnotherC;
 };
 
 namespace n::h {

tests/cpp/rpytest/ft/include/templates/tvbase.h

@@ -10,6 +10,8 @@ struct TVParam {
 template <typename T>
 struct TVBase {
 
+    virtual ~TVBase() = default;
+
     virtual std::string get(T t) const {
         return "TVBase " + std::to_string(t.get());
     }