From d855547661ee4173bd01ab89ad18418d4dbf508a Mon Sep 17 00:00:00 2001
From: Robert Estelle <robertestelle@gmail.com>
Date: Fri, 12 Jul 2019 17:01:10 -0400
Subject: [PATCH] clipboard: Reduce unnecessary special-casing on stdin

Ideally the parameter would just be removed-users could always
just do "clipcopy < some-file". but removing the parameter would break
backwards compatibility.

In any case, this simplifies the logic considerably.
---
 lib/clipboard.zsh | 27 ++++++---------------------
 1 file changed, 6 insertions(+), 21 deletions(-)

diff --git a/lib/clipboard.zsh b/lib/clipboard.zsh
index 2c93d1bb5..15ad6d916 100644
--- a/lib/clipboard.zsh
+++ b/lib/clipboard.zsh
@@ -17,32 +17,17 @@
 #
 function clipcopy() {
   emulate -L zsh
-  local file=$1
+  local file="${1:-/dev/stdin}"
+
   if [[ $OSTYPE == darwin* ]]; then
-    if [[ -z $file ]]; then
-      pbcopy
-    else
-      cat $file | pbcopy
-    fi
+    pbcopy < "${file}"
   elif [[ $OSTYPE == cygwin* ]]; then
-    if [[ -z $file ]]; then
-      cat > /dev/clipboard
-    else
-      cat $file > /dev/clipboard
-    fi
+    cat "${file}" > /dev/clipboard
   else
     if (( $+commands[xclip] )); then
-      if [[ -z $file ]]; then
-        xclip -in -selection clipboard
-      else
-        xclip -in -selection clipboard $file
-      fi
+      xclip -in -selection clipboard < "${file}"
     elif (( $+commands[xsel] )); then
-      if [[ -z $file ]]; then
-        xsel --clipboard --input 
-      else
-        cat "$file" | xsel --clipboard --input
-      fi
+      xsel --clipboard --input  < "${file}"
     else
       print "clipcopy: Platform $OSTYPE not supported or xclip/xsel not installed" >&2
       return 1