Bug 1613985 - Use default for equivalent-to-default constructors/dest…

…ructors in mozglue. r=glandium Differential Revision: https://phabricator.services.mozilla.com/D65495
ambroff · Mar 10, 2020 · 900f48f · 900f48f
1 parent d4b991c
commit 900f48f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/mozglue/baseprofiler/public/BaseProfilingStack.h b/mozglue/baseprofiler/public/BaseProfilingStack.h
@@ -359,7 +359,7 @@ class ProfilingStackFrame {
 //
 class ProfilingStack final {
  public:
-  ProfilingStack() : stackPointer(0) {}
+  ProfilingStack() = default;
 
   MFBT_API ~ProfilingStack();
 
@@ -465,7 +465,7 @@ class ProfilingStack final {
   // This is an atomic variable that uses ReleaseAcquire memory ordering.
   // See the "Concurrency considerations" paragraph at the top of this file
   // for more details.
-  Atomic<uint32_t, ReleaseAcquire> stackPointer;
+  Atomic<uint32_t, ReleaseAcquire> stackPointer{0};
 };
 
 class AutoGeckoProfilerEntry;

diff --git a/mozglue/misc/Printf.h b/mozglue/misc/Printf.h
@@ -90,7 +90,7 @@ class PrintfTarget {
 
  protected:
   MFBT_API PrintfTarget();
-  virtual ~PrintfTarget() {}
+  virtual ~PrintfTarget() = default;
 
   /* Subclasses override this.  It is called when more output is
      available.  It may be called with len==0.  This should return