[jira] [Commented] (GROOVY-7592) Problem in switch statement docs

2015-09-22 Thread Pascal Schumacher (JIRA) 

[ 
https://issues.apache.org/jira/browse/GROOVY-7592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14902963#comment-14902963
 ] 

Pascal Schumacher commented on GROOVY-7592:
---

I changed two sections: 
[https://github.com/apache/incubator-groovy/commit/5d4f7b46edbe0d5332645af623fe3f8e2cf0f75c]
 if you find more that need to be changed let me know or send a pull request.

> Problem in switch statement docs
> 
>
> Key: GROOVY-7592
> URL: https://issues.apache.org/jira/browse/GROOVY-7592
> Project: Groovy
>  Issue Type: Documentation
>  Components: Documentation
>Affects Versions: 2.4.5
>Reporter: Aseem Bansal
>Assignee: Pascal Schumacher
>Priority: Minor
> Fix For: 2.4.6
>
>
> As per http://www.groovy-lang.org/semantics.html default must be the last 
> thing in switch case. 
> Based on that I sent a PR which has been accepted
> https://github.com/apache/incubator-groovy/pull/82
> But when I tried to use default somewhere else it worked fine
> {noformat}
> String str = "aseem"
> switch(str) {
> default:
> println "default"
> break
> case "aseem":
> println "Aseem"
> break
> } 
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (GROOVY-7592) Problem in switch statement docs

2015-09-21 Thread Aseem Bansal (JIRA) 

[ 
https://issues.apache.org/jira/browse/GROOVY-7592?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14901833#comment-14901833
 ] 

Aseem Bansal commented on GROOVY-7592:
--

[~pascalschumacher]
Did you fix the docs? Because I remember that there is another place where that 
is written. Don't remember exactly where.

> Problem in switch statement docs
> 
>
> Key: GROOVY-7592
> URL: https://issues.apache.org/jira/browse/GROOVY-7592
> Project: Groovy
>  Issue Type: Documentation
>  Components: Documentation
>Affects Versions: 2.4.5
>Reporter: Aseem Bansal
>Assignee: Pascal Schumacher
>Priority: Minor
> Fix For: 2.4.6
>
>
> As per http://www.groovy-lang.org/semantics.html default must be the last 
> thing in switch case. 
> Based on that I sent a PR which has been accepted
> https://github.com/apache/incubator-groovy/pull/82
> But when I tried to use default somewhere else it worked fine
> {noformat}
> String str = "aseem"
> switch(str) {
> default:
> println "default"
> break
> case "aseem":
> println "Aseem"
> break
> } 
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)