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