RFR: JDK-8173664: Typo in https://java.net/downloads/heap-snapshot/hprof-binary-format.html

[Fairoz Matte]

Hi David,

After having discussion with you over IM things are clear
Expected change is - "There will be a "LOAD CLASS" tag for the type of each array in the dump" and current changeset missed "the" before type.

I will make the changes and send it again.

Thanks,
Fairoz

> -----Original Message-----
> From: David Holmes
> Sent: Thursday, May 11, 2017 7:52 AM
> To: Fairoz Matte <fairoz.matte at oracle.com>; serviceability-
> dev at openjdk.java.net
> Subject: Re: RFR: JDK-8173664: Typo in https://java.net/downloads/heap-
> snapshot/hprof-binary-format.html
> 
> On 10/05/2017 9:30 PM, Fairoz Matte wrote:
> > Hi David,
> >
> >> -----Original Message-----
> >> From: David Holmes
> >> Sent: Wednesday, May 10, 2017 3:26 PM
> >> To: Fairoz Matte <fairoz.matte at oracle.com>; serviceability-
> >> dev at openjdk.java.net
> >> Subject: Re: RFR: JDK-8173664: Typo in
> >> https://java.net/downloads/heap- snapshot/hprof-binary-format.html
> >>
> >> Hi Fairoz,
> >>
> >> On 10/05/2017 5:53 PM, Fairoz Matte wrote:
> >>> Hi,
> >>>
> >>> Kindly review the small typo fix, applicable only for JDK8
> >>> BugID: https://bugs.openjdk.java.net/browse/JDK-8173664
> >>> Webrev: http://cr.openjdk.java.net/~rpatil/8173664/webrev/
> >>
> >> I think "for type type" was intended to be "for the type".
> >>
> > Yes it does look like after reading multiple times.
> > Thanks for the review I will close as Not an issue
> 
> ??? It is still a typo that can be fixed.
> 
> David
> 
> > Thanks,
> > Fairoz
> >
> >> David
> >>
> >>> Thanks,
> >>> Fairoz
> >>>