gh-107162: Remove errcode.h header file

Rename Include/errcode.h to Include/internal/pycore_errcode.h.

Author: vstinner

Doc/whatsnew/3.13.rst

@@ -981,6 +981,11 @@ Removed
   be used instead.
   (Contributed by Serhiy Storchaka in :gh:`86493`.)
 
+* Remove the ``errcode.h`` header file. There was never any public tokenizer C
+  API. The ``errcode.h`` header file was only designed to be used by Python
+  internals.
+  (Contributed by Victor Stinner in :gh:`107162`.)
+
 Pending Removal in Python 3.14
 ------------------------------
 

Include/errcode.h renamed to Include/internal/pycore_errcode.h

@@ -1,10 +1,3 @@
-#ifndef Py_ERRCODE_H
-#define Py_ERRCODE_H
-#ifdef __cplusplus
-extern "C" {
-#endif
-
-
 /* Error codes passed around between file input, tokenizer, parser and
    interpreter.  This is necessary so we can turn them into Python
    exceptions at a higher level.  Note that some errors have a
@@ -13,6 +6,16 @@ extern "C" {
    the parser only returns E_EOF when it hits EOF immediately, and it
    never returns E_OK. */
 
+#ifndef Py_INTERNAL_ERRCODE_H
+#define Py_INTERNAL_ERRCODE_H
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+#ifndef Py_BUILD_CORE
+#  error "this header requires Py_BUILD_CORE define"
+#endif
+
 #define E_OK            10      /* No error */
 #define E_EOF           11      /* End Of File */
 #define E_INTR          12      /* Interrupted */
@@ -35,4 +38,4 @@ extern "C" {
 #ifdef __cplusplus
 }
 #endif
-#endif /* !Py_ERRCODE_H */
+#endif  // !Py_INTERNAL_ERRCODE_H

Makefile.pre.in

@@ -1625,7 +1625,6 @@ PYTHON_HEADERS= \
 		$(srcdir)/Include/dictobject.h \
 		$(srcdir)/Include/dynamic_annotations.h \
 		$(srcdir)/Include/enumobject.h \
-		$(srcdir)/Include/errcode.h \
 		$(srcdir)/Include/fileobject.h \
 		$(srcdir)/Include/fileutils.h \
 		$(srcdir)/Include/floatobject.h \
@@ -1750,10 +1749,11 @@ PYTHON_HEADERS= \
 		$(srcdir)/Include/internal/pycore_complexobject.h \
 		$(srcdir)/Include/internal/pycore_condvar.h \
 		$(srcdir)/Include/internal/pycore_context.h \
+		$(srcdir)/Include/internal/pycore_descrobject.h \
 		$(srcdir)/Include/internal/pycore_dict.h \
 		$(srcdir)/Include/internal/pycore_dict_state.h \
-		$(srcdir)/Include/internal/pycore_descrobject.h \
 		$(srcdir)/Include/internal/pycore_dtoa.h \
+		$(srcdir)/Include/internal/pycore_errcode.h \
 		$(srcdir)/Include/internal/pycore_exceptions.h \
 		$(srcdir)/Include/internal/pycore_faulthandler.h \
 		$(srcdir)/Include/internal/pycore_fileutils.h \

Misc/NEWS.d/next/C API/2023-07-24-00-35-20.gh-issue-107162.7YCEp7.rst

@@ -0,0 +1,3 @@
+Remove the ``errcode.h`` header file. There was never any public tokenizer C
+API. The ``errcode.h`` header file was only designed to be used by Python
+internals. Patch by Victor Stinner.

PCbuild/pythoncore.vcxproj

@@ -188,7 +188,6 @@
     <ClInclude Include="..\Include\dictobject.h" />
     <ClInclude Include="..\Include\dynamic_annotations.h" />
     <ClInclude Include="..\Include\enumobject.h" />
-    <ClInclude Include="..\Include\errcode.h" />
     <ClInclude Include="..\Include\fileobject.h" />
     <ClInclude Include="..\Include\fileutils.h" />
     <ClInclude Include="..\Include\floatobject.h" />
@@ -218,6 +217,7 @@
     <ClInclude Include="..\Include\internal\pycore_dict.h" />
     <ClInclude Include="..\Include\internal\pycore_dict_state.h" />
     <ClInclude Include="..\Include\internal\pycore_dtoa.h" />
+    <ClInclude Include="..\Include\internal\pycore_errcode.h" />
     <ClInclude Include="..\Include\internal\pycore_exceptions.h" />
     <ClInclude Include="..\Include\internal\pycore_faulthandler.h" />
     <ClInclude Include="..\Include\internal\pycore_fileutils.h" />

PCbuild/pythoncore.vcxproj.filters

@@ -81,9 +81,6 @@
     <ClInclude Include="..\Include\enumobject.h">
       <Filter>Include</Filter>
     </ClInclude>
-    <ClInclude Include="..\Include\errcode.h">
-      <Filter>Include</Filter>
-    </ClInclude>
     <ClInclude Include="..\Include\fileobject.h">
       <Filter>Include</Filter>
     </ClInclude>
@@ -558,6 +555,9 @@
     <ClInclude Include="..\Include\internal\pycore_dtoa.h">
       <Filter>Include\internal</Filter>
     </ClInclude>
+    <ClInclude Include="..\Include\internal\pycore_errcode.h">
+      <Filter>Include\internal</Filter>
+    </ClInclude>
     <ClInclude Include="..\Include\internal\pycore_exceptions.h">
       <Filter>Include\internal</Filter>
     </ClInclude>

Parser/pegen.c

@@ -1,7 +1,7 @@
 #include <Python.h>
 #include "pycore_ast.h"           // _PyAST_Validate(),
+#include "pycore_errcode.h"       // E_BADSINGLE
 #include "pycore_pystate.h"       // _PyThreadState_GET()
-#include <errcode.h>
 
 #include "tokenizer.h"
 #include "pegen.h"

Parser/pegen_errors.c

@@ -1,6 +1,5 @@
 #include <Python.h>
-#include <errcode.h>
-
+#include "pycore_errcode.h"       // E_TOKEN
 #include "pycore_pyerrors.h"      // _PyErr_ProgramDecodedTextObject()
 #include "tokenizer.h"
 #include "pegen.h"

Parser/tokenizer.c

@@ -3,12 +3,12 @@
 
 #include "Python.h"
 #include "pycore_call.h"          // _PyObject_CallNoArgs()
+#include "pycore_errcode.h"       // E_OK
 
 #include <ctype.h>
 #include <assert.h>
 
 #include "tokenizer.h"
-#include "errcode.h"
 
 /* Alternate tab spacing */
 #define ALTTABSIZE 1

Python/Python-tokenize.c

@@ -1,5 +1,5 @@
 #include "Python.h"
-#include "errcode.h"
+#include "pycore_errcode.h"       // E_TOKEN
 #include "../Parser/tokenizer.h"
 #include "../Parser/pegen.h"      // _PyPegen_byte_offset_to_character_offset()
 #include "../Parser/pegen.h"      // _PyPegen_byte_offset_to_character_offset()