JACOBIN-563 Added polish and documentary comments to convenience func…

…tion to identify the type of array

src/object/array_test.go

@@ -48,6 +48,11 @@ func TestGetArrayType(t *testing.T) {
  t.Errorf("did not get expected 'I' for I, got: %s", retVal)
  }
 
+ retVal = GetArrayType("[Ljava/lang/Object;")
+ if retVal != "Ljava/lang/Object;" {
+ t.Errorf("did not get expected 'Ljava/lang/Object;', got: %s", retVal)
+ }
+
  retVal = GetArrayType("")
  if retVal != "" {
  t.Errorf("did not get expected empty string for an empty string, got %s", retVal)

src/object/arrays.go

@@ -75,6 +75,9 @@ func JdkArrayTypeToJacobinType(jdkType int) int {
  }
 }
 
+// identifies the type of entry that the array is made up of
+// i.e., primitives or specific references. Note if the array is a
+// reference array, the trailing ; is *not* removed.
 func GetArrayType(arrayType string) string {
  typeChars := []byte(arrayType)
  for index, char := range typeChars {