]> git.saurik.com Git - redis.git/commitdiff
Replace unnecessary calls to echo and cat
authorMichael Schlenker <MichaelSchlenker@gmx.net>
Tue, 17 Apr 2012 20:20:54 +0000 (22:20 +0200)
committerantirez <antirez@gmail.com>
Tue, 24 Apr 2012 17:33:54 +0000 (19:33 +0200)
Tcl's exec can send data to stdout itself, no need to call cat/echo for
that usually.

tests/integration/aof.tcl
tests/support/server.tcl
tests/unit/aofrw.tcl

index 9c93b6a6ed66a47122371884673ed6686c9c0e6e..ebf9cb5649888843192bebd4eff8eeb4b365251b 100644 (file)
@@ -35,7 +35,7 @@ tags {"aof"} {
             set pattern "*Unexpected end of file reading the append only file*"
             set retry 10
             while {$retry} {
-                set result [exec cat [dict get $srv stdout] | tail -n1]
+                set result [exec tail -n1 < [dict get $srv stdout]]
                 if {[string match $pattern $result]} {
                     break
                 }
@@ -59,7 +59,7 @@ tags {"aof"} {
             set pattern "*Bad file format reading the append only file*"
             set retry 10
             while {$retry} {
-                set result [exec cat [dict get $srv stdout] | tail -n1]
+                set result [exec tail -n1 < [dict get $srv stdout]]
                 if {[string match $pattern $result]} {
                     break
                 }
@@ -81,7 +81,7 @@ tags {"aof"} {
     }
 
     test "Short read: Utility should be able to fix the AOF" {
-        set result [exec echo y | src/redis-check-aof --fix $aof_path]
+        set result [exec src/redis-check-aof --fix $aof_path << "y\n"]
         assert_match "*Successfully truncated AOF*" $result
     }
 
index b1ab38fc1089afcfae386c14d16f5ff041fecc3f..35c1cb87076444b9d9d1c7b4f2a56b782e0d0088 100644 (file)
@@ -252,7 +252,7 @@ proc start_server {options {code undefined}} {
 
         while 1 {
             # check that the server actually started and is ready for connections
-            if {[exec cat $stdout | grep "ready to accept" | wc -l] > 0} {
+            if {[exec grep "ready to accept" | wc -l < $stdout] > 0} {
                 break
             }
             after 10
index e341d77b215159e1e47867b140387bb809cce2c2..4716d08e60e874341e6de3c398cf5021501ae585 100644 (file)
@@ -7,7 +7,7 @@ start_server {tags {"aofrw"}} {
         r bgrewriteaof
         r config set appendonly no
         r exec
-        set result [exec cat [srv 0 stdout] | tail -n1]
+        set result [exec tail -n1 < [srv 0 stdout] ]
     } {*Killing*AOF*child*}
 
     foreach d {string int} {