On Thu, 29 Sep 2022 13:40:37 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:
>> This fixes misleading indentations, which allows enabling the (currently >> disabled) `misleading-indentation` warning flag on two `.gmk` files. > > src/java.base/share/native/libfdlibm/e_asin.c line 102: > >> 100: } else >> 101: t = x*x; >> 102: p = t*(pS0+t*(pS1+t*(pS2+t*(pS3+t*(pS4+t*pS5))))); > > should we add an opening brace after the `else` (and close it before line > 102) to make it more obvious that there is no problem with the indentation? It seems that this code is almost a verbatim copy of the upstream code available on the indicated link. So, while I agree that it would be better to add braces, it would make the code diverge from the upstream source, even if the change would be benign. and welcome ------------- PR: https://git.openjdk.org/jdk/pull/10487