Remove redundant pointer to array association

This redundant step which is already done in process_parameters, was
detected in string constraint generation as an attempt to access two
array to the same pointers.
Although the array are in fact the same here, the second call to
add_pointer_to_array_association is unecessary.

src/java_bytecode/java_string_library_preprocess.cpp

@@ -1696,14 +1696,6 @@ codet java_string_library_preprocesst::make_init_from_array_code(
   /// \todo this assumes the array to be constant between all calls to
   /// string primitives, which may not be true in general.
   refined_string_exprt string_arg = to_string_expr(args[1]);
-  add_pointer_to_array_association(
-    string_arg.content(),
-    dereference_exprt(
-      string_arg.content(),
-      array_typet(java_char_type(), infinity_exprt(java_int_type()))),
-    symbol_table,
-    loc,
-    code);
 
   // The third argument is `count`, whereas the third argument of substring
   // is `end` which corresponds to `offset+count`