From 7291c7aacb0754c97046121f7aa5fd4accf0c076 Mon Sep 17 00:00:00 2001
From: jbrockmendel <jbrockmendel@gmail.com>
Date: Thu, 20 Feb 2020 18:22:06 -0800
Subject: [PATCH] REF: standardize CategoricalIndex._shallow_copy usage

---
 pandas/core/indexes/category.py | 20 ++++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/pandas/core/indexes/category.py b/pandas/core/indexes/category.py
index adb2ed9211bfe..caa6a9a93141f 100644
--- a/pandas/core/indexes/category.py
+++ b/pandas/core/indexes/category.py
@@ -264,10 +264,14 @@ def _simple_new(cls, values, name=None, dtype=None):
     # --------------------------------------------------------------------
 
     @Appender(Index._shallow_copy.__doc__)
-    def _shallow_copy(self, values=None, dtype=None, **kwargs):
-        if dtype is None:
-            dtype = self.dtype
-        return super()._shallow_copy(values=values, dtype=dtype, **kwargs)
+    def _shallow_copy(self, values=None, **kwargs):
+        if values is None:
+            values = self.values
+
+        cat = Categorical(values, dtype=self.dtype)
+
+        name = kwargs.get("name", self.name)
+        return type(self)._simple_new(cat, name=name)
 
     def _is_dtype_compat(self, other) -> bool:
         """
@@ -422,9 +426,9 @@ def unique(self, level=None):
         if level is not None:
             self._validate_index_level(level)
         result = self.values.unique()
-        # CategoricalIndex._shallow_copy keeps original dtype
-        # if not otherwise specified
-        return self._shallow_copy(result, dtype=result.dtype)
+        # Use _simple_new instead of _shallow_copy to ensure we keep dtype
+        #  of result, not self.
+        return type(self)._simple_new(result, name=self.name)
 
     @Appender(Index.duplicated.__doc__)
     def duplicated(self, keep="first"):
@@ -450,7 +454,7 @@ def where(self, cond, other=None):
             other = self._na_value
         values = np.where(cond, self.values, other)
         cat = Categorical(values, dtype=self.dtype)
-        return self._shallow_copy(cat, **self._get_attributes_dict())
+        return self._shallow_copy(cat)
 
     def reindex(self, target, method=None, level=None, limit=None, tolerance=None):
         """