Remove unneeded parenthesis from calls in macro expression (#5493)

* Remove parenthesis from macro calls without arguments: not needed anymore as of 0.24.0 * Resolve TODO in urandom
crystal-lang · Jan 20, 2018 · 3515968 · 3515968
1 parent a288123
commit 3515968
Show file tree

Hide file tree

Showing 11 changed files with 12 additions and 13 deletions.
diff --git a/spec/compiler/semantic/macro_spec.cr b/spec/compiler/semantic/macro_spec.cr
@@ -1049,7 +1049,7 @@ describe "Semantic: macro" do
         class A
         end
 
-        {% skip_file() %}
+        {% skip_file %}
 
         class B
         end
@@ -1066,7 +1066,7 @@ describe "Semantic: macro" do
 
         {% if true %}
           class C; end
-          {% skip_file() %}
+          {% skip_file %}
           class D; end
         {% end %}
 

diff --git a/src/callstack.cr b/src/callstack.cr
@@ -1,4 +1,4 @@
-{% skip_file() if flag?(:win32) %}
+{% skip_file if flag?(:win32) %}
 
 require "c/dlfcn"
 require "c/stdio"

diff --git a/src/compiler/crystal/macros/interpreter.cr b/src/compiler/crystal/macros/interpreter.cr
@@ -77,7 +77,7 @@ module Crystal
                    @scope : Type, @path_lookup : Type, @location : Location?,
                    @vars = {} of String => ASTNode, @block : Block? = nil, @def : Def? = nil,
                    @in_macro = false)
-      @str = IO::Memory.new(512) # Can't be String::Builder because of `{{debug()}}
+      @str = IO::Memory.new(512) # Can't be String::Builder because of `{{debug}}`
       @last = Nop.new
     end
 

diff --git a/src/crystal/system/unix/arc4random.cr b/src/crystal/system/unix/arc4random.cr
@@ -1,4 +1,4 @@
-{% skip_file() unless flag?(:openbsd) %}
+{% skip_file unless flag?(:openbsd) %}
 
 require "c/stdlib"
 

diff --git a/src/crystal/system/unix/getrandom.cr b/src/crystal/system/unix/getrandom.cr
@@ -1,4 +1,4 @@
-{% skip_file() unless flag?(:linux) %}
+{% skip_file unless flag?(:linux) %}
 
 require "c/unistd"
 require "c/sys/syscall"

diff --git a/src/crystal/system/unix/sysconf_cpucount.cr b/src/crystal/system/unix/sysconf_cpucount.cr
@@ -1,4 +1,4 @@
-{% skip_file() if flag?(:openbsd) || flag?(:freebsd) %}
+{% skip_file if flag?(:openbsd) || flag?(:freebsd) %}
 
 require "c/unistd"
 

diff --git a/src/crystal/system/unix/sysctl_cpucount.cr b/src/crystal/system/unix/sysctl_cpucount.cr
@@ -1,4 +1,4 @@
-{% skip_file() unless flag?(:openbsd) || flag?(:freebsd) %}
+{% skip_file unless flag?(:openbsd) || flag?(:freebsd) %}
 
 require "c/sysctl"
 

diff --git a/src/crystal/system/unix/urandom.cr b/src/crystal/system/unix/urandom.cr
@@ -1,5 +1,4 @@
-# TODO: replace with `flag?(:unix) && !flag?(:openbsd) && !flag?(:linux)` after crystal > 0.22.0 is released
-{% skip_file() if flag?(:openbsd) && flag?(:linux) %}
+{% skip_file unless flag?(:unix) && !flag?(:openbsd) && !flag?(:linux) %}
 
 module Crystal::System::Random
   @@initialized = false

diff --git a/src/io/console.cr b/src/io/console.cr
@@ -1,4 +1,4 @@
-{% skip_file() if flag?(:win32) %}
+{% skip_file if flag?(:win32) %}
 
 require "termios"
 

diff --git a/src/io/encoding.cr b/src/io/encoding.cr
@@ -15,7 +15,7 @@ class IO
     end
   end
 
-  {% skip_file() if flag?(:win32) %}
+  {% skip_file if flag?(:win32) %}
 
   private class Encoder
     def initialize(@encoding_options : EncodingOptions)

diff --git a/src/io/syscall.cr b/src/io/syscall.cr
@@ -1,4 +1,4 @@
-{% skip_file() if flag?(:win32) %}
+{% skip_file if flag?(:win32) %}
 
 module IO::Syscall
   @read_timed_out = false