Polish interop documentation

Author: eregon

doc/contributor/interop_implicit_api.md

@@ -21,9 +21,9 @@ Format: `Ruby code` sends `InteropLibrary message`
 - `foreign_object.size` sends `getArraySize(foreign_object)`
 - `foreign_object.keys` sends `getMembers(foreign_object)`
 - `foreign_object.method_name` sends `invokeMember(foreign_object, method_name)` if member is invocable
-- `foreign_object.method_name` sends `readMember(foreign_object, method_name)` if member is readable
-- `foreign_object.method_name` sends `readMember(foreign_object, method_name)` and raises if member is not invocable or readable
-- `foreign_object.method_name(*arguments)` sends `invokeMember(foreign_object, method_name, *arguments)` if member is readable and invokable
+- `foreign_object.method_name` sends `readMember(foreign_object, method_name)` if member is readable but not invocable
+- `foreign_object.method_name` sends `readMember(foreign_object, method_name)` and raises if member is neither invocable nor readable
+- `foreign_object.method_name(*arguments)` sends `invokeMember(foreign_object, method_name, *arguments)`
 - `foreign_object.method_name(*arguments, &block)` sends `invokeMember(foreign_object, method_name, *arguments, block)`
 - `foreign_object.new(*arguments)` sends `instantiate(foreign_object, *arguments)`
 - `foreign_object.class` sends `readMember(foreign_object, "class")` when `foreign_object` is a `java.lang.Class`

spec/truffle/interop/special_forms_spec.rb

@@ -152,22 +152,22 @@
     l.log.should include(["invokeMember", "foo"])
   end
 
-  it description['.method_name', :readMember, ['method_name'], 'if member is readable'] do
+  it description['.method_name', :readMember, ['method_name'], 'if member is readable but not invocable'] do
     pfo, _, l = proxy[TruffleInteropSpecs::PolyglotMember.new]
     pfo.foo = :bar
     pfo.foo
     l.log.should include(["isMemberInvocable", "foo"])
     l.log.should include(["readMember", "foo"])
   end
 
-  it description['.method_name', :readMember, ['method_name'], 'and raises if member is not invocable or readable'] do
+  it description['.method_name', :readMember, ['method_name'], 'and raises if member is neither invocable nor readable'] do
     pfo, _, l = proxy[Object.new]
     -> { pfo.foo }.should raise_error(NameError)
     l.log.should include(["isMemberInvocable", "foo"])
     l.log.should include(["readMember", "foo"])
   end
 
-  it description['.method_name(*arguments)', :invokeMember, ['method_name', '*arguments'], 'if member is readable and invokable'] do
+  it description['.method_name(*arguments)', :invokeMember, ['method_name', '*arguments']] do
     pfo, _, l = proxy[Object.new]
     -> { pfo.bar(1, 2, 3) }.should raise_error(NoMethodError)
     l.log.should include(["invokeMember", "bar", 1, 2, 3])