style: pre-commit fixes

MikePopoloski · Nov 4, 2022 · 1ba07a0 · 1ba07a0
1 parent dc1788a
commit 1ba07a0
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 15 deletions.
diff --git a/bindings/python/types.cpp b/bindings/python/types.cpp
@@ -119,8 +119,8 @@ void registerTypes(py::module_& m) {
         .def_readwrite("elideScopeNames", &TypePrintingOptions::elideScopeNames)
         .def_readwrite("printAKA", &TypePrintingOptions::printAKA)
         .def_readwrite("anonymousTypeStyle", &TypePrintingOptions::anonymousTypeStyle);
-        .def_readwrite("skipScopedTypeNames", &TypePrintingOptions::anonymousTypeStyle);
-        .def_readwrite("fullEnumType", &TypePrintingOptions::anonymousTypeStyle);
+    .def_readwrite("skipScopedTypeNames", &TypePrintingOptions::anonymousTypeStyle);
+    .def_readwrite("fullEnumType", &TypePrintingOptions::anonymousTypeStyle);
 
     py::enum_<TypePrintingOptions::AnonymousTypeStyle>(typePrintingOptions, "AnonymousTypeStyle")
         .value("SystemName", TypePrintingOptions::SystemName)

diff --git a/include/slang/numeric/ConstantValue.h b/include/slang/numeric/ConstantValue.h
@@ -147,8 +147,9 @@ class SLANG_EXPORT ConstantValue {
     Variant& getVariant() { return value; }
     const Variant& getVariant() const { return value; }
 
-    std::string toString(bitwidth_t abbreviateThresholdBits = SVInt::DefaultStringAbbreviationThresholdBits,
-                         bool exactUnknowns = false, bool useAssignmentPatterns = false) const;
+    std::string toString(
+        bitwidth_t abbreviateThresholdBits = SVInt::DefaultStringAbbreviationThresholdBits,
+        bool exactUnknowns = false, bool useAssignmentPatterns = false) const;
     size_t hash() const;
 
     [[nodiscard]] bool empty() const;

diff --git a/include/slang/numeric/SVInt.h b/include/slang/numeric/SVInt.h
@@ -296,14 +296,15 @@ class SLANG_EXPORT SVInt : SVIntStorage {
     static SVInt createFillZ(bitwidth_t bitWidth, bool isSigned);
 
     [[nodiscard]] size_t hash() const;
-    void writeTo(SmallVectorBase<char>& buffer, LiteralBase base, bitwidth_t
-        abbreviateThresholdBits = DefaultStringAbbreviationThresholdBits) const;
+    void writeTo(SmallVectorBase<char>& buffer, LiteralBase base,
+                 bitwidth_t abbreviateThresholdBits = DefaultStringAbbreviationThresholdBits) const;
     void writeTo(SmallVectorBase<char>& buffer, LiteralBase base, bool includeBase,
                  bitwidth_t abbreviateThresholdBits = MAX_BITS) const;
-    std::string toString(bitwidth_t abbreviateThresholdBits =
-            DefaultStringAbbreviationThresholdBits, bool exactUnknowns = false) const;
+    std::string toString(
+        bitwidth_t abbreviateThresholdBits = DefaultStringAbbreviationThresholdBits,
+        bool exactUnknowns = false) const;
     std::string toString(LiteralBase base, bitwidth_t abbreviateThresholdBits =
-            DefaultStringAbbreviationThresholdBits) const;
+                                               DefaultStringAbbreviationThresholdBits) const;
     std::string toString(LiteralBase base, bool includeBase,
                          bitwidth_t abbreviateThresholdBits = MAX_BITS) const;
 

diff --git a/source/ast/types/TypePrinter.cpp b/source/ast/types/TypePrinter.cpp
@@ -105,7 +105,8 @@ void TypePrinter::visit(const EnumType& type, string_view overrideName) {
         }
         buffer->append("}");
 
-        if (options.skipScopedTypeNames);
+        if (options.skipScopedTypeNames)
+            ;
         else if (!overrideName.empty())
             buffer->append(overrideName);
         else {
@@ -147,7 +148,8 @@ void TypePrinter::visit(const PackedStructType& type, string_view overrideName)
 
         appendMembers(type);
 
-        if (options.skipScopedTypeNames);
+        if (options.skipScopedTypeNames)
+            ;
         else if (!overrideName.empty())
             buffer->append(overrideName);
         else {
@@ -173,7 +175,8 @@ void TypePrinter::visit(const PackedUnionType& type, string_view overrideName) {
 
         appendMembers(type);
 
-        if (options.skipScopedTypeNames);
+        if (options.skipScopedTypeNames)
+            ;
         else if (!overrideName.empty())
             buffer->append(overrideName);
         else {
@@ -248,7 +251,8 @@ void TypePrinter::visit(const UnpackedStructType& type, string_view overrideName
         buffer->append("struct");
         appendMembers(type);
 
-        if (options.skipScopedTypeNames);
+        if (options.skipScopedTypeNames)
+            ;
         else if (!overrideName.empty())
             buffer->append(overrideName);
         else {
@@ -271,7 +275,8 @@ void TypePrinter::visit(const UnpackedUnionType& type, string_view overrideName)
         buffer->append("union");
         appendMembers(type);
 
-        if (options.skipScopedTypeNames);
+        if (options.skipScopedTypeNames)
+            ;
         else if (!overrideName.empty())
             buffer->append(overrideName);
         else {

diff --git a/source/numeric/ConstantValue.cpp b/source/numeric/ConstantValue.cpp
@@ -84,7 +84,8 @@ std::string ConstantValue::toString(bitwidth_t abbreviateThresholdBits, bool exa
                 if (!arg->activeMember)
                     return "(unset)"s;
 
-                return fmt::format("({}) {}", *arg->activeMember, arg->value.toString(abbreviateThresholdBits));
+                return fmt::format("({}) {}", *arg->activeMember,
+                                   arg->value.toString(abbreviateThresholdBits));
             }
             else {
                 static_assert(always_false<T>::value, "Missing case");