On Thu, 29 Sep 2022 13:44:40 GMT, Raffaello Giulietti <rgiulie...@openjdk.org> wrote:
>> 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 OK - that's a reasonable concern indeed. ------------- PR: https://git.openjdk.org/jdk/pull/10487