diff options
author | ayumin <ayumin@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-05-08 18:22:29 +0000 |
---|---|---|
committer | ayumin <ayumin@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-05-08 18:22:29 +0000 |
commit | bef8e4557c260e4eccb6266c874b37b32b3ee7a9 (patch) | |
tree | 98874b138722af9c288165aeff3a46141f31fcab | |
parent | 16b8aadc75a50f8c9472f049de4e68678380b823 (diff) | |
download | ruby-bef8e4557c260e4eccb6266c874b37b32b3ee7a9.tar.gz |
* lib/prime.rb (Prime#prime?): negative numbers can't be primes
by definition. reported by Ivan Kataitsev. [Bug #7395]
* test/test_prime.rb: add test.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@45878 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | lib/prime.rb | 1 | ||||
-rw-r--r-- | test/test_prime.rb | 4 |
3 files changed, 8 insertions, 3 deletions
@@ -1,3 +1,9 @@ +Fri May 9 03:07:09 2014 Ayumu AIZAWA <ayumu.aizawa@gmail.com> + + * lib/prime.rb (Prime#prime?): negative numbers can't be primes + by definition. reported by Ivan Kataitsev. [Bug #7395] + * test/test_prime.rb: add test. + Thu May 8 14:34:29 2014 Nobuyoshi Nakada <nobu@ruby-lang.org> * class.c (rb_mod_init_copy): always clear instance variable, diff --git a/lib/prime.rb b/lib/prime.rb index f359c12693..6d3936e317 100644 --- a/lib/prime.rb +++ b/lib/prime.rb @@ -157,7 +157,6 @@ class Prime # +value+:: an arbitrary integer to be checked. # +generator+:: optional. A pseudo-prime generator. def prime?(value, generator = Prime::Generator23.new) - value = -value if value < 0 return false if value < 2 for num in generator q,r = value.divmod num diff --git a/test/test_prime.rb b/test/test_prime.rb index e89952948c..885406f9bd 100644 --- a/test/test_prime.rb +++ b/test/test_prime.rb @@ -141,8 +141,8 @@ class TestPrime < Test::Unit::TestCase # negative assert !-1.prime? - assert(-2.prime?) - assert(-3.prime?) + assert !-2.prime? + assert !-3.prime? assert !-4.prime? end end |