style(clang-tidy): fix bugprone-branch-clone errors

These are reported with sourcemeta#1754
Refs: sourcemeta#1654

Signed-off-by: Balakrishna Avulapati <ba@bavulapati.com>

Author: bavulapati

src/core/jsonschema/frame.cc

@@ -669,16 +669,9 @@ auto SchemaFrame::analyse(const JSON &root, const SchemaWalker &walker,
             if (!maybe_relative_is_absolute ||
                 maybe_match == this->locations_.cend()) {
               assert(entry.common.base_dialect.has_value());
-              if (entry.common.orphan) {
-                store(this->locations_, this->instances_,
-                      SchemaReferenceType::Static,
-                      SchemaFrame::LocationType::Resource, new_id, root_id,
-                      new_id, entry.common.pointer,
-                      sourcemeta::core::empty_pointer,
-                      entry.common.dialect.value(),
-                      entry.common.base_dialect.value(), {},
-                      entry.common.parent);
-              } else if (this->mode_ == SchemaFrame::Mode::Instances) {
+
+              if (!(entry.common.orphan) &&
+                  this->mode_ == SchemaFrame::Mode::Instances) {
                 store(this->locations_, this->instances_,
                       SchemaReferenceType::Static,
                       SchemaFrame::LocationType::Resource, new_id, root_id,
@@ -880,15 +873,8 @@ auto SchemaFrame::analyse(const JSON &root, const SchemaWalker &walker,
           const auto subschema{subschemas.find(pointer)};
           if (subschema != subschemas.cend()) {
             // Handle orphan schemas
-            if (subschema->second.orphan) {
-              store(this->locations_, this->instances_,
-                    SchemaReferenceType::Static,
-                    SchemaFrame::LocationType::Subschema, result, root_id,
-                    current_base, pointer,
-                    pointer.resolve_from(nearest_bases.second),
-                    dialects.first.front(), current_base_dialect, {},
-                    subschema->second.parent);
-            } else if (this->mode_ == SchemaFrame::Mode::Instances) {
+            if (!(subschema->second.orphan) &&
+                this->mode_ == SchemaFrame::Mode::Instances) {
               store(this->locations_, this->instances_,
                     SchemaReferenceType::Static,
                     SchemaFrame::LocationType::Subschema, result, root_id,

src/core/jsonschema/walker.cc

@@ -376,15 +376,10 @@ auto walk(const std::optional<sourcemeta::core::Pointer> &parent,
 
         break;
       case sourcemeta::core::SchemaKeywordType::Assertion:
-        break;
       case sourcemeta::core::SchemaKeywordType::Annotation:
-        break;
       case sourcemeta::core::SchemaKeywordType::Reference:
-        break;
       case sourcemeta::core::SchemaKeywordType::Other:
-        break;
       case sourcemeta::core::SchemaKeywordType::Comment:
-        break;
       case sourcemeta::core::SchemaKeywordType::Unknown:
         break;
     }

src/core/uri/escaping.cc

@@ -23,50 +23,22 @@ auto uri_escape(const char character, std::ostream &output,
   } else if (mode == URIEscapeMode::SkipSubDelims) {
     switch (character) {
       case '!':
-        output.put(character);
-        break;
       case '$':
-        output.put(character);
-        break;
       case '&':
-        output.put(character);
-        break;
       case '\'':
-        output.put(character);
-        break;
       case '(':
-        output.put(character);
-        break;
       case ')':
-        output.put(character);
-        break;
       case '*':
-        output.put(character);
-        break;
       case '+':
-        output.put(character);
-        break;
       case ',':
-        output.put(character);
-        break;
       case ';':
-        output.put(character);
-        break;
       case '=':
-        output.put(character);
-        break;
 
       // unreserved = ALPHA / DIGIT / "-" / "." / "_" / "~"
       // See https://www.rfc-editor.org/rfc/rfc3986#appendix-A
       case '-':
-        output.put(character);
-        break;
       case '_':
-        output.put(character);
-        break;
       case '.':
-        output.put(character);
-        break;
       case '~':
         output.put(character);
         break;
@@ -82,14 +54,8 @@ auto uri_escape(const char character, std::ostream &output,
     // See https://www.rfc-editor.org/rfc/rfc3986#appendix-A
     switch (character) {
       case '-':
-        output.put(character);
-        break;
       case '_':
-        output.put(character);
-        break;
       case '.':
-        output.put(character);
-        break;
       case '~':
         output.put(character);
         break;