Remove unnecessary cast to_code

These were casting expression that already had type codet.

Author: romainbrenguier

jbmc/src/java_bytecode/replace_java_nondet.cpp

@@ -92,7 +92,7 @@ get_nondet_instruction_info(const goto_programt::const_targett &instr)
   {
     return nondet_instruction_infot();
   }
-  const auto &code = to_code(instr->code);
+  const auto &code = instr->code;
   if(code.get_statement() != ID_function_call)
   {
     return nondet_instruction_infot();

src/analyses/flow_insensitive_analysis.cpp

@@ -49,8 +49,7 @@ exprt flow_insensitive_abstract_domain_baset::get_return_lhs(locationt to) const
   // must be the function call
   assert(to->is_function_call());
 
-  const code_function_callt &code=
-    to_code_function_call(to_code(to->code));
+  const code_function_callt &code = to_code_function_call(to->code);
 
   return code.lhs();
 }
@@ -164,8 +163,7 @@ bool flow_insensitive_analysis_baset::visit(
     if(l->is_function_call())
     {
       // this is a big special case
-      const code_function_callt &code=
-        to_code_function_call(to_code(l->code));
+      const code_function_callt &code = to_code_function_call(l->code);
 
       changed=
         do_function_call_rec(
@@ -214,7 +212,7 @@ bool flow_insensitive_analysis_baset::do_function_call(
   if(!goto_function.body_available())
   {
     const code_function_callt &code =
-      to_code_function_call(to_code(l_call->code));
+      to_code_function_call(l_call->code);
 
     goto_programt temp;
 

src/ansi-c/c_typecheck_code.cpp

@@ -626,7 +626,7 @@ void c_typecheck_baset::typecheck_ifthenelse(code_ifthenelset &code)
 
   implicit_typecast_bool(cond);
 
-  if(to_code(code.then_case()).get_statement()==ID_decl_block)
+  if(code.then_case().get_statement()==ID_decl_block)
   {
     code_blockt code_block;
     code_block.add_source_location()=code.then_case().source_location();
@@ -635,11 +635,11 @@ void c_typecheck_baset::typecheck_ifthenelse(code_ifthenelset &code)
     code.then_case().swap(code_block);
   }
 
-  typecheck_code(to_code(code.then_case()));
+  typecheck_code(code.then_case());
 
   if(!code.else_case().is_nil())
   {
-    if(to_code(code.else_case()).get_statement()==ID_decl_block)
+    if(code.else_case().get_statement()==ID_decl_block)
     {
       code_blockt code_block;
       code_block.add_source_location()=code.else_case().source_location();
@@ -648,7 +648,7 @@ void c_typecheck_baset::typecheck_ifthenelse(code_ifthenelset &code)
       code.else_case().swap(code_block);
     }
 
-    typecheck_code(to_code(code.else_case()));
+    typecheck_code(code.else_case());
   }
 }
 

src/ansi-c/expr2c.cpp

@@ -2594,8 +2594,8 @@ std::string expr2ct::convert_code_ifthenelse(
   }
   else
     dest+=convert_code(
-        to_code(src.then_case()),
-        to_code(src.then_case()).get_statement()==ID_block ? indent : indent+2);
+        src.then_case(),
+        src.then_case().get_statement()==ID_block ? indent : indent+2);
   dest+="\n";
 
   if(!src.else_case().is_nil())
@@ -2604,8 +2604,8 @@ std::string expr2ct::convert_code_ifthenelse(
     dest+=indent_str(indent);
     dest+="else\n";
     dest+=convert_code(
-        to_code(src.else_case()),
-        to_code(src.else_case()).get_statement()==ID_block ? indent : indent+2);
+        src.else_case(),
+        src.else_case().get_statement()==ID_block ? indent : indent+2);
   }
 
   return dest;

src/goto-instrument/goto_program2code.cpp

@@ -1117,17 +1117,17 @@ goto_programt::const_targett goto_program2codet::convert_goto_if(
   if(has_else)
   {
     for(++target; target!=before_else; ++target)
-      target=convert_instruction(target, before_else, to_code(i.then_case()));
+      target=convert_instruction(target, before_else, i.then_case());
 
-    convert_labels(before_else, to_code(i.then_case()));
+    convert_labels(before_else, i.then_case());
 
     for(++target; target!=end_if; ++target)
-      target=convert_instruction(target, end_if, to_code(i.else_case()));
+      target=convert_instruction(target, end_if, i.else_case());
   }
   else
   {
     for(++target; target!=end_if; ++target)
-      target=convert_instruction(target, end_if, to_code(i.then_case()));
+      target=convert_instruction(target, end_if, i.then_case());
   }
 
   dest.move_to_operands(i);
@@ -1355,11 +1355,11 @@ goto_programt::const_targett goto_program2codet::convert_start_thread(
   // use pthreads if "code in new thread" is a function call to a function with
   // suitable signature
   if(thread_start->is_function_call() &&
-     to_code_function_call(to_code(thread_start->code)).arguments().size()==1 &&
+     to_code_function_call(thread_start->code).arguments().size()==1 &&
      after_thread_start==thread_end)
   {
     const code_function_callt &cf=
-      to_code_function_call(to_code(thread_start->code));
+      to_code_function_call(thread_start->code);
 
     system_headers.insert("pthread.h");
 
@@ -1731,13 +1731,13 @@ void goto_program2codet::cleanup_code_ifthenelse(
   // assert(false) expands to if(true) assert(false), simplify again (and also
   // simplify other cases)
   if(cond.is_true() &&
-      (i_t_e.else_case().is_nil() || !has_labels(to_code(i_t_e.else_case()))))
+      (i_t_e.else_case().is_nil() || !has_labels(i_t_e.else_case())))
   {
     codet tmp;
     tmp.swap(i_t_e.then_case());
     code.swap(tmp);
   }
-  else if(cond.is_false() && !has_labels(to_code(i_t_e.then_case())))
+  else if(cond.is_false() && !has_labels(i_t_e.then_case()))
   {
     if(i_t_e.else_case().is_nil())
       code=code_skipt();
@@ -1751,7 +1751,7 @@ void goto_program2codet::cleanup_code_ifthenelse(
   else
   {
     if(i_t_e.then_case().is_not_nil() &&
-       to_code(i_t_e.then_case()).get_statement()==ID_ifthenelse)
+       i_t_e.then_case().get_statement()==ID_ifthenelse)
     {
       // we re-introduce 1-code blocks with if-then-else to avoid dangling-else
       // ambiguity
@@ -1761,8 +1761,8 @@ void goto_program2codet::cleanup_code_ifthenelse(
     }
 
     if(i_t_e.else_case().is_not_nil() &&
-       to_code(i_t_e.then_case()).get_statement()==ID_skip &&
-       to_code(i_t_e.else_case()).get_statement()==ID_ifthenelse)
+       i_t_e.then_case().get_statement()==ID_skip &&
+       i_t_e.else_case().get_statement()==ID_ifthenelse)
     {
       // we re-introduce 1-code blocks with if-then-else to avoid dangling-else
       // ambiguity
@@ -1796,7 +1796,7 @@ void goto_program2codet::cleanup_code_ifthenelse(
 
   // remove empty then/else
   if(code.get_statement()==ID_ifthenelse &&
-      to_code(i_t_e.then_case()).get_statement()==ID_skip)
+      i_t_e.then_case().get_statement()==ID_skip)
   {
     not_exprt tmp(i_t_e.cond());
     simplify(tmp, ns);
@@ -1807,13 +1807,13 @@ void goto_program2codet::cleanup_code_ifthenelse(
   }
   if(code.get_statement()==ID_ifthenelse &&
       i_t_e.else_case().is_not_nil() &&
-      to_code(i_t_e.else_case()).get_statement()==ID_skip)
+      i_t_e.else_case().get_statement()==ID_skip)
     i_t_e.else_case().make_nil();
   // or even remove the if altogether if the then case is now empty
   if(code.get_statement()==ID_ifthenelse &&
       i_t_e.else_case().is_nil() &&
       (i_t_e.then_case().is_nil() ||
-       to_code(i_t_e.then_case()).get_statement()==ID_skip))
+       i_t_e.then_case().get_statement()==ID_skip))
     code=code_skipt();
 }
 

src/goto-programs/goto_convert.cpp

@@ -1569,12 +1569,12 @@ void goto_convertt::convert_ifthenelse(
 
   // convert 'then'-branch
   goto_programt tmp_then;
-  convert(to_code(code.then_case()), tmp_then, mode);
+  convert(code.then_case(), tmp_then, mode);
 
   goto_programt tmp_else;
 
   if(has_else)
-    convert(to_code(code.else_case()), tmp_else, mode);
+    convert(code.else_case(), tmp_else, mode);
 
   exprt tmp_guard=code.cond();
   clean_expr(tmp_guard, dest, mode);

src/goto-programs/remove_unused_functions.cpp

@@ -75,8 +75,7 @@ void find_used_functions(
       {
         if(it->type==FUNCTION_CALL)
         {
-          const code_function_callt &call =
-            to_code_function_call(to_code(it->code));
+          const code_function_callt &call = to_code_function_call(it->code);
 
           // check that this is actually a simple call
           assert(call.function().id()==ID_symbol);

src/goto-symex/memory_model_tso.cpp

@@ -107,7 +107,7 @@ void memory_model_tsot::program_order(
 
         if((*e_it2)->is_memory_barrier())
         {
-          const codet &code=to_code((*e_it2)->source.pc->code);
+          const codet &code = (*e_it2)->source.pc->code;
 
           if((*e_it)->is_shared_read() &&
              !code.get_bool(ID_RRfence) &&

src/goto-symex/symex_dead.cpp

@@ -21,7 +21,7 @@ void goto_symext::symex_dead(statet &state)
 {
   const goto_programt::instructiont &instruction=*state.source.pc;
 
-  const codet &code=to_code(instruction.code);
+  const codet &code = instruction.code;
 
   if(code.operands().size()!=1)
     throw "dead expects one operand";

src/goto-symex/symex_decl.cpp

@@ -23,7 +23,7 @@ void goto_symext::symex_decl(statet &state)
 {
   const goto_programt::instructiont &instruction=*state.source.pc;
 
-  const codet &code=to_code(instruction.code);
+  const codet &code = instruction.code;
 
   if(code.operands().size()==2)
     throw "two-operand decl not supported here";