summaryrefslogtreecommitdiff
path: root/thirdparty/assimp/code/PostProcessing/OptimizeGraph.cpp
diff options
context:
space:
mode:
authorRyan Roden-Corrent <ryan@rcorre.net>2023-03-02 07:34:28 -0500
committerYuri Sizov <yuris@humnom.net>2023-03-13 21:18:53 +0100
commit1421838ba79bd14a978831672b28efe0b5fff866 (patch)
treec639a8e94fe274359cab74ed33abcfcba227f342 /thirdparty/assimp/code/PostProcessing/OptimizeGraph.cpp
parent30e81fcc26ee4e9394abe2504c6dd613b06a85db (diff)
Don't strip whitespace when converting 3to4.
Fixes #74204. The style guide says > Always use one space around operators and after commas The 3to4 conversion tool currently strips space in certain scenarios. I've updated it to add space whenever it is generating new code. In any case where it substitutes existing code, it leaves it as-is. For example, connect(a,b,c) becomes `connect(a, callable(b, c))`, because the converter is adding new commads/parens. However, `xform(Vector3(a,b,c))` becomes `Transform * Vector3(a,b,c)` because it uses the user's original Vector3 string whole. If the user originally had `xform(Vector3(a, b, c))`, then it becomes `Transform * Vector3(a, b, c)`. Ideally we'd always preserve original formatting, but this seems quite difficult, so I tried to preserve it where we can, but air on the side of following the style guide whenever we're transforming code. (cherry picked from commit d3684e662fdf2611214efc721ea2276364ef9fe1)
Diffstat (limited to 'thirdparty/assimp/code/PostProcessing/OptimizeGraph.cpp')
0 files changed, 0 insertions, 0 deletions