diff options
author | 2018-06-28 02:31:02 -0700 | |
---|---|---|
committer | 2018-06-28 02:33:33 -0700 | |
commit | 74a9f015270083d51e0e32e8390a33c3aecb9a2c (patch) | |
tree | 3d794a3187a0520675c891686f3ac75252e990f0 /src/tools/launcher/java_launcher.h | |
parent | 3dc384760a141f23ffdd098066d9e0661a20430f (diff) |
Add virtual destructor and override to BinaryLauncherBase and subclasses
Class with virtual functions must have virtual destructor. Subclasses should annotate overriden functions as `override`.
Found by Clang's `-Winconsistent-missing-override`.
Closes #5474.
PiperOrigin-RevId: 202445230
Diffstat (limited to 'src/tools/launcher/java_launcher.h')
-rw-r--r-- | src/tools/launcher/java_launcher.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/tools/launcher/java_launcher.h b/src/tools/launcher/java_launcher.h index d02a7165db..1881d587d9 100644 --- a/src/tools/launcher/java_launcher.h +++ b/src/tools/launcher/java_launcher.h @@ -35,7 +35,8 @@ class JavaBinaryLauncher : public BinaryLauncherBase { singlejar(false), print_javabin(false), classpath_limit(MAX_ARG_STRLEN) {} - ExitCode Launch(); + ~JavaBinaryLauncher() override = default; + ExitCode Launch() override; private: // If present, these flags should either be at the beginning of the command |