<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    These changes you have look good for Method except a lot of the
    comment is out of date.&nbsp; The block comment about the layout of
    fields in Method is redundant with the actual declaration and
    wrong.&nbsp; It's a nice visual but I don't see the point of keeping
    redundant information like that.&nbsp; Also, it the comment describes
    some of the fields that were moved to ConstMethod years ago.&nbsp;&nbsp; I
    don't think you want to try to correct these things with this typo
    change, but we should (I could) file an RFE to correct them later.<br>
    <br>
    thanks,<br>
    Coleen<br>
    <br>
    <div class="moz-cite-prefix">On 01/14/2014 12:53 PM, Krystal Mok
      wrote:<br>
    </div>
    <blockquote
cite="mid:CA+cQ+tS+hytNx55hazNTPDHQ1qrnx=-QYGp_hCM=f84bCaZ2EQ@mail.gmail.com"
      type="cite">
      <div dir="ltr">Hi Jesper,
        <div><br>
        </div>
        <div>That looks fine to me. Thank you!</div>
        <div>It's still better if someone from the runtime team could
          verify the change in oops/method.hpp, and someone from the
          compiler team could verify the change in ci/ciField.hpp|cpp</div>
        <div><br>
        </div>
        <div>Thanks,</div>
        <div>Kris</div>
      </div>
      <div class="gmail_extra"><br>
        <br>
        <div class="gmail_quote">On Tue, Jan 14, 2014 at 9:38 PM, Jesper
          Wilhelmsson <span dir="ltr">&lt;<a moz-do-not-send="true"
              href="mailto:jesper.wilhelmsson@oracle.com"
              target="_blank">jesper.wilhelmsson@oracle.com</a>&gt;</span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">Sure, no
            problem. Webrev is updated, please verify.<br>
            I took the liberty of changing the layout of the comment in
            ciField.cpp. Since there was only one case left it didn't
            feel motivated to have a list of cases.<br>
            /Jesper<br>
            <br>
            Krystal Mok skrev 13/1/14 7:36 PM:<br>
            <blockquote class="gmail_quote" style="margin:0 0 0
              .8ex;border-left:1px #ccc solid;padding-left:1ex">
              <div class="im">
                Hi Jesper,<br>
                <br>
                Thanks! I took a look and the change was fine.<br>
                <br>
                I do have a couple of other comment fixes that I'd like
                to hitchhike:<br>
                <br>
                1. oops/method.hpp<br>
                <br>
                The comments say ConstMethod* and MethodData* are oops,
                which they aren't anymore.<br>
                I'm not sure if the wording for "putting oops and
                method_size first for better<br>
                gc cache locality" still matters. It probably doesn't
                matter anymore, since GC<br>
                doesn't have to mark through these fields with the
                PermGen removal.<br>
                <br>
                2. ci/ciField.hpp, ci/ciField.cpp<br>
                <br>
                The comments say in order to consider a field as a
                constant, it cannot hold an<br>
                non-perm-space oop. I believe this limitation has been
                removed. Could someone<br>
                from the compiler team verify if that's true?<br>
                <br>
                The diff is at the end of this mail.<br>
                <br>
                Thanks,<br>
                Kris<br>
                <br>
                diff -r 9d39e8a8ff61 src/share/vm/ci/ciField.cpp<br>
              </div>
              --- a/src/share/vm/ci/ciField.cppFri Dec 27 07:51:07 2013
              -0800<br>
              +++ b/src/share/vm/ci/ciField.cppMon Jan 13 10:31:17 2014
              -0800
              <div class="im"><br>
                @@ -202,13 +202,9 @@<br>
                &nbsp; &nbsp; &nbsp; }<br>
                &nbsp; &nbsp; &nbsp; // This field just may be constant. &nbsp;The only
                cases where it will<br>
                - &nbsp; &nbsp;// not be constant are:<br>
                + &nbsp; &nbsp;// not be constant is:<br>
                &nbsp; &nbsp; &nbsp; //<br>
                - &nbsp; &nbsp;// 1. The field holds a non-perm-space oop. &nbsp;The
                field is, strictly<br>
                - &nbsp; &nbsp;// &nbsp; &nbsp;speaking, constant but we cannot embed
                non-perm-space oops into<br>
                - &nbsp; &nbsp;// &nbsp; &nbsp;generated code. &nbsp;For the time being we need
                to consider the<br>
                - &nbsp; &nbsp;// &nbsp; &nbsp;field to be not constant.<br>
                - &nbsp; &nbsp;// 2. The field is a *special* static&amp;final
                field whose value<br>
                + &nbsp; &nbsp;// 1. The field is a *special* static&amp;final
                field whose value<br>
                &nbsp; &nbsp; &nbsp; // &nbsp; &nbsp;may change. &nbsp;The three examples are <a
                  moz-do-not-send="true"
                  href="http://java.lang.System.in" target="_blank">java.lang.System.in</a><br>
              </div>
              &lt;<a moz-do-not-send="true"
                href="http://java.lang.System.in" target="_blank">http://java.lang.System.in</a>&gt;,
              <div class="im"><br>
                &nbsp; &nbsp; &nbsp; // &nbsp; &nbsp;java.lang.System.out, and
                java.lang.System.err.<br>
                diff -r 9d39e8a8ff61 src/share/vm/ci/ciField.hpp<br>
              </div>
              --- a/src/share/vm/ci/ciField.hppFri Dec 27 07:51:07 2013
              -0800<br>
              +++ b/src/share/vm/ci/ciField.hppMon Jan 13 10:31:17 2014
              -0800
              <div class="im"><br>
                @@ -130,9 +130,7 @@<br>
                &nbsp; &nbsp; // &nbsp; 1. The field is both static and final<br>
                &nbsp; &nbsp; // &nbsp; 2. The canonical holder of the field has
                undergone<br>
                &nbsp; &nbsp; // &nbsp; &nbsp; &nbsp;static initialization.<br>
                - &nbsp;// &nbsp; 3. If the field is an object or array, then the
                oop<br>
                - &nbsp;// &nbsp; &nbsp; &nbsp;in question is allocated in perm space.<br>
                - &nbsp;// &nbsp; 4. The field is not one of the special
                static/final<br>
                + &nbsp;// &nbsp; 3. The field is not one of the special
                static/final<br>
                &nbsp; &nbsp; // &nbsp; &nbsp; &nbsp;non-constant fields. &nbsp;These are <a
                  moz-do-not-send="true"
                  href="http://java.lang.System.in" target="_blank">java.lang.System.in</a><br>
              </div>
              &lt;<a moz-do-not-send="true"
                href="http://java.lang.System.in" target="_blank">http://java.lang.System.in</a>&gt;
              <div class="im"><br>
                &nbsp; &nbsp; // &nbsp; &nbsp; &nbsp;and java.lang.System.out. &nbsp;Abomination.<br>
                &nbsp; &nbsp; //<br>
                diff -r 9d39e8a8ff61 src/share/vm/oops/method.hpp<br>
              </div>
              --- a/src/share/vm/oops/method.hppFri Dec 27 07:51:07 2013
              -0800<br>
              +++ b/src/share/vm/oops/method.hppMon Jan 13 10:31:17 2014
              -0800
              <div>
                <div class="h5"><br>
                  @@ -38,13 +38,11 @@<br>
                  &nbsp; #include "utilities/accessFlags.hpp"<br>
                  &nbsp; #include "utilities/growableArray.hpp"<br>
                  -// A Method* represents a Java method.<br>
                  +// A Method represents a Java method.<br>
                  &nbsp; //<br>
                  &nbsp; // Memory layout (each line represents a word). Note
                  that most applications<br>
                  load thousands of methods,<br>
                  &nbsp; // so keeping the size of this structure small has a
                  big impact on footprint.<br>
                  &nbsp; //<br>
                  -// We put all oops and method_size first for better
                  gc cache locality.<br>
                  -//<br>
                  &nbsp; // The actual bytecodes are inlined after the end of
                  the Method struct.<br>
                  &nbsp; //<br>
                  &nbsp; // There are bits in the access_flags telling
                  whether inlined tables are present.<br>
                  @@ -64,17 +62,17 @@<br>
                  &nbsp; // | header &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; &nbsp; |<br>
                  &nbsp; // | klass &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; &nbsp;|<br>
                  &nbsp; // |------------------------------------------------------|<br>
                  -// | ConstMethod* &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; (oop) &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; |<br>
                  +// | ConstMethod* &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; (metadata) &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; &nbsp;|<br>
                  &nbsp; // |------------------------------------------------------|<br>
                  -// | methodData &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; (oop) &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; |<br>
                  -// | methodCounters &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; |<br>
                  +// | MethodData* &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;(metadata) &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; &nbsp;|<br>
                  +// | MethodCounters &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; |<br>
                  &nbsp; // |------------------------------------------------------|<br>
                  &nbsp; // | access_flags &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; &nbsp; |<br>
                  &nbsp; // | vtable_index &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; &nbsp; |<br>
                  &nbsp; // |------------------------------------------------------|<br>
                  &nbsp; // | result_index (C++ interpreter only) &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; &nbsp;|<br>
                  &nbsp; // |------------------------------------------------------|<br>
                  -// | method_size &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; | &nbsp; intrinsic_id| &nbsp;
                  flags &nbsp; &nbsp;|<br>
                  +// | method_size &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; | intrinsic_id &nbsp;| &nbsp;
                  flags &nbsp; &nbsp;|<br>
                  &nbsp; // |------------------------------------------------------|<br>
                  &nbsp; // | code &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; (pointer) &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; &nbsp; |<br>
                  &nbsp; // | i2i &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;(pointer) &nbsp; &nbsp; &nbsp;
                  &nbsp; &nbsp; &nbsp; |<br>
                  <br>
                  <br>
                  <br>
                  <br>
                  On Mon, Jan 13, 2014 at 9:07 PM, Jesper Wilhelmsson<br>
                </div>
              </div>
              <div class="im">
                &lt;<a moz-do-not-send="true"
                  href="mailto:jesper.wilhelmsson@oracle.com"
                  target="_blank">jesper.wilhelmsson@oracle.com</a>
                &lt;mailto:<a moz-do-not-send="true"
                  href="mailto:jesper.wilhelmsson@oracle.com"
                  target="_blank">jesper.wilhelmsson@oracle.com</a>&gt;&gt;
                wrote:<br>
                <br>
                &nbsp; &nbsp; Hi Kris!<br>
                <br>
                &nbsp; &nbsp; No problem, I'll add it to the change. Please verify
                that it looks OK in the<br>
                &nbsp; &nbsp; updated webrev (same link as before).<br>
                <br>
              </div>
              &nbsp; &nbsp; <a moz-do-not-send="true"
                href="http://cr.openjdk.java.net/%7E__jwilhelm/8025856/webrev.1/"
                target="_blank">http://cr.openjdk.java.net/~__jwilhelm/8025856/webrev.1/</a>
              <div class="im"><br>
                &nbsp; &nbsp; &lt;<a moz-do-not-send="true"
                  href="http://cr.openjdk.java.net/%7Ejwilhelm/8025856/webrev.1/"
                  target="_blank">http://cr.openjdk.java.net/~jwilhelm/8025856/webrev.1/</a>&gt;<br>
                <br>
                &nbsp; &nbsp; /Jesper<br>
                <br>
                <br>
                &nbsp; &nbsp; Krystal Mok skrev 10/1/14 7:17 PM:<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; Hi Jesper,<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; Nice to see cleaner code!<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; Hitchhike: It'd be nice if you could include
                this typo in:<br>
              </div>
              &nbsp; &nbsp; &nbsp; &nbsp; <a moz-do-not-send="true"
href="http://mail.openjdk.java.net/__pipermail/hotspot-compiler-__dev/2014-January/012944.html"
                target="_blank">http://mail.openjdk.java.net/__pipermail/hotspot-compiler-__dev/2014-January/012944.html</a>
              <div class="im">
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &lt;<a moz-do-not-send="true"
href="http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-January/012944.html"
                  target="_blank">http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-January/012944.html</a>&gt;<br>
                &nbsp; &nbsp; &nbsp; &nbsp; I didn't really like sending one-liner comment
                typo fix anyway...with a<br>
                &nbsp; &nbsp; &nbsp; &nbsp; lot of<br>
                &nbsp; &nbsp; &nbsp; &nbsp; other typos, the story is different ;-)<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; Thanks,<br>
                &nbsp; &nbsp; &nbsp; &nbsp; Kris<br>
                <br>
                <br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; On Fri, Jan 10, 2014 at 11:54 PM, Coleen
                Phillmore<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &lt;<a moz-do-not-send="true"
                  href="mailto:coleen.phillimore@oracle.com"
                  target="_blank">coleen.phillimore@oracle.com</a>
                &lt;mailto:<a moz-do-not-send="true"
                  href="mailto:coleen.phillimore@oracle.com"
                  target="_blank">coleen.phillimore@oracle.com</a>&gt;<br>
              </div>
              &nbsp; &nbsp; &nbsp; &nbsp; &lt;mailto:<a moz-do-not-send="true"
                href="mailto:coleen.phillimore@" target="_blank">coleen.phillimore@</a>__<a
                moz-do-not-send="true" href="http://oracle.com"
                target="_blank">oracle.com</a>
              <div class="im"><br>
                &nbsp; &nbsp; &nbsp; &nbsp; &lt;mailto:<a moz-do-not-send="true"
                  href="mailto:coleen.phillimore@oracle.com"
                  target="_blank">coleen.phillimore@oracle.com</a>&gt;&gt;&gt;
                wrote:<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;Seems fine with me also. &nbsp;Could you find
                typos in the comments in the<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;runtime code "by accident" too? :)<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;thanks,<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;Coleen<br>
                <br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;On 1/10/2014 10:39 AM, Daniel D. Daugherty
                wrote:<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;On 1/10/14 5:49 AM, Jesper Wilhelmsson
                wrote:<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;Hi,<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;I have a change out for review that
                fixes a huge pile of<br>
                &nbsp; &nbsp; &nbsp; &nbsp; typos in<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;the comments in the GC code. The
                RFR was sent to the GC<br>
                &nbsp; &nbsp; &nbsp; &nbsp; list, but I<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;want to give a heads up in case
                anyone else is changing GC<br>
                &nbsp; &nbsp; &nbsp; &nbsp; code and<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;want to avoid merge conflicts.<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;The patch:<br>
              </div>
              &nbsp; &nbsp; &nbsp; &nbsp; <a moz-do-not-send="true"
                href="http://cr.openjdk.java.net/%7E____jwilhelm/8025856/webrev.1/"
                target="_blank">http://cr.openjdk.java.net/~____jwilhelm/8025856/webrev.1/</a><br>
              &nbsp; &nbsp; &nbsp; &nbsp; &lt;<a moz-do-not-send="true"
                href="http://cr.openjdk.java.net/%7E__jwilhelm/8025856/webrev.1/"
                target="_blank">http://cr.openjdk.java.net/~__jwilhelm/8025856/webrev.1/</a>&gt;
              <div class="im"><br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;&lt;<a moz-do-not-send="true"
                  href="http://cr.openjdk.java.net/%7E__jwilhelm/8025856/webrev.1/"
                  target="_blank">http://cr.openjdk.java.net/~__jwilhelm/8025856/webrev.1/</a><br>
                &nbsp; &nbsp; &nbsp; &nbsp; &lt;<a moz-do-not-send="true"
                  href="http://cr.openjdk.java.net/%7Ejwilhelm/8025856/webrev.1/"
                  target="_blank">http://cr.openjdk.java.net/~jwilhelm/8025856/webrev.1/</a>&gt;&gt;<br>
                <br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;There are also a few files where I
                happened to find a few<br>
                &nbsp; &nbsp; &nbsp; &nbsp; typos "by<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;accident" in code that is not
                strictly GC code. These are:<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;src/share/vm/memory/heap.cpp<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;src/share/vm/memory/heap.hpp<br>
              </div>
              &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;src/share/vm/memory/____allocation.hpp<br>
              &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;src/share/vm/memory/____resourceArea.hpp<br>
              &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;src/share/vm/runtime/thread.____cpp
              <div class="im"><br>
                <br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;There is a total of eight typos
                fixed in these files so I<br>
                &nbsp; &nbsp; &nbsp; &nbsp; think the<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;risk of merge conflicts here is
                minimal. Are there any<br>
                &nbsp; &nbsp; &nbsp; &nbsp; objections to<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;including these fixes in the
                change?<br>
                <br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;Vote: go for it!<br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;Dan<br>
                <br>
                <br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;Thanks,<br>
                &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;/Jesper<br>
                <br>
                <br>
                <br>
                <br>
                <br>
              </div>
            </blockquote>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
  </body>
</html>