Eliminate unnecessary, trailing else conditional blocks.

src/frontend/terminaloverlay.cc

@@ -137,9 +137,8 @@ Validity ConditionalCursorMove::get_validity( const Framebuffer &fb,
     if ( (fb.ds.get_cursor_col() == col)
 	 && (fb.ds.get_cursor_row() == row) ) {
       return Correct;
-    } else {
-      return IncorrectOrExpired;
     }
+    return IncorrectOrExpired;
   }
 
   return Pending;

src/statesync/completeterminal.cc

@@ -173,9 +173,8 @@ int Complete::wait_time( uint64_t now ) const
   uint64_t next_echo_ack_time = it->second + ECHO_TIMEOUT;
   if ( next_echo_ack_time <= now ) {
     return 0;
-  } else {
-    return next_echo_ack_time - now;
   }
+  return next_echo_ack_time - now;
 }
 
 bool Complete::compare( const Complete &other ) const

src/terminal/terminaldispatcher.cc

@@ -226,12 +226,11 @@ void Dispatcher::dispatch( Function_Type type, const Parser::Action *act, Frameb
     /* unknown function */
     fb->ds.next_print_will_wrap = false;
     return;
-  } else {
+  }
   if ( i->second.clears_wrap_state ) {
     fb->ds.next_print_will_wrap = false;
   }
-    return i->second.function( fb, this );
-  }
+  i->second.function( fb, this );
 }
 
 void Dispatcher::OSC_put( const Parser::OSC_Put *act )

src/terminal/terminalframebuffer.cc

@@ -229,7 +229,7 @@ int DrawState::get_next_tab( int count ) const
       }
     }
     return -1;
-  } else {
+  }
   for ( int i = cursor_col - 1; i > 0; i-- ) {
     if ( tabs[ i ] && ++count == 0 ) {
       return i;
@@ -237,7 +237,6 @@ int DrawState::get_next_tab( int count ) const
   }
   return 0;
 }
-}
 
 void DrawState::set_scrolling_region( int top, int bottom )
 {
@@ -597,7 +596,7 @@ std::string Cell::debug_contents( void ) const
 {
   if ( contents.empty() ) {
     return "'_' ()";
-  } else {
+  }
   std::string chars( 1, '\'' );
   print_grapheme( chars );
   chars.append( "' [" );
@@ -614,7 +613,6 @@ std::string Cell::debug_contents( void ) const
   chars.append( "]" );
   return chars;
 }
-}
 
 bool Cell::compare( const Cell &other ) const
 {

src/terminal/terminaluserinput.cc

@@ -52,17 +52,14 @@ string UserInput::input( const Parser::UserByte *act,
       state = ESC;
     }
     return string( &act->c, 1 );
-    break;
 
   case ESC:
     if ( act->c == 'O' ) { /* ESC O = 7-bit SS3 */
       state = SS3;
       return string();
-    } else {
+    }
     state = Ground;
     return string( &act->c, 1 );
-    }
-    break;
 
   case SS3:
     state = Ground;
@@ -75,12 +72,11 @@ string UserInput::input( const Parser::UserByte *act,
       char original_cursor[ 2 ] = { 'O', act->c };
       return string( original_cursor, 2 );
     }
-    break;
-  }
 
+  default:
     /* This doesn't handle the 8-bit SS3 C1 control, which would be
        two octets in UTF-8. Fortunately nobody seems to send this. */
-
     assert( false );
     return string();
   }
+}

src/util/locale_utils.cc

@@ -51,9 +51,8 @@ const string LocaleVar::str( void ) const
 {
   if ( name.empty() ) {
     return string( "[no charset variables]" );
-  } else {
-    return name + "=" + value;
   }
+  return name + "=" + value;
 }
 
 const LocaleVar get_ctype( void )
@@ -65,9 +64,8 @@ const LocaleVar get_ctype( void )
     return LocaleVar( "LC_CTYPE", ctype );
   } else if ( const char *lang = getenv( "LANG" ) ) {
     return LocaleVar( "LANG", lang );
-  } else {
-    return LocaleVar( "", "" );
   }
+  return LocaleVar( "", "" );
 }
 
 const char *locale_charset( void )

src/util/swrite.cc

@@ -46,9 +46,8 @@ int swrite( int fd, const char *str, ssize_t len )
     if ( bytes_written <= 0 ) {
       perror( "write" );
       return -1;
-    } else {
-      total_bytes_written += bytes_written;
     }
+    total_bytes_written += bytes_written;
   }
 
   return 0;