code style (readability: scope)
GROOVY-6389: groovysh: Print results not using String.valueOf

Using java.lang.String.valueOf(obj) for instances of Arrays and Maps causes
Object.toString to be used for String generation, causing ugly output in groovysh

This commit avoids that instead which uses pretty output for collections.

As opposed to an earlier commit for Groovy 2.2, Arrays will be displayed
without their type.

Avoiding the GString here because of the earlier code
comment, not sure whether there is any risk using GString here or not
(Comment was from 2007, e222d4f5cf8cc7).
replace other occurences of String.valueOf in groovysh
Merge pull request #268 from tkruse/avoid_valueOf

GROOVY-6389: groovysh flawed display of collections 

print all results using toString() instead of String.valueOf()