[PATCH] D33180: Add functionality to cvtres to parse all entries in res file.

[Eric Beckmann via Phabricator via llvm-commits]

ecbeckmann added inline comments.


================
Comment at: llvm/include/llvm/Object/WindowsResource.h:59
+ public:
+  ~WindowsResource() override;
+
----------------
ruiu wrote:
> Do you need this?
Yes because the destructor is declared virtual in the base class.


https://reviews.llvm.org/D33180