Print "warning" for ignored custom section errors (#843)

The previous message said "error", which makes it look like the output
is not created, so change the message to "warning" instead.

The error handling code is pretty ugly and can use a refactor, but that
would be a much larger change.

1 files changed, 5 insertions, 2 deletions

diff --git a/src/wast-parser.cc b/src/wast-parser.cc
index 3381b65e..ae793bf8 100644
--- a/src/wast-parser.cc
+++ b/src/wast-parser.cc
@@ -108,7 +108,8 @@ void RemoveEscapes(const TextVector& texts, OutputIter out) {
 class BinaryErrorHandlerModule : public ErrorHandler {
  public:
   BinaryErrorHandlerModule(Location* loc, WastParser* parser);
-  bool OnError(const Location&,
+  bool OnError(ErrorLevel,
+               const Location&,
                const std::string& error,
                const std::string& source_line,
                size_t source_line_column_offset) override;
@@ -125,10 +126,12 @@ BinaryErrorHandlerModule::BinaryErrorHandlerModule(Location* loc,
                                                    WastParser* parser)
     : ErrorHandler(Location::Type::Binary), loc_(loc), parser_(parser) {}
 
-bool BinaryErrorHandlerModule::OnError(const Location& binary_loc,
+bool BinaryErrorHandlerModule::OnError(ErrorLevel error_level,
+                                       const Location& binary_loc,
                                        const std::string& error,
                                        const std::string& source_line,
                                        size_t source_line_column_offset) {
+  assert(error_level == ErrorLevel::Error);
   if (binary_loc.offset == kInvalidOffset) {
     parser_->Error(*loc_, "error in binary module: %s", error.c_str());
   } else {