Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: jruby/jruby
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 964ca3a9ca5a
Choose a base ref
...
head repository: jruby/jruby
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: cd7f7e22530a
Choose a head ref
  • 6 commits
  • 34 files changed
  • 1 contributor

Commits on May 20, 2015

  1. [Truffle] Findbugs.

    chrisseaton committed May 20, 2015
    Copy the full SHA
    365d53a View commit details
  2. [Truffle] Socket specs.

    chrisseaton committed May 20, 2015
    Copy the full SHA
    bc3902c View commit details
  3. [Truffle] Digest specs.

    chrisseaton committed May 20, 2015
    Copy the full SHA
    fe9d301 View commit details
  4. [Truffle] csv specs.

    chrisseaton committed May 20, 2015
    Copy the full SHA
    6c184e7 View commit details
  5. [Truffle] drb specs.

    chrisseaton committed May 20, 2015
    Copy the full SHA
    4869b78 View commit details
  6. Copy the full SHA
    cd7f7e2 View commit details
Showing with 99 additions and 41 deletions.
  1. +1 −0 lib/ruby/truffle/mri/csv.rb
  2. +1 −0 lib/ruby/truffle/mri/drb.rb
  3. +1 −0 lib/ruby/truffle/mri/drb/acl.rb
  4. +1 −0 lib/ruby/truffle/mri/drb/drb.rb
  5. +1 −0 lib/ruby/truffle/mri/drb/eq.rb
  6. +1 −0 lib/ruby/truffle/mri/drb/extserv.rb
  7. +1 −0 lib/ruby/truffle/mri/drb/extservm.rb
  8. +1 −0 lib/ruby/truffle/mri/drb/gw.rb
  9. +1 −0 lib/ruby/truffle/mri/drb/invokemethod.rb
  10. +1 −0 lib/ruby/truffle/mri/drb/observer.rb
  11. +1 −0 lib/ruby/truffle/mri/drb/ssl.rb
  12. +1 −0 lib/ruby/truffle/mri/drb/timeridconv.rb
  13. +1 −0 lib/ruby/truffle/mri/drb/unix.rb
  14. +1 −0 lib/ruby/truffle/shims/weakref.rb
  15. +5 −0 spec/truffle/tags/library/csv/generate_line_tags.txt
  16. +3 −0 spec/truffle/tags/library/csv/generate_tags.txt
  17. +15 −0 spec/truffle/tags/library/csv/parse_tags.txt
  18. +3 −0 spec/truffle/tags/library/csv/readlines_tags.txt
  19. +2 −0 spec/truffle/tags/library/drb/start_service_tags.txt
  20. +1 −0 spec/truffle/tags/library/drb/stop_service_tags.txt
  21. +6 −0 spec/truffle/tags/library/socket/basicsocket/close_read_tags.txt
  22. +1 −0 spec/truffle/tags/library/socket/basicsocket/ioctl_tags.txt
  23. +8 −0 spec/truffle/tags/library/socket/constants/constants_tags.txt
  24. +2 −0 spec/truffle/tags/library/socket/socket/accept_nonblock_tags.txt
  25. +10 −0 spec/truffle/tags/library/socket/socket/bind_tags.txt
  26. +3 −0 spec/truffle/tags/library/socket/socket/connect_nonblock_tags.txt
  27. +1 −0 spec/truffle/tags/library/socket/socket/for_fd_tags.txt
  28. +5 −0 spec/truffle/tags/library/socket/socket/getaddrinfo_tags.txt
  29. +2 −0 spec/truffle/tags/library/socket/socket/gethostbyname_tags.txt
  30. +7 −0 spec/truffle/tags/library/socket/socket/getnameinfo_tags.txt
  31. +1 −0 spec/truffle/tags/library/socket/socket/listen_tags.txt
  32. +2 −0 spec/truffle/tags/library/socket/socket/unpack_sockaddr_in_tags.txt
  33. +1 −41 spec/truffle/truffle.mspec
  34. +7 −0 tool/truffle-findbugs-exclude.xml
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/csv.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../stdlib/csv'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../stdlib/drb'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/acl.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/acl'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/drb.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/drb'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/eq.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/eq'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/extserv.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/extserv'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/extservm.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/extsrvm'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/gw.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/gw'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/invokemethod.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/invokemethod'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/observer.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/observer'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/ssl.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/ssl'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/timeridconv.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/timeridconv'
1 change: 1 addition & 0 deletions lib/ruby/truffle/mri/drb/unix.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require_relative '../../../stdlib/drb/unix'
1 change: 1 addition & 0 deletions lib/ruby/truffle/shims/weakref.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
# Empty file
5 changes: 5 additions & 0 deletions spec/truffle/tags/library/csv/generate_line_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
fails:CSV.generate_line generates an empty string
fails:CSV.generate_line generates the string 'foo,bar'
fails:CSV.generate_line generates the string 'foo;bar'
fails:CSV.generate_line generates the string 'foo,,bar'
fails:CSV.generate_line generates the string 'foo;;bar'
3 changes: 3 additions & 0 deletions spec/truffle/tags/library/csv/generate_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
fails:CSV.generate returns CSV string
fails:CSV.generate accepts a col separator
fails:CSV.generate appends and returns the argument itself
15 changes: 15 additions & 0 deletions spec/truffle/tags/library/csv/parse_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
fails:CSV.parse parses '' into []
fails:"CSV.parse parses '\n' into [[]]"
fails:CSV.parse parses 'foo' into [['foo']]
fails:CSV.parse parses 'foo,bar,baz' into [['foo','bar','baz']]
fails:CSV.parse parses 'foo,baz' into [[foo,nil,baz]]
fails:"CSV.parse parses '\nfoo' into [[],['foo']]"
fails:"CSV.parse parses 'foo\n' into [['foo']]"
fails:"CSV.parse parses 'foo\nbar' into [['foo'],['bar']]"
fails:"CSV.parse parses 'foo,bar\nbaz,quz' into [['foo','bar'],['baz','quz']]"
fails:"CSV.parse parses 'foo,bar'\nbaz' into [['foo','bar'],['baz']]"
fails:"CSV.parse parses 'foo\nbar,baz' into [['foo'],['bar','baz']]"
fails:"CSV.parse parses '\n\nbar' into [[],[],'bar']]"
fails:CSV.parse parses 'foo' into [['foo']] with a separator of ;
fails:CSV.parse parses 'foo;bar' into [['foo','bar']] with a separator of ;
fails:"CSV.parse parses 'foo;bar\nbaz;quz' into [['foo','bar'],['baz','quz']] with a separator of ;"
3 changes: 3 additions & 0 deletions spec/truffle/tags/library/csv/readlines_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
fails:CSV#readlines returns an Array of Array containing each element in a one-line CSV file
fails:CSV#readlines returns an Array of Arrays containing each element in a multi-line CSV file
fails:CSV#readlines returns nil for a missing value
2 changes: 2 additions & 0 deletions spec/truffle/tags/library/drb/start_service_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
fails:DRb.start_service runs a basic remote call
fails:DRb.start_service runs a basic remote call passing a block
1 change: 1 addition & 0 deletions spec/truffle/tags/library/drb/stop_service_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
fails:DRb.stop_service clears the port so a new server can start
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
fails:Socket::BasicSocket#close_read closes the reading end of the socket
fails:Socket::BasicSocket#close_read it works on sockets with closed ends
fails:Socket::BasicSocket#close_read does not close the socket
fails:Socket::BasicSocket#close_read fully closes the socket if it was already closed for writing
fails:Socket::BasicSocket#close_read raises IOError on closed socket
fails:Socket::BasicSocket#close_read returns nil
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
fails:Socket::BasicSocket#ioctl passes data from and to a String correctly
8 changes: 8 additions & 0 deletions spec/truffle/tags/library/socket/constants/constants_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
fails:Socket::Constants defines socket types
fails:Socket::Constants defines protocol families
fails:Socket::Constants defines address families
fails:Socket::Constants defines send/receive options
fails:Socket::Constants defines socket level options
fails:Socket::Constants defines socket options
fails:Socket::Constants defines multicast options
fails:Socket::Constants defines TCP options
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
fails:Socket#accept_nonblock raises Errno::EAGAIN if the connection is not accepted yet
fails:Socket#accept_nonblock raises IO::WaitReadable if the connection is not accepted yet
10 changes: 10 additions & 0 deletions spec/truffle/tags/library/socket/socket/bind_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
fails:Socket#bind on SOCK_DGRAM socket binds to a port
fails:Socket#bind on SOCK_DGRAM socket returns 0 if successful
fails:Socket#bind on SOCK_DGRAM socket raises Errno::EINVAL when binding to an already bound port
fails:Socket#bind on SOCK_DGRAM socket raises Errno::EADDRNOTAVAIL when the specified sockaddr is not available from the local machine
fails:Socket#bind on SOCK_DGRAM socket raises Errno::EACCES when the current user does not have permission to bind
fails:Socket#bind on SOCK_STREAM socket binds to a port
fails:Socket#bind on SOCK_STREAM socket returns 0 if successful
fails:Socket#bind on SOCK_STREAM socket raises Errno::EINVAL when binding to an already bound port
fails:Socket#bind on SOCK_STREAM socket raises Errno::EADDRNOTAVAIL when the specified sockaddr is not available from the local machine
fails:Socket#bind on SOCK_STREAM socket raises Errno::EACCES when the current user does not have permission to bind
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
fails:Socket#connect_nonblock connects the socket to the remote side
fails:Socket#connect_nonblock raises Errno::EINPROGRESS when the connect would block
fails:Socket#connect_nonblock raises Errno::EINPROGRESS with IO::WaitWritable mixed in when the connect would block
1 change: 1 addition & 0 deletions spec/truffle/tags/library/socket/socket/for_fd_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
fails:Socket.for_fd creates a new Socket that aliases the existing Socket's file descriptor
5 changes: 5 additions & 0 deletions spec/truffle/tags/library/socket/socket/getaddrinfo_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
fails:Socket#getaddrinfo gets the address information
fails:Socket#getaddrinfo accepts empty addresses for IPv4 passive sockets
fails:Socket#getaddrinfo accepts empty addresses for IPv4 non-passive sockets
fails:Socket#getaddrinfo accepts empty addresses for IPv6 passive sockets
fails:Socket#getaddrinfo accepts empty addresses for IPv6 non-passive sockets
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
fails:Socket#gethostbyname returns broadcast address info for '<broadcast>'
fails:Socket#gethostbyname returns broadcast address info for '<any>'
7 changes: 7 additions & 0 deletions spec/truffle/tags/library/socket/socket/getnameinfo_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
fails:Socket.getnameinfo gets the name information and don't resolve it
fails:Socket.getnameinfo gets the name information and resolve the host
fails:Socket.getnameinfo gets the name information and resolves the service
fails:Socket.getnameinfo gets a 3-element array and doesn't resolve hostname
fails:Socket.getnameinfo gets a 3-element array and resolves the service
fails:Socket.getnameinfo gets a 4-element array and doesn't resolve hostname
fails:Socket.getnameinfo gets a 4-element array and resolves the service
1 change: 1 addition & 0 deletions spec/truffle/tags/library/socket/socket/listen_tags.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
fails:Socket#listen verifies we can listen for incoming connections
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
fails:Socket#unpack_sockaddr_in decodes the host name and port number of a packed sockaddr_in
fails:Socket#unpack_sockaddr_in raises an ArgumentError when the sin_family is not AF_INET
42 changes: 1 addition & 41 deletions spec/truffle/truffle.mspec
Original file line number Diff line number Diff line change
@@ -88,49 +88,10 @@ class MSpecScript
set :core, core

set :library, [
"spec/ruby/library/abbrev",
"spec/ruby/library/base64",
"spec/ruby/library/complex",
"spec/ruby/library/conditionvariable",
"spec/ruby/library/date",
"spec/ruby/library/datetime",
"spec/ruby/library/delegate",
"spec/ruby/library/digest/md5",
"spec/ruby/library/cgi",
"spec/ruby/library/erb",
"spec/ruby/library/getoptlong",
"spec/ruby/library/ipaddr",
"spec/ruby/library/matrix",
"spec/ruby/library/logger",
"spec/ruby/library/observer",
"spec/ruby/library/open3",
"spec/ruby/library/openstruct",
"spec/ruby/library/pathname",
"spec/ruby/library/prime",
"spec/ruby/library/resolv",
"spec/ruby/library/scanf",
"spec/ruby/library/securerandom",
"spec/ruby/library/set",
"spec/ruby/library/shellwords",
"spec/ruby/library/singleton",
"spec/ruby/library/stringio",
"spec/ruby/library/stringscanner",
"spec/ruby/library/tempfile",
"spec/ruby/library/thread",
"spec/ruby/library/time",
"spec/ruby/library/tmpdir",
"spec/ruby/library/uri",
"spec/ruby/library/bigdecimal",
"spec/ruby/library/zlib",
"spec/ruby/library",

# Not yet explored
"^spec/ruby/library/continuation",
"^spec/ruby/library/csv",
"^spec/ruby/library/digest/sha1",
"^spec/ruby/library/digest/sha256",
"^spec/ruby/library/digest/sha384",
"^spec/ruby/library/digest/sha512",
"^spec/ruby/library/drb",
"^spec/ruby/library/etc",
"^spec/ruby/library/expect",
"^spec/ruby/library/fiber",
@@ -144,7 +105,6 @@ class MSpecScript
"^spec/ruby/library/weakref",
"^spec/ruby/library/win32ole",
"^spec/ruby/library/yaml",
"^spec/ruby/library/socket",

# Load issues with 'delegate'.
"^spec/ruby/library/delegate/delegate_class/instance_method_spec.rb",
7 changes: 7 additions & 0 deletions tool/truffle-findbugs-exclude.xml
Original file line number Diff line number Diff line change
@@ -64,6 +64,13 @@
<Bug pattern="FE_FLOATING_POINT_EQUALITY" />
</Match>

<!-- Sometimes we really do want to compare strings by identity -->

<Match>
<Class name="org.jruby.truffle.nodes.constants.LookupConstantNode" />
<Bug pattern="ES_COMPARING_PARAMETER_STRING_WITH_EQ" />
</Match>

<!-- Stuff where we're genuinely doing something complicated and we do know what we're doing -->

<Match>