[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: Re: [OpenJDK 2D-Dev] [9] RFR JDK-8152561: Is it allowed to have zero value for count in TIFFField.cr
From: Brian Burkhalter <brian.burkhalter () oracle ! com>
Date: 2017-01-25 15:33:22
Message-ID: E031BFF5-3CBF-4EE3-B579-7E2E31842C4B () oracle ! com
[Download RAW message or body]
Hi Jay,
+1
Thanks,
Brian
On Jan 24, 2017, at 11:26 PM, Jayathirth D V <jayathirth.d.v@oracle.com> wrote:
> Thanks for your review.
> FYI, I have updated the webrev to include your suggestions :
> http://cr.openjdk.java.net/~jdv/8152561/webrev.02/
>
[Attachment #3 (unknown)]
<html><head><meta http-equiv="Content-Type" content="text/html \
charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: \
space; -webkit-line-break: after-white-space;">Hi \
Jay,<div><br></div><div>+1</div><div><br></div><div>Thanks,</div><div><br></div><div>Brian</div><div><br><div><div>On \
Jan 24, 2017, at 11:26 PM, Jayathirth D V <<a \
href="mailto:jayathirth.d.v@oracle.com">jayathirth.d.v@oracle.com</a>> \
wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div \
style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', \
serif; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: \
normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; \
text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; \
-webkit-text-stroke-width: 0px; background-color: rgb(255, 255, 255);"><span \
style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, \
125);">Thanks for your review.<o:p></o:p></span></div><div style="margin: 0in 0in \
0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; font-style: normal; \
font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: \
normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; \
white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; \
background-color: rgb(255, 255, 255);"><span style="font-size: 11pt; font-family: \
Calibri, sans-serif; color: rgb(31, 73, 125);">FYI, I have updated the webrev to \
include your suggestions :<o:p></o:p></span></div><div style="margin: 0in 0in \
0.0001pt; font-size: 12pt; font-family: 'Times New Roman', serif; font-style: normal; \
font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: \
normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; \
white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; \
background-color: rgb(255, 255, 255);"><span style="font-size: 11pt; font-family: \
Calibri, sans-serif; color: rgb(31, 73, 125);"><a \
href="http://cr.openjdk.java.net/~jdv/8152561/webrev.02/" style="color: purple; \
text-decoration: underline;">http://cr.openjdk.java.net/~jdv/8152561/webrev.02/</a><o:p></o:p></span></div><div \
style="margin: 0in 0in 0.0001pt; font-size: 12pt; font-family: 'Times New Roman', \
serif; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: \
normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; \
text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; \
-webkit-text-stroke-width: 0px; background-color: rgb(255, 255, 255);"><span \
style="font-size: 11pt; font-family: Calibri, sans-serif; color: rgb(31, 73, \
125);"> </span></div></blockquote></div><br></div></body></html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic