Sync leading and final newlines in source code files

This patch adds missing newlines, trims multiple redundant final
newlines into a single one, and trims redundant leading newlines.

According to POSIX, a line is a sequence of zero or more non-' <newline>'
characters plus a terminating '<newline>' character. [1] Files should
normally have at least one final newline character.

C89 [2] and later standards [3] mention a final newline:
"A source file that is not empty shall end in a new-line character,
which shall not be immediately preceded by a backslash character."

Although it is not mandatory for all files to have a final newline
fixed, a more consistent and homogeneous approach brings less of commit
differences issues and a better development experience in certain text
editors and IDEs.

[1] http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap03.html#tag_03_206
[2] https://port70.net/~nsz/c/c89/c89-draft.html#2.1.1.2
[3] https://port70.net/~nsz/c/c99/n1256.html#5.1.1.2

Author: petk

.appveyor.yml

@@ -1,4 +1,3 @@
-
 version: "{branch}.build.{build}"
 
 image: Visual Studio 2015
@@ -47,4 +46,3 @@ build_script:
 
 test_script:
         - appveyor\test.bat
-

Makefile.frag

@@ -1,4 +1,3 @@
-
 #
 # Zend
 #

Makefile.gcov

@@ -1,4 +1,3 @@
-
 #
 # LCOV
 #

README.EXT_SKEL

@@ -185,4 +185,3 @@ PHP_FUNCTION(module_name_drawtext)
     php_error(E_WARNING, "module_name_drawtext: not yet implemented");
 }
 /* }}} */
-

README.REDIST.BINS

@@ -414,4 +414,3 @@ OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
 WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR
 OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
 ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-

README.SELF-CONTAINED-EXTENSIONS

@@ -167,4 +167,3 @@ PECL SITE CONFORMITY
 
     This macros has to be used within your foo_module_entry to indicate the
     extension version.
-

README.TESTING2

@@ -132,6 +132,3 @@ regular expect sections.  For example:
 
   --EXPECTHEADERS--
   Status: 404
-
-
-

README.WIN32-BUILD-SYSTEM

@@ -3,4 +3,3 @@ The Win32 Build System.
 See http://wiki.php.net/internals/windows/stepbystepbuild
 
 vim:tw=78:sw=1:ts=1:et
-

README.md

@@ -171,4 +171,3 @@ TODO
   * Add getNamespace() methods to ReflectionClass and ReflectionFunction
 
 * Rename namespaces to packages?
-

README.namespaces

@@ -1,5 +1,3 @@
-
 AC_DEFUN([AM_SET_LIBTOOL_VARIABLE],[
   LIBTOOL='$(SHELL) $(top_builddir)/libtool $1'
 ])
-

TSRM/acinclude.m4

@@ -3,4 +3,3 @@
 
 ADD_SOURCES("TSRM", "TSRM.c tsrm_strtok_r.c tsrm_win32.c");
 ADD_FLAG("CFLAGS_BD_TSRM", "/D ZEND_ENABLE_STATIC_TSRMLS_CACHE=1");
-

TSRM/config.w32

@@ -194,4 +194,3 @@ esac
 AC_LANG_POP
 
 ]) dnl end AC_DEFUN
-

TSRM/m4/gethostbyname.m4

@@ -523,4 +523,3 @@ PHP 7.1 UPGRADE NOTES
 ========================================
 13. Other Changes
 ========================================
-

UPGRADING

@@ -88,4 +88,3 @@ changes. See: https://wiki.php.net/phpng-upgrading
 ========================
 3. Module changes
 ========================
-

UPGRADING.INTERNALS

@@ -53,4 +53,3 @@ depend:
 
 zend_execute.lo: $(srcdir)/zend_execute.c
 	$(LIBTOOL) --mode=compile $(CC) $(DEFS) $(DEFAULT_INCLUDES) $(INCLUDES) $(CPPFLAGS) $(INLINE_CFLAGS) -c $(srcdir)/zend_execute.c
-

Zend/Makefile.am

@@ -105,4 +105,3 @@ Zend Engine II currently includes two executors during the build process, one
 is the specialized version and the other is the old one non-specialized with
 function handlers. By default Zend Engine II uses the specialized one but you
 can switch to the old executor at runtime by calling zend_vm_use_old_executor().
-

Zend/README.ZEND_VM

@@ -133,4 +133,3 @@ The new object model is not dependent on other features.  Many of the
 other Zend Engine 2.0 features, such as the $foo->bar()->barbara()
 syntax, destructors and others completely rely on this new object
 model.
-

Zend/RFCs/001.txt

@@ -66,7 +66,3 @@ following could not reliably run.
 
 Old code that does not take advantage of this will run without
 modifications.
-
-
-
-

Zend/RFCs/003.txt

@@ -30,4 +30,3 @@ if cmp configure.old configure > /dev/null 2>&1; then
 else
     echo "buildconf: created or modified ${1}configure"
 fi
-

Zend/buildconf

@@ -7,4 +7,4 @@ function func5() { }
 function func6() { }
 function func7() { }
 print ("ok\n");
-?>
+?>

Zend/tests/bug40236.inc

@@ -1,4 +1,4 @@
 <?php
 function nested() {
   throw new LogicException('Should not be called');
-}
+}

Zend/tests/methods-on-non-objects-nested.inc

@@ -8,4 +8,3 @@ class d { public function __construct() { $this->d = 4; } };
 $d = new d;
 
 ?>
-

Zend/tests/nowdoc.inc

@@ -2,4 +2,3 @@
 
 class A {}
 class B extends A {}
-

Zend/tests/return_types/classes.php.inc

@@ -114,4 +114,3 @@ ZEND_API void dtrace_execute_internal(zend_execute_data *execute_data, zval *ret
 /* }}} */
 
 #endif /* HAVE_DTRACE */
-

Zend/zend_dtrace.c

@@ -240,4 +240,3 @@ ZEND_API void zend_strip(void)
  * indent-tabs-mode: t
  * End:
  */
-

Zend/zend_highlight.c

@@ -120,4 +120,3 @@ static zend_always_inline void smart_str_setl(smart_str *dest, const char *src,
 }
 
 #endif
-

Zend/zend_smart_str.h

@@ -145,4 +145,3 @@ typedef unsigned long int uint32_t;
 #endif
 
 #endif /* ZEND_STRTOD_INT_H */
-

Zend/zend_strtod_int.h

@@ -82,4 +82,3 @@ ZEND_API void zend_deserialize_opcode_handler(zend_op *op)
 {
 	op->handler = zend_opcode_handlers[(zend_uintptr_t)op->handler];
 }
-

Zend/zend_vm_execute.skl

@@ -42,4 +42,3 @@ cmd /c %SDK_RUNNER% -t %APPVEYOR_BUILD_FOLDER%\appveyor\build_task.bat
 if %errorlevel% neq 0 exit /b 3
 
 exit /b 0
-

appveyor/build.bat

@@ -57,4 +57,3 @@ nmake /NOLOGO
 if %errorlevel% neq 0 exit /b 3
 
 exit /b 0
-

appveyor/build_task.bat

@@ -10,4 +10,3 @@ cmd /c %SDK_RUNNER% -t %APPVEYOR_BUILD_FOLDER%\appveyor\test_task.bat
 if %errorlevel% neq 0 exit /b 3
 
 exit /b 0
-

appveyor/test.bat

@@ -86,4 +86,3 @@ cd "%APPVEYOR_BUILD_FOLDER%"
 nmake test TESTS="%OPCACHE_OPTS% -q --offline --show-diff --set-timeout 120 -g FAIL,XFAIL,BORK,WARN,LEAK,SKIP --temp-source c:\tests_tmp --temp-target c:\tests_tmp"
 
 exit /b %errorlevel%
-

appveyor/test_task.bat

@@ -52,4 +52,3 @@ configure: aclocal.m4 configure.in $(config_m4_files)
 	@echo rebuilding $@
 	@rm -f $@
 	$(PHP_AUTOCONF) -f $(SUPPRESS_WARNINGS)
-

build/build2.mk

@@ -38,5 +38,3 @@ cat $infile | \
 	-e "s'@EXT_MODULE_PTRS@'$module_ptrs'" \
 	-e 's/@NEWLINE@/\
 /g'
-
-

build/genif.sh

@@ -4,4 +4,4 @@ SET PHP_BUILDCONF_PATH=%~dp0
 copy %PHP_BUILDCONF_PATH%\win32\build\configure.bat %PHP_BUILDCONF_PATH% > nul
 SET PHP_SDK_SCRIPT_PATH=
 
-IF NOT EXIST %PHP_BUILDCONF_PATH% (echo Error generating configure script, configure script was not copied) ELSE (echo Now run 'configure --help')
+IF NOT EXIST %PHP_BUILDCONF_PATH% (echo Error generating configure script, configure script was not copied) ELSE (echo Now run 'configure --help')

buildconf.bat

@@ -1,3 +1 @@
 Phil Nelson <[email protected]> wrote bcmath library.
-
-

ext/bcmath/libbcmath/AUTHORS

@@ -1,4 +1,3 @@
-
                   GNU LESSER GENERAL PUBLIC LICENSE
                        Version 2.1, February 1999
 
@@ -511,5 +510,3 @@ Random Hacker.
   Ty Coon, President of Vice
 
 That's all there is to it!
-
-

ext/bcmath/libbcmath/COPYING.LIB

@@ -7,4 +7,3 @@ Wed Jun  7 09:39:02 2000  Phil Nelson  <[email protected]>
 2000-05-21  Phil Nelson  <[email protected]>
 
 	* Initial setup of bcmath library., calling it version 0.1.
-

ext/bcmath/libbcmath/ChangeLog

@@ -18,4 +18,3 @@ calculate an exact number of digits after the decimal point regardless
 of the number of digits in the integer part.  GMP floats have a
 "fixed, but arbitrary" mantissa and so multiplying two floats will end
 up dropping digits BC must calculate.
-

ext/bcmath/libbcmath/FAQ

@@ -9,4 +9,3 @@ dist-hook:
 	cp $(srcdir)/doc/bcmath.1 $(distdir)/doc
 	cp $(srcdir)/src/private.h $(distdir)/src
 	cp $(srcdir)/FAQ $(distdir)
-

ext/bcmath/libbcmath/Makefile.am

@@ -6,4 +6,3 @@
 
 /* Define to `size_t' if <sys/types.h> and <stddef.h> don't define.  */
 #undef ptrdiff_t
-

ext/bcmath/libbcmath/acconfig.h

@@ -124,4 +124,3 @@ for am_file in <<$1>>; do
   am_indx=`expr "<<$>>am_indx" + 1`
 done<<>>dnl>>)
 changequote([,]))])
-

ext/bcmath/libbcmath/aclocal.m4

@@ -38,4 +38,3 @@
 
 /* Version number of package */
 #undef VERSION
-

ext/bcmath/libbcmath/config.h.in

@@ -1855,5 +1855,3 @@ EOF
 chmod +x $CONFIG_STATUS
 rm -fr confdefs* $ac_clean_files
 test "$no_create" = yes || ${CONFIG_SHELL-/bin/sh} $CONFIG_STATUS || exit 1
-
-

ext/bcmath/libbcmath/configure

@@ -15,4 +15,3 @@ AC_TYPE_SIZE_T
 AC_CHECK_TYPE(ptrdiff_t, size_t)
 
 AC_OUTPUT(Makefile src/Makefile doc/Makefile)
-

ext/bcmath/libbcmath/configure.in

@@ -85,4 +85,3 @@ bc_add (n1, n2, result, scale_min)
   bc_free_num (result);
   *result = sum;
 }
-

ext/bcmath/libbcmath/src/add.c

@@ -158,4 +158,3 @@ bc_compare (n1, n2)
 {
   return _bc_do_compare (n1, n2, TRUE, FALSE);
 }
-

ext/bcmath/libbcmath/src/compare.c

@@ -271,4 +271,3 @@ bc_divide (bc_num n1, bc_num n2, bc_num *quot, int scale)
 
   return 0;	/* Everything is OK. */
 }
-

ext/bcmath/libbcmath/src/div.c

@@ -84,4 +84,3 @@ bc_modulo (bc_num num1, bc_num num2, bc_num *result, int scale)
 {
   return bc_divmod (num1, num2, NULL, result, scale);
 }
-

ext/bcmath/libbcmath/src/divmod.c

@@ -229,4 +229,3 @@ _bc_do_sub (n1, n2, scale_min)
   _bc_rm_leading_zeros (diff);
   return diff;
 }
-

ext/bcmath/libbcmath/src/doaddsub.c

@@ -131,4 +131,3 @@ bc_init_num (bc_num *num)
 {
   *num = bc_copy_num (BCG(_zero_));
 }
-

ext/bcmath/libbcmath/src/init.c

@@ -81,4 +81,3 @@ bc_int2num (num, val)
   while (ix-- > 0)
     *vptr++ = *--bptr;
 }
-

ext/bcmath/libbcmath/src/int2num.c

@@ -66,4 +66,3 @@ bc_is_near_zero (num, scale)
   else
     return TRUE;
 }
-

ext/bcmath/libbcmath/src/nearzero.c

@@ -46,4 +46,3 @@ bc_is_neg (num)
 {
   return num->n_sign == MINUS;
 }
-

ext/bcmath/libbcmath/src/neg.c

@@ -67,4 +67,3 @@ bc_num2long (num)
   else
     return (-val);
 }
-

ext/bcmath/libbcmath/src/num2long.c

@@ -121,4 +121,3 @@ bc_raise (bc_num num1, bc_num num2, bc_num *result, int scale)
      }
    bc_free_num (&power);
 }
-

ext/bcmath/libbcmath/src/raise.c

@@ -52,4 +52,3 @@ _bc_rm_leading_zeros (num)
     num->n_len--;
   }
 }
-

ext/bcmath/libbcmath/src/rmzero.c

@@ -126,4 +126,3 @@ bc_sqrt (bc_num *num, int scale)
   bc_free_num (&diff);
   return 1;
 }
-

ext/bcmath/libbcmath/src/sqrt.c

@@ -109,4 +109,3 @@ bc_str2num (bc_num *num, char *str, int scale)
   if (bc_is_zero (*num))
     (*num)->n_sign = PLUS;
 }
-

ext/bcmath/libbcmath/src/str2num.c

@@ -87,4 +87,3 @@ bc_sub (n1, n2, result, scale_min)
   bc_free_num (result);
   *result = diff;
 }
-

ext/bcmath/libbcmath/src/sub.c

@@ -61,4 +61,3 @@ bc_is_zero (bc_num num)
   else
     return TRUE;
 }
-

ext/bcmath/libbcmath/src/zero.c

@@ -1,4 +1,3 @@
-
 /* $selId: dow.c,v 2.0 1995/10/24 01:13:06 lees Exp $
  * Copyright 1993-1995, Scott E. Lee, all rights reserved.
  * Permission granted to use, copy, modify, distribute and sell so long as

ext/calendar/dow.c

@@ -243,4 +243,3 @@ zend_object_iterator *php_com_iter_get(zend_class_entry *ce, zval *object, int b
 	}
 	return NULL;
 }
-

ext/com_dotnet/com_iterator.c

@@ -563,4 +563,3 @@ zend_object_iterator *php_com_saproxy_iter_get(zend_class_entry *ce, zval *objec
 
 	return &I->iter;
 }
-

ext/com_dotnet/com_saproxy.c

@@ -604,5 +604,3 @@ int php_com_process_typeinfo(ITypeInfo *typeinfo, HashTable *id_to_name, int pri
 
 	return ret;
 }
-
-

ext/com_dotnet/com_typeinfo.c

@@ -1124,4 +1124,3 @@ PHP_FUNCTION(variant_cast)
 	VariantClear(&vres);
 }
 /* }}} */
-

ext/com_dotnet/com_variant.c

@@ -645,5 +645,3 @@ PHP_COM_DOTNET_API IDispatch *php_com_wrapper_export(zval *val)
 
 	return (IDispatch*)disp;
 }
-
-

ext/com_dotnet/com_wrapper.c

@@ -242,4 +242,3 @@ sgml-local-catalogs:nil
 sgml-local-ecat-files:nil
 End:
 -->
-

ext/ctype/ctype.xml

@@ -83,4 +83,3 @@ function curl_cli_server_start() {
 
     return PHP_CURL_SERVER_ADDRESS;
 }
-

ext/curl/tests/server.inc

@@ -1,2 +1,2 @@
 DBA
-Sascha Schumann, Marcus Boerger
+Sascha Schumann, Marcus Boerger

ext/dba/CREDITS

@@ -51,4 +51,4 @@ This shows whether your combination of handlers works. Most problematic
 are dbm and ndbm which conflict with many installations. The reason for
 this is that on several systems these libraries are part of more than one
 other library. The configuration test only prevents you from configuring
-malfaunctioning single handlers but not combinations.
+malfaunctioning single handlers but not combinations.

ext/dba/README

@@ -95,4 +95,4 @@ do {
 	}
 } while(1);
 
-?>
+?>