Wrap return value in std::move

This fixes diffblue#3238 (again).

Author: karkhaz

jbmc/src/java_bytecode/java_string_library_preprocess.cpp

@@ -1171,7 +1171,8 @@ codet java_string_library_preprocesst::make_assign_function_from_call(
 /// \return An expression contaning a symbol `tmp_type_name` where `type_name`
 ///         is the given argument (ie. boolean, char etc.). Which represents the
 ///         primitive value contained in the given object.
-exprt java_string_library_preprocesst::get_primitive_value_of_object(
+optionalt<symbol_exprt>
+java_string_library_preprocesst::get_primitive_value_of_object(
   const exprt &object,
   irep_idt type_name,
   const source_locationt &loc,
@@ -1222,7 +1223,7 @@ exprt java_string_library_preprocesst::get_primitive_value_of_object(
     object_type=symbol_typet("java::java.lang.Double");
   }
   else if(type_name==ID_void)
-    return nil_exprt();
+    return optionalt<symbol_exprt>();
   else
     UNREACHABLE;
 
@@ -1251,14 +1252,14 @@ exprt java_string_library_preprocesst::get_primitive_value_of_object(
         typecast_exprt(object, pointer_type), pointer_type.subtype());
       member_exprt deref_value(deref, value_comp.get_name(), value_comp.type());
       code.add(code_assignt(value, deref_value), loc);
-      return value;
+      return optionalt<symbol_exprt>(value);
     }
   }
 
   warning() << object_type->get_identifier()
             << " not available to format function" << eom;
   code.add(code_declt(value), loc);
-  return value;
+  return optionalt<symbol_exprt>(value);
 }
 
 /// Helper for format function. Returns the expression:
@@ -1398,9 +1399,12 @@ exprt java_string_library_preprocesst::make_argument_for_format(
     }
     else if(name==ID_int || name==ID_float || name==ID_char || name==ID_boolean)
     {
-      exprt value=get_primitive_value_of_object(
+      optionalt<symbol_exprt> value = get_primitive_value_of_object(
         arg_i, name, loc, symbol_table, code_not_null);
-      code_not_null.add(code_assignt(field_expr, value), loc);
+      if(value.has_value())
+        code_not_null.add(code_assignt(field_expr, *value), loc);
+      else
+        code_not_null.add(code_assignt(field_expr, nil_exprt()), loc);
     }
     else
     {

jbmc/src/java_bytecode/java_string_library_preprocess.h

@@ -21,6 +21,7 @@ Date:   March 2017
 #include <util/string_expr.h>
 
 #include <util/ieee_float.h> // should get rid of this
+#include <util/optional.h>
 
 #include <array>
 #include <unordered_set>
@@ -324,7 +325,7 @@ class java_string_library_preprocesst:public messaget
     symbol_table_baset &symbol_table,
     code_blockt &code);
 
-  exprt get_primitive_value_of_object(
+  optionalt<symbol_exprt> get_primitive_value_of_object(
     const exprt &object,
     irep_idt type_name,
     const source_locationt &loc,

src/ansi-c/literals/convert_float_literal.cpp

@@ -101,5 +101,5 @@ exprt convert_float_literal(const std::string &src)
     return complex_exprt(zero_real_component, result, complex_type);
   }
 
-  return result;
+  return std::move(result);
 }