Merge pull request #12 from nodanaonlyzuul/fix-this-thing

Fix this thing
nodanaonlyzuul · Jan 10, 2022 · 954753a · 954753a
2 parents 9641a1a + fa2f462
commit 954753a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/asciiart.rb b/lib/asciiart.rb
@@ -14,7 +14,7 @@ class AsciiArt
 
   def initialize(path_to_file)
     # open-uri open will fallback to IO open
-    open(path_to_file) { |file| @data = file.read }
+    URI.open(path_to_file) { |file| @data = file.read }
     self
   end
 
@@ -25,4 +25,3 @@ def to_ascii_art(options = {})
     renderer.to_ascii_art(options)
   end
 end
-
diff --git a/lib/asciiart/text.rb b/lib/asciiart/text.rb
@@ -45,11 +45,12 @@ def image_chars
     private
     def unified_rgb_value(number)
       if defined?(Magick::QuantumDepth)
-        return (Magick::QuantumDepth == 16) ? (number / 256) : number
+        num = (Magick::QuantumDepth == 16) ? (number / 256) : number
       else
-        return (Magick::QuantumRange == 65535) ? (number / 256) : number
+        num = (Magick::QuantumRange == 65535) ? (number / 256) : number
       end
+      (num > 255) ? 255 : num
     end
 
   end
-end
+end