[Github-comments] [geany/geany-plugins] Remove -Werror=aggregate-return from travis flags (#529)

Jiří Techet notifications at xxxxx
Sat Feb 18 13:16:22 UTC 2017


While in general one should avoid returning big structs for performance
reasons, it's no big problem for small structs and it can lead to more
readable code.

Having this check on for everything seems to be too strict - if someone
is returning big structs like crazy, someone will probably notice during
the review. Otherwise it's no problem.
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany-plugins/pull/529

-- Commit Summary --

  * Remove -Werror=aggregate-return from travis flags

-- File Changes --

    M .travis.yml (2)

-- Patch Links --

https://github.com/geany/geany-plugins/pull/529.patch
https://github.com/geany/geany-plugins/pull/529.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/529
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20170218/580bc60b/attachment.html>


More information about the Github-comments mailing list