Use an enum class.
authorRafael Espindola <rafael.espindola@gmail.com>
Wed, 11 Jun 2014 01:09:09 +0000 (01:09 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Wed, 11 Jun 2014 01:09:09 +0000 (01:09 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@210620 91177308-0d34-0410-b5e6-96231b3b80d8

tools/obj2yaml/Error.cpp
tools/obj2yaml/Error.h

index 7f25d745e1cb5a8c5107108974dd0b9687d9fc86..3aa472fc3dd34e304aecbdbb1934bea388463028 100644 (file)
@@ -24,7 +24,7 @@ public:
 const char *_obj2yaml_error_category::name() const { return "obj2yaml"; }
 
 std::string _obj2yaml_error_category::message(int ev) const {
-  switch (ev) {
+  switch (static_cast<obj2yaml_error>(ev)) {
   case obj2yaml_error::success:
     return "Success";
   case obj2yaml_error::file_not_found:
@@ -33,15 +33,14 @@ std::string _obj2yaml_error_category::message(int ev) const {
     return "Unrecognized file type.";
   case obj2yaml_error::unsupported_obj_file_format:
     return "Unsupported object file format.";
-  default:
-    llvm_unreachable("An enumerator of obj2yaml_error does not have a message "
-                     "defined.");
   }
+  llvm_unreachable("An enumerator of obj2yaml_error does not have a message "
+                   "defined.");
 }
 
 error_condition
 _obj2yaml_error_category::default_error_condition(int ev) const {
-  if (ev == obj2yaml_error::success)
+  if (static_cast<obj2yaml_error>(ev) == obj2yaml_error::success)
     return error_condition();
   return errc::invalid_argument;
 }
index a32666455514c5ce407f34e5844c5a9fc0eff1b8..4aaf5c7942112b81a4c205bcd007c83b5cc66640 100644 (file)
@@ -16,18 +16,11 @@ namespace llvm {
 
 const error_category &obj2yaml_category();
 
-struct obj2yaml_error {
-  enum _ {
-    success = 0,
-    file_not_found,
-    unrecognized_file_format,
-    unsupported_obj_file_format
-  };
-  _ v_;
-
-  obj2yaml_error(_ v) : v_(v) {}
-  explicit obj2yaml_error(int v) : v_(_(v)) {}
-  operator int() const {return v_;}
+enum class obj2yaml_error {
+  success = 0,
+  file_not_found,
+  unrecognized_file_format,
+  unsupported_obj_file_format
 };
 
 inline error_code make_error_code(obj2yaml_error e) {
@@ -35,7 +28,6 @@ inline error_code make_error_code(obj2yaml_error e) {
 }
 
 template <> struct is_error_code_enum<obj2yaml_error> : std::true_type { };
-template <> struct is_error_code_enum<obj2yaml_error::_> : std::true_type { };
 
 } // namespace llvm