I'd remove this comment as there aren't any provisions to guarantee it is thread safe.  For example, multiple threads could see different values for fields that are cloned (there's no synchronization to ensure a consistent state across threads).  While true that this object should be a singleton, I think documenting this method as thread-safe could be misleading.