Change attribute keys to reduce conflicts with @attr uses

src/PrettyPrinting.jl

@@ -1407,8 +1407,8 @@ If `override` is `false`, the name is only set if there is no name already set.
 This function errors if `obj` does not support attribute storage.
 """
 function set_name!(obj, name::String; override::Bool=true)
-  override || isnothing(get_attribute(obj, :name)) || return
-  set_attribute!(obj, :name => name)
+  override || isnothing(get_attribute(obj, :_name)) || return
+  set_attribute!(obj, :_name => name)
 end
 
 """
@@ -1422,7 +1422,7 @@ If `override` is `false`, the name is only set if there is no name already set.
 This function errors if `obj` does not support attribute storage.
 """
 function set_name!(obj; override::Bool=true)
-  override || isnothing(get_attribute(obj, :name)) || return
+  override || isnothing(get_attribute(obj, :_name)) || return
   sy = find_name(obj)
   isnothing(sy) && return
   set_name!(obj, string(sy); override=true)
@@ -1457,7 +1457,7 @@ For this to work in doctests, one should call
 function find_name(obj, M=Main; all::Bool=false)
   AbstractAlgebra._is_attribute_storing_type(typeof(obj)) || return find_new_name(obj, M; all)
 
-  cached_name = get_attribute(obj, :cached_name)
+  cached_name = get_attribute(obj, :_cached_name)
   if !isnothing(cached_name)
    cached_name_sy = Symbol(cached_name)
     if M === Main && get_current_module() != Main
@@ -1470,7 +1470,7 @@ function find_name(obj, M=Main; all::Bool=false)
     end
   end
   name = find_new_name(obj, M; all)
-  set_attribute!(obj, :cached_name => name)
+  set_attribute!(obj, :_cached_name => name)
   return name
 end
 
@@ -1506,7 +1506,7 @@ This function tries to find a name in the following order:
 """
 function get_name(obj)
   if AbstractAlgebra._is_attribute_storing_type(typeof(obj))
-    name = get_attribute(obj, :name)
+    name = get_attribute(obj, :_name)
     isnothing(name) || return name
   end