godot.git/drivers/png/SCsub, branch master Godot Engine – Multi-platform 2D and 3D game engine https://godotengine.org https://git.neuromancer.sk/godot.git/atom/drivers/png/SCsub?h=master 2017-09-25T18:36:30Z Use BoolVariable for third-party options. 2017-09-25T18:36:30Z Elliott Sales de Andrade 2017-09-25T04:22:58Z urn:sha1:45a9a680a3cf54d4f43c46c3ec43a108ee62b834 scons: Reorder options for clarity 2016-11-03T06:45:08Z Rémi Verschelde 2016-11-02T23:23:55Z urn:sha1:cc95d4448c6005c3007c8460f09b8be1595eb3c0 Also prefix all thirdparty-related toggles with `builtin`. style: Various other PEP8 fixes in Python files 2016-10-31T23:35:16Z Rémi Verschelde 2016-10-31T23:24:30Z urn:sha1:f34151ff0f91e8f0df8eaf829334b2205eb7da3c Done with `autopep8 --select=E7`, fixes: - E701 - Put colon-separated compound statement on separate lines. - E702 - Put semicolon-separated compound statement on separate lines. - E703 - Put semicolon-separated compound statement on separate lines. - E711 - Fix comparison with None. - E712 - Fix (trivial case of) comparison with boolean. - E713 - Fix (trivial case of) non-membership check. - E721 - Fix various deprecated code (via lib2to3). style: Fix PEP8 whitespace issues in Python files 2016-10-31T23:35:16Z Rémi Verschelde 2016-10-30T17:57:40Z urn:sha1:d4c17700aa2f36f69978beda04e42ff2749de270 Done with `autopep8 --select=E2,W2`, fixes: - E201 - Remove extraneous whitespace. - E202 - Remove extraneous whitespace. - E203 - Remove extraneous whitespace. - E211 - Remove extraneous whitespace. - E221 - Fix extraneous whitespace around keywords. - E222 - Fix extraneous whitespace around keywords. - E223 - Fix extraneous whitespace around keywords. - E224 - Remove extraneous whitespace around operator. - E225 - Fix missing whitespace around operator. - E226 - Fix missing whitespace around operator. - E227 - Fix missing whitespace around operator. - E228 - Fix missing whitespace around operator. - E231 - Add missing whitespace. - E231 - Fix various deprecated code (via lib2to3). - E241 - Fix extraneous whitespace around keywords. - E242 - Remove extraneous whitespace around operator. - E251 - Remove whitespace around parameter '=' sign. - E261 - Fix spacing after comment hash. - E262 - Fix spacing after comment hash. - E265 - Format block comments. - E271 - Fix extraneous whitespace around keywords. - E272 - Fix extraneous whitespace around keywords. - E273 - Fix extraneous whitespace around keywords. - E274 - Fix extraneous whitespace around keywords. - W291 - Remove trailing whitespace. - W293 - Remove trailing whitespace. style: Start applying PEP8 to Python files, indentation issues 2016-10-31T23:33:51Z Rémi Verschelde 2016-10-30T17:44:57Z urn:sha1:97c8508f5e4f57b1048830d44e76e1f4517fd449 Done with `autopep8 --select=E1`, fixes: - E101 - Reindent all lines. - E112 - Fix under-indented comments. - E113 - Fix over-indented comments. - E115 - Fix under-indented comments. - E116 - Fix over-indented comments. - E121 - Fix a badly indented line. - E122 - Fix a badly indented line. - E123 - Fix a badly indented line. - E124 - Fix a badly indented line. - E125 - Fix indentation undistinguish from the next logical line. - E126 - Fix a badly indented line. - E127 - Fix a badly indented line. - E128 - Fix a badly indented line. - E129 - Fix a badly indented line. SCsub: Add python shebang as a hint for syntax highlighting 2016-10-17T18:10:46Z Rémi Verschelde 2016-10-17T06:50:25Z urn:sha1:fc8ccd5b8c8b779bffd0f4d7f22f2f964c939163 Also switch existing shebangs to "better" /usr/bin/env python. png: Try to fix neon issue on iphone armv7 2016-10-15T16:10:19Z Rémi Verschelde 2016-10-15T13:36:18Z urn:sha1:e57042e8a93e4f3d65cc91633f5af0daedf69a2a drivers: Refactor SCsub and drop redundant env_drivers clone 2016-10-15T16:10:18Z Rémi Verschelde 2016-10-15T10:39:28Z urn:sha1:248bc9159c0b515e3e276db644744161283881ef The reordering of the SConscript includes allows to ensure that stuff like the builtin zlib headers will be available for libpng. Also moved glew back into global env, otherwise windows seems not to find it... Kind of shooting in the dark with this multi-env setup. modules: Clone env in each module 2016-10-15T09:50:41Z Rémi Verschelde 2016-10-13T16:54:00Z urn:sha1:da09c6131bcdace7e8e62c3dabc62890e9564c97 This allows to pass include paths and flags only to a given thirdparty library, thus preventing conflicts between their files (e.g. between opus and openssl which both provide modes.h. This also has the nice effect of making the compilation command smaller for each module as it no longer related to all other modules, only the final linking brings them together. This however requires adding manually the ogg include path in opus and vorbis when building against the builtin ogg, since it is no longer in the global env. Also simplified template 'thirdparty_<module>_sources' to 'thirdparty_sources'. "Core" modules like cscript, gdscript, gridmap, ik and virtual_script still use the main env_modules, but it could be changed if need be. png: Split library to thirdparty dir and allow unbundling 2016-10-15T09:50:39Z Rémi Verschelde 2016-10-09T21:36:17Z urn:sha1:5fef84a1358310304cb1114924525ec4df794b49 Uses the new structure agreed upon in #6157, but the thirdparty/ folder does not behave following a logic similar to that of modules/ yet. The png driver can't be moved to a module as discussed in #6157, as it's required by core together with a few other ImageLoader implementations (see drivers/register_driver_types.cpp:register_core_driver_types()) Dropped the possibility to disable PNG support, it's a core component of Godot.