[skip-ci] Minor Markdown and typo fixes

Fixes a few typo and Markdown list numbering patterns in
`CODING_STANDARDS.md`, `CONTRIBUTING.md`, and `UPGRADING` files

Author: Ayesh

CODING_STANDARDS.md

@@ -9,11 +9,11 @@ rewritten to comply with these rules.
 
 1. Document your code in source files and the manual. (tm)
 
-1. PHP is implemented in C99.  The optional fixed-width integers from
+2. PHP is implemented in C99.  The optional fixed-width integers from
     stdint.h (int8_t, int16_t, int32_t, int64_t and their unsigned
     counterparts) must be available.
 
-1. Functions that are given pointers to resources should not free them.
+3. Functions that are given pointers to resources should not free them.
 
     For instance, `function int mail(char *to, char *from)` should NOT free `to`
     and/or `from`.
@@ -30,42 +30,42 @@ rewritten to comply with these rules.
     * Low-level parser routines, that are tightly integrated with the token
       cache and the bison code for minimum memory copying overhead.
 
-1. Functions that are tightly integrated with other functions within the same
+4. Functions that are tightly integrated with other functions within the same
     module, and rely on each other's non-trivial behavior, should be documented as
     such and declared `static`. They should be avoided if possible.
 
-1. Use definitions and macros whenever possible, so that constants have
+5. Use definitions and macros whenever possible, so that constants have
     meaningful names and can be easily manipulated. Any use of a numeric
     constant to specify different behavior or actions should be done through
     a `#define`.
 
-1. When writing functions that deal with strings, be sure to remember that PHP
+6. When writing functions that deal with strings, be sure to remember that PHP
     holds the length property of each string, and that it shouldn't be
     calculated with `strlen()`. Write your functions in such a way so that
     they'll take advantage of the length property, both for efficiency and in
     order for them to be binary-safe. Functions that change strings and obtain
     their new lengths while doing so, should return that new length, so it
     doesn't have to be recalculated with `strlen()` (e.g. `php_addslashes()`).
 
-1. NEVER USE `strncat()`. If you're absolutely sure you know what you're doing,
+7. NEVER USE `strncat()`. If you're absolutely sure you know what you're doing,
     check its man page again, and only then, consider using it, and even then,
     try avoiding it.
 
-1. Use `PHP_*` macros in the PHP source, and `ZEND_*` macros in the Zend part of
+8. Use `PHP_*` macros in the PHP source, and `ZEND_*` macros in the Zend part of
     the source. Although the `PHP_*` macros are mostly aliased to the `ZEND_*`
     macros it gives a better understanding on what kind of macro you're calling.
 
-1. When commenting out code using a `#if` statement, do NOT use `0` only.
+9. When commenting out code using a `#if` statement, do NOT use `0` only.
     Instead, use `"<git username here>_0"`. For example, `#if FOO_0`,
     where `FOO` is your git user `foo`. This allows easier tracking of why
     code was commented out, especially in bundled libraries.
 
-1. Do not define functions that are not available. For instance, if a library is
+10. Do not define functions that are not available. For instance, if a library is
     missing a function, do not define the PHP version of the function, and do
     not raise a run-time error about the function not existing. End users should
     use `function_exists()` to test for the existence of a function.
 
-1. Prefer `emalloc()`, `efree()`, `estrdup()`, etc. to their standard C library
+11. Prefer `emalloc()`, `efree()`, `estrdup()`, etc. to their standard C library
     counterparts. These functions implement an internal "safety-net" mechanism
     that ensures the deallocation of any unfreed memory at the end of a request.
     They also provide useful allocation and overflow information while running
@@ -78,7 +78,7 @@ rewritten to comply with these rules.
     may need to control or free the memory, or when the memory in question needs
     to survive between multiple requests.
 
-1. The return type of "is" or "has" style functions should be `bool`,
+12. The return type of "is" or "has" style functions should be `bool`,
     which return a "yes"/"no" answer.  `zend_result` is an appropriate
     return value for functions that perform some operation that may
     succeed or fail.
@@ -115,7 +115,7 @@ rewritten to comply with these rules.
     jf_n_s_i
     ```
 
-1. If they are part of a "parent set" of functions, that parent should be
+2. If they are part of a "parent set" of functions, that parent should be
     included in the user function name, and should be clearly related to the
     parent program or function family. This should be in the form of `parent_*`:
 
@@ -137,19 +137,19 @@ rewritten to comply with these rules.
     delete_foo_baz
     ```
 
-1. Function names used by user functions should be prefixed with `_php_`, and
+3. Function names used by user functions should be prefixed with `_php_`, and
     followed by a word or an underscore-delimited list of words, in lowercase
     letters, that describes the function. If applicable, they should be declared
     `static`.
 
-1. Variable names must be meaningful. One letter variable names must be avoided,
+4. Variable names must be meaningful. One letter variable names must be avoided,
     except for places where the variable has no real meaning or a trivial
     meaning (e.g. `for (i=0; i<100; i++) ...`).
 
-1. Variable names should be in lowercase. Use underscores to separate between
+5. Variable names should be in lowercase. Use underscores to separate between
     words.
 
-1. Method names follow the *studlyCaps* (also referred to as *bumpy case* or
+6. Method names follow the *studlyCaps* (also referred to as *bumpy case* or
     *camel caps*) naming convention, with care taken to minimize the letter
     count. The initial letter of the name is lowercase, and each letter that
     starts a new `word` is capitalized:
@@ -170,7 +170,7 @@ rewritten to comply with these rules.
     getI()
     ```
 
-1. Class names should be descriptive nouns in *PascalCase* and as short as
+7. Class names should be descriptive nouns in *PascalCase* and as short as
     possible. Each word in the class name should start with a capital letter,
     without underscore delimiters. The class name should be prefixed with the
     name of the "parent set" (e.g. the name of the extension) if no namespaces
@@ -228,9 +228,9 @@ rewritten to comply with these rules.
     static int php_session_destroy()
     ```
 
-1. Main module source file must be named `modulename.c`.
+2. Main module source file must be named `modulename.c`.
 
-1. Header file that is used by other sources must be named `php_modulename.h`.
+3. Header file that is used by other sources must be named `php_modulename.h`.
 
 ## Syntax and indentation
 
@@ -241,7 +241,7 @@ rewritten to comply with these rules.
     indentation and comment styles and up to function declaration syntax. Also
     see [Indentstyle](http://www.catb.org/~esr/jargon/html/I/indent-style.html).
 
-1. Be generous with whitespace and braces. Keep one empty line between the
+2. Be generous with whitespace and braces. Keep one empty line between the
     variable declaration section and the statements in a block, as well as
     between logical statement groups in a block. Maintain at least one empty
     line between two functions, preferably two. Always prefer:
@@ -258,15 +258,15 @@ rewritten to comply with these rules.
     if(foo)bar;
     ```
 
-1. When indenting, use the tab character. A tab is expected to represent four
+3. When indenting, use the tab character. A tab is expected to represent four
     spaces. It is important to maintain consistency in indentation so that
     definitions, comments, and control structures line up correctly.
 
-1. Preprocessor statements (`#if` and such) MUST start at column one. To indent
+4. Preprocessor statements (`#if` and such) MUST start at column one. To indent
     preprocessor directives you should put the `#` at the beginning of a line,
     followed by any number of spaces.
 
-1. The length of constant string literals should be calculated via ``strlen()``
+5. The length of constant string literals should be calculated via ``strlen()``
    instead of using ``sizeof()-1`` as it is clearer and any modern compiler
    will optimize it away. Legacy usages of the latter style exists within the
    codebase but should not be refactored, unless larger refactoring around that

CONTRIBUTING.md

@@ -75,8 +75,8 @@ accompanied by [pull requests](#pull-requests). You can find the extremely large
 list of RFCs that have been previously considered on the
 [PHP Uncyclo](https://wiki.php.net/rfc).
 
-To create a RFC, discuss it with the extension maintainer, and discuss it on the
-development mailing list [email protected]. RFC Uncyclo accounts can be
+To create an RFC, discuss it with the extension maintainer, and discuss it on
+the development mailing list [email protected]. RFC Uncyclo accounts can be
 requested on https://wiki.php.net/start?do=register. PHP extension maintainers
 can be found in the [EXTENSIONS](/EXTENSIONS) file in the PHP source code
 repository. Mailing list subscription is explained on the
@@ -318,7 +318,7 @@ detailed [information on Git](https://git-scm.com/).
 
 PHP is developed through the efforts of a large number of people. Collaboration
 is a Good Thing(tm), and Git lets us do this. Thus, following some basic rules
-with regards to Git usage will:
+with regard to Git usage will:
 
 * Make everybody happier, especially those responsible for maintaining PHP
   itself.
@@ -348,7 +348,7 @@ Having said that, here are the organizational rules:
    `--enable-zts` switch to ensure your code handles TSRM correctly and doesn't
    break for those who need that.
 
-Currently we have the following branches in use:
+Currently, we have the following branches in use:
 
 | Branch    |           |
 | --------- | --------- |

UPGRADING

@@ -252,7 +252,7 @@ PHP 8.3 UPGRADE NOTES
     now returns true on success, previously null was returned.
   . IntlBreakiterator::setText() now returns false on failure, previously
     null was returned.
-    now returns true on sucess, previously null was returned.
+    now returns true on success, previously null was returned.
   . IntlChar::enumCharNames is now returning a boolean.
     Previously it returned null on success and false on failure.
 
@@ -443,7 +443,7 @@ PHP 8.3 UPGRADE NOTES
     A warning is emitted when trying to decrement values of type null, as
     this will change in the next major version.
     Internal objects that implement an _IS_NUMBER cast but not a do_operator
-    handler that overrides addition and substraction now can be incremented
+    handler that overrides addition and subtraction now can be incremented
     and decrement as if one would do $o += 1 or $o -= 1
 
 - DOM: