On Sat, 8 Jul 2023 02:05:27 GMT, Jenny Shivayogi <d...@openjdk.org> wrote:
> Free-ing 'buf' before two conditional return statements introduced by > JDK-8307990 Hi @kspeeyu , Thanks for fixing this issue. The fix looks good except the whitespace issue already mentioned before. Notice that we don't use TABs for code indentation in OpenJDK. For the native Hotspot sources (i.e. the ones under `src/hotspot/`) we use an indentation of two spaces whereas for Java code and the native part of the class library (i.e. the native code outside `src/hotspot`) we use four spaces. We don't change the indentation of imported libraries like HarfBuzz or zlib at all. You can find more details on the coding style under https://github.com/openjdk/jdk/tree/master/doc. Just in case you haven't seen it yet, the [OpenJDK Developers’ Guide](https://openjdk.org/guide/) also contains a lot of useful information. ------------- PR Comment: https://git.openjdk.org/jdk/pull/14809#issuecomment-1629328688
