performance: let's do no add compactValue attr if it is the same as regular value

Author: os97673

lib/ruby-debug-ide/xml_printer.rb

@@ -171,10 +171,9 @@ def print_variable(name, value, kind)
         end
       end
       value_str = handle_binary_data(value_str)
-      compact_value_str = build_compact_name(value_str, value)
       escaped_value_str = CGI.escapeHTML(value_str)
-      print("<variable name=\"%s\" compactValue=\"%s\" kind=\"%s\" value=\"%s\" type=\"%s\" hasChildren=\"%s\" objectId=\"%#+x\">",
-          CGI.escapeHTML(name), CGI.escapeHTML(compact_value_str), kind, escaped_value_str, value.class,
+      print("<variable name=\"%s\" %s kind=\"%s\" value=\"%s\" type=\"%s\" hasChildren=\"%s\" objectId=\"%#+x\">",
+          CGI.escapeHTML(name), build_compact_value_attr(value), kind, escaped_value_str, value.class,
           has_children, value.respond_to?(:object_id) ? value.object_id : value.id)
       print("<value><![CDATA[%s]]></value>", escaped_value_str)
       print('</variable>')
@@ -342,10 +341,10 @@ def current_thread_attr(context)
       end
     end
 
-    def build_compact_name(value_str, value)
+    def build_compact_name(value)
       return compact_array_str(value) if value.is_a?(Array)
       return compact_hash_str(value) if value.is_a?(Hash)
-      value_str
+      nil
     end
 
     def compact_array_str(value)
@@ -363,6 +362,11 @@ def compact_hash_str(value)
       "{" + compact + (slice.size != value.size ? ", ..." : "") + "}"
     end
 
+    def build_compact_value_attr(value)
+      compact_value_str  = build_compact_name(value)
+      compact_value_str.nil? ? '' : "compactValue=\"#{CGI.escapeHTML(compact_value_str)}\""
+    end
+
     instance_methods.each do |m|
       if m.to_s.index('print_') == 0
         protect m