Refactor timezones functions out of tslib

pandas/_libs/index.pyx

@@ -24,6 +24,8 @@ from datetime import datetime, timedelta
 from datetime cimport (get_datetime64_value, _pydatetime_to_dts,
                        pandas_datetimestruct)
 
+from tslibs.timezones cimport _get_utcoffset, _is_utc
+
 from cpython cimport PyTuple_Check, PyList_Check
 
 cdef extern from "datetime.h":
@@ -554,14 +556,11 @@ cdef inline _to_i8(object val):
             # Save the original date value so we can get the utcoffset from it.
             ival = _pydatetime_to_dts(val, &dts)
             if tzinfo is not None and not _is_utc(tzinfo):
-                offset = tslib._get_utcoffset(tzinfo, val)
+                offset = _get_utcoffset(tzinfo, val)
                 ival -= tslib._delta_to_nanoseconds(offset)
             return ival
         return val
 
-cdef inline bint _is_utc(object tz):
-    return tz is UTC or isinstance(tz, _du_utc)
-
 
 cdef class MultiIndexObjectEngine(ObjectEngine):
     """

pandas/_libs/period.pyx

@@ -10,19 +10,18 @@ from numpy cimport (int8_t, int32_t, int64_t, import_array, ndarray,
                     NPY_INT64, NPY_DATETIME, NPY_TIMEDELTA)
 import numpy as np
 
-cdef extern from "datetime_helper.h":
-    double total_seconds(object)
-
 from libc.stdlib cimport free
 
 from pandas import compat
 from pandas.compat import PY2
 
 cimport cython
 
+cdef extern from "datetime.h":
+    void PyDateTime_IMPORT()
+
 from datetime cimport (
     is_leapyear,
-    PyDateTime_IMPORT,
     pandas_datetimestruct,
     pandas_datetimestruct_to_datetime,
     pandas_datetime_to_datetimestruct,
@@ -32,18 +31,19 @@ from datetime cimport (
 cimport util, lib
 from lib cimport is_null_datetimelike, is_period
 from pandas._libs import tslib, lib
-from pandas._libs.tslib import (Timedelta, Timestamp, iNaT,
-                                NaT, _get_utcoffset)
-from tslib cimport (
-    maybe_get_tz,
-    _is_utc,
-    _is_tzlocal,
+from pandas._libs.tslib import Timestamp, iNaT, NaT
+from tslib cimport _nat_scalar_rules
+
+from tslibs.timezones cimport (
+    total_seconds,
+    _get_utcoffset,
     _get_dst_info,
-    _nat_scalar_rules)
+    _is_tzlocal,
+    _is_utc,
+    maybe_get_tz)
 
-from pandas.tseries import offsets
 from pandas.core.tools.datetimes import parse_time_string
-from pandas.tseries import frequencies
+from pandas.tseries import frequencies, offsets
 
 cdef int64_t NPY_NAT = util.get_nat()
 
@@ -117,6 +117,7 @@ cdef extern from "period_helper.h":
 
 initialize_daytime_conversion_factor_matrix()
 
+
 # Period logic
 #----------------------------------------------------------------------
 
@@ -726,8 +727,7 @@ cdef class _Period(object):
         return hash((self.ordinal, self.freqstr))
 
     def _add_delta(self, other):
-        if isinstance(other, (timedelta, np.timedelta64,
-                              offsets.Tick, Timedelta)):
+        if isinstance(other, (timedelta, np.timedelta64, offsets.Tick)):
             offset = frequencies.to_offset(self.freq.rule_code)
             if isinstance(offset, offsets.Tick):
                 nanos = tslib._delta_to_nanoseconds(other)
@@ -751,9 +751,8 @@ cdef class _Period(object):
 
     def __add__(self, other):
         if isinstance(self, Period):
-            if isinstance(other, (timedelta, np.timedelta64,
-                                  offsets.Tick, offsets.DateOffset,
-                                  Timedelta)):
+            if isinstance(other,
+                          (timedelta, np.timedelta64, offsets.DateOffset)):
                 return self._add_delta(other)
             elif other is NaT:
                 return NaT
@@ -769,9 +768,8 @@ cdef class _Period(object):
 
     def __sub__(self, other):
         if isinstance(self, Period):
-            if isinstance(other, (timedelta, np.timedelta64,
-                                  offsets.Tick, offsets.DateOffset,
-                                  Timedelta)):
+            if isinstance(other,
+                          (timedelta, np.timedelta64, offsets.DateOffset)):
                 neg_other = -other
                 return self + neg_other
             elif lib.is_integer(other):

pandas/_libs/tslib.pxd

@@ -2,9 +2,5 @@ from numpy cimport ndarray, int64_t
 
 cdef convert_to_tsobject(object, object, object, bint, bint)
 cpdef convert_to_timedelta64(object, object)
-cpdef object maybe_get_tz(object)
-cdef bint _is_utc(object)
-cdef bint _is_tzlocal(object)
-cdef object _get_dst_info(object)
 cdef bint _nat_scalar_rules[6]
 cdef bint _check_all_nulls(obj)