8143911: java/lang/StackWalker tests fail on Solaris with IllegalStateException

[Coleen Phillimore]

This looks good to me also.
Thanks,
Coleen

On 11/24/15 6:03 PM, serguei.spitsyn at oracle.com wrote:
> Looks good.
>
> Thanks,
> Serguei
>
>
> On 11/24/15 14:37, Mandy Chung wrote:
>>> On Nov 24, 2015, at 2:20 PM, Daniel D. Daugherty 
>>> <daniel.daugherty at oracle.com> wrote:
>>>
>>> You use both 'this.anchor' and 'anchor'. Seems inconsistent.
>>
>> Oh yeah.  I took out “this.” from it.
>>
>> diff --git 
>> a/src/java.base/share/classes/java/lang/StackStreamFactory.java 
>> b/src/java.base/share/classes/java/lang/StackStreamFactory.java
>> --- a/src/java.base/share/classes/java/lang/StackStreamFactory.java
>> +++ b/src/java.base/share/classes/java/lang/StackStreamFactory.java
>> @@ -225,17 +225,17 @@
>>               }
>>               switch (state) {
>>                   case NEW:
>> -                    if (this.anchor != 0) {
>> +                    if (anchor != 0) {
>>                           throw new IllegalStateException("This stack 
>> stream is being reused.");
>>                       }
>>                       break;
>>                   case OPEN:
>> -                    if (this.anchor <= 0) {
>> -                        throw new IllegalStateException("This stack 
>> stream is not valid for walking");
>> +                    if (anchor == 0 || anchor == -1L) {
>> +                        throw new IllegalStateException("This stack 
>> stream is not valid for walking: " + anchor);
>>                       }
>>                       break;
>>                   case CLOSED:
>> -                    if (this.anchor != -1L) {
>> +                    if (anchor != -1L) {
>>                           throw new IllegalStateException("This stack 
>> stream is not closed.");
>>                       }
>>               }
>>
>> Mandy
>