BUG: Fixed passing keep_default_na=False when na_values=None (GH4318)

Author: jreback

doc/source/release.rst

@@ -82,6 +82,7 @@ pandas 0.13
     local variable was undefined (:issue:`4381`)
   - In ``to_json``, raise if a passed ``orient`` would cause loss of data because
     of a duplicate index (:issue:`4359`)
+  - Fixed passing ``keep_default_na=False`` when ``na_values=None`` (:issue:`4318`)
 
 pandas 0.12
 ===========

doc/source/v0.13.0.txt

@@ -66,6 +66,8 @@ Bug Fixes
 
   - Suppressed DeprecationWarning associated with internal calls issued by repr() (:issue:`4391`)
 
+  - Fixed passing ``keep_default_na=False`` when ``na_values=None`` (:issue:`4318`)
+
 See the :ref:`full release notes
 <release>` or issue tracker
 on GitHub for a complete list.

pandas/io/parsers.py

@@ -1774,8 +1774,11 @@ def _try_convert_dates(parser, colspec, data_dict, columns):
 
 def _clean_na_values(na_values, keep_default_na=True):
 
-    if na_values is None and keep_default_na:
-        na_values = _NA_VALUES
+    if na_values is None:
+        if keep_default_na:
+            na_values = _NA_VALUES
+        else:
+            na_values = []
         na_fvalues = set()
     elif isinstance(na_values, dict):
         if keep_default_na:

pandas/io/tests/test_parsers.py

@@ -108,6 +108,27 @@ def test_empty_string(self):
                                   np.nan, 'seven']})
         tm.assert_frame_equal(xp.reindex(columns=df.columns), df)
 
+
+        # GH4318, passing na_values=None and keep_default_na=False yields 'None' as a na_value
+        data = """\
+One,Two,Three
+a,1,None
+b,2,two
+,3,None
+d,4,nan
+e,5,five
+nan,6,
+g,7,seven
+"""
+        df = self.read_csv(
+            StringIO(data), keep_default_na=False)
+        xp = DataFrame({'One': ['a', 'b', '', 'd', 'e', 'nan', 'g'],
+                        'Two': [1, 2, 3, 4, 5, 6, 7],
+                        'Three': ['None', 'two', 'None', 'nan', 'five', '',
+                                  'seven']})
+        tm.assert_frame_equal(xp.reindex(columns=df.columns), df)
+
+
     def test_read_csv(self):
         if not compat.PY3:
             if 'win' in sys.platform: