RubyModule this-escape is necessary and harmless

Author: eregon

src/main/java/org/truffleruby/core/module/ModuleFields.java

@@ -135,7 +135,7 @@ public ModuleFields(
             SourceSection sourceSection,
             RubyModule lexicalParent,
             String givenBaseName,
-            RubyModule rubyModule) {
+            RubyModule rubyModule /* not fully initialized yet, should not access any field of it */) {
         super(null);
         this.language = language;
         this.sourceSection = sourceSection;

src/main/java/org/truffleruby/core/module/RubyModule.java

@@ -48,6 +48,7 @@ public class RubyModule extends RubyDynamicObject implements ObjectGraphNode {
     @DynamicField private Object object3;
     @DynamicField private Object object4;
 
+    @SuppressWarnings("this-escape")
     public RubyModule(
             RubyClass rubyClass,
             Shape shape,
@@ -59,6 +60,7 @@ public RubyModule(
         this.fields = new ModuleFields(language, sourceSection, lexicalParent, givenBaseName, this);
     }
 
+    @SuppressWarnings("this-escape")
     protected RubyModule(RubyLanguage language, Shape classShape, String constructorOnlyForClassClass) {
         super(classShape, constructorOnlyForClassClass);
         this.fields = new ModuleFields(language, null, null, "Class", this);