Merge pull request #77 from phalcon/fix/coverage

Minor fixes

Author: sergeyklay

.editorconfig

@@ -2,22 +2,17 @@
 
 # top-most EditorConfig file
 root = true
-charset = utf-8
-trim_trailing_whitespace = true
 
 [*]
 end_of_line = lf
 insert_final_newline = true
 indent_style = tab
 indent_size = 4
+charset = utf-8
+trim_trailing_whitespace = true
 
-[*.{php,phpt,xml,json,w32}]
+[*.{yml,m4,sh,md,php,phpt,xml,json,w32}]
 indent_style = space
 
-[*.{yml,m4}]
+[*.{yml,m4,sh}]
 indent_size = 2
-indent_style = space
-
-[*.md]
-indent_style = space
-trim_trailing_whitespace = false

.gitignore

@@ -8,6 +8,8 @@
 *.o
 *.lo
 *.la
+*.profraw
+
 .deps
 .libs
 acinclude.m4

.travis.yml

@@ -62,10 +62,7 @@ install:
         --with-php-config=$(phpenv which php-config) \
         --enable-zephir-parser \
         --enable-zephir-parser-debug \
-        --enable-coverage \
-        LDFLAGS="${LDFLAGS} --coverage" \
-        CFLAGS="${CFLAGS} -O0 -ggdb -fprofile-arcs -ftest-coverage" \
-        CXXFLAGS="${CXXFLAGS} -O0 -ggdb -fprofile-arcs -ftest-coverage"
+        --enable-coverage
   - make $MAKEJOBS
 
 before_script:

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ## [Unreleased]
 ### Fixed
 - Fixed underscore only identifiers to allow more than 4 characters
+- Update LDFLAGS to reduce linker warnings
 
 ## [1.3.1] - 2019-05-01
 ### Fixed

CONTRIBUTING.md

@@ -58,5 +58,5 @@ Zephir Team
 
 [pr]: https://help.github.com/articles/using-pull-requests/
 [forum]: https://forum.zephir-lang.com
-[gb]: https://docs.phalconphp.com/en/latest/generating-backtrace
-[support]: https://phalconphp.com/support
+[gb]: https://docs.phalcon.io/en/latest/generating-backtrace
+[support]: https://phalcon.io/support

config.m4

@@ -110,11 +110,16 @@ You can disable ccache by setting environment variable CCACHE_DISABLE=1.
   CFLAGS=`echo "$CFLAGS" | $SED -e 's/-O[0-9s]*//g'`
   CXXFLAGS=`echo "$CXXFLAGS" | $SED -e 's/-O[0-9s]*//g'`
   dnl Remove --coverage flag from LDFLAGS
-  LDFLAGS=`echo "$LDFLAGS" | $SED -e 's/--coverage//g'`
+  LDFLAGS=`echo "$LDFLAGS" | $SED -e 's/--coverage)//g' -e 's/-fprofile-arcs//g' -e 's/-ftest-coverage//g'`
   changequote([,])
 
   dnl Add the special flags
-  LDFLAGS="$LDFLAGS --coverage"
+  if test "$($CC --version | head -n 1 | cut -d' ' -f1)" = "Apple"; then
+    LDFLAGS="$LDFLAGS -fprofile-arcs -ftest-coverage"
+  else
+    LDFLAGS="$LDFLAGS -coverage"
+  fi
+
   CFLAGS="$CFLAGS -O0 -ggdb -fprofile-arcs -ftest-coverage"
   CXXFLAGS="$CXXFLAGS -O0 -ggdb -fprofile-arcs -ftest-coverage"
 

tests/comments/dockblocks/simple.phpt

@@ -11,7 +11,7 @@ namespace Example;
 /**
  * DocBlockFail
  *
- * @author Paul Scarrone <[email protected]>
+ * @author Paul Scarrone <[email protected]>
  */
 class DocBlockTest {
 }
@@ -25,5 +25,5 @@ echo $ir[1]["value"];
 **
  * DocBlockFail
  *
- * @author Paul Scarrone <[email protected]>
+ * @author Paul Scarrone <[email protected]>
  *