Unless I'm missing something, pkg_str == NULL iff pkg_name == NULL, and since NULLness is all is_shared_class_visible_for_classloader ever cares about it seems wasteful to always allocate the C string version of the pkg_name symbol.