On Fri, 20 Oct 2023 12:49:46 GMT, Johan Sjölen <jsjo...@openjdk.org> wrote:

>> I think that NMT is deserving of its own subdirectory. Can we do a review of 
>> the changes before I fix the merge conflicts?
>> 
>> 1. Moved all the nmt source code from services/ to nmt/
>> 2. Renamed all the include statements and sorted them
>> 3. Fixed the include guards
>
> Johan Sjölen has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Fix messed up include

Still good.

(Your merge commit seems to have confused GitHub for some reason.)

-------------

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16276#pullrequestreview-1691770241

Reply via email to