Change code structure, style

modules/clipboard/README.md

@@ -3,7 +3,4 @@ Clipboard
 
 Integrates zsh clipboard with system clipboard.
 
-Settings
+On OSX, uses pbcopy and pbpaste. On cygwin, uses /dev/clipboard. On linux, requires either xclip or xsel be installed.
---------
-
-Make sure to set IS_MAC_OS correctly in init.zsh.

modules/clipboard/init.zsh

@@ -5,20 +5,18 @@
 #   Nir Friedman <quicknir@gmail.com>
 #
 
-# If running Mac OS, set this to 1
+function cutbuffer {
-local IS_MAC_OS=0
+  zle .$WIDGET
-
+  if [[ "$OSTYPE" == darwin* ]]; then
-if [[ IS_MAC_OS -eq 0 ]]; then
-  function cutbuffer() {
-    zle .$WIDGET
-    echo $CUTBUFFER | xclip -selection clipboard
-  }
-else
-  function cutbuffer() {
-    zle .$WIDGET
     echo $CUTBUFFER | pbcopy
-  }
+  elif [[ "$OSTYPE" == cygwin* ]]; then
-fi
+    echo $CUTBUFFER | tee > /dev/clipboard
+  elif (( $+commands[xclip] )); then
+    echo $CUTBUFFER | xclip -selection clipboard
+  elif (( $+commands[xsel] )); then
+    echo $CUTBUFFER | xsel --clipboard --input
+  fi
+}
 
 zle_cut_widgets=(
   vi-backward-delete-char
@@ -33,30 +31,28 @@ zle_cut_widgets=(
   vi-yank-eol
 )
 
-for widget in $zle_cut_widgets
+for widget in $zle_cut_widgets; do
-do
   zle -N $widget cutbuffer
 done
 
-
+function putbuffer {
-if [[ IS_MAC_OS -eq 0 ]]; then
+  if [[ "$OSTYPE" == darwin* ]]; then
-  function putbuffer() {
+    zle copy-region-as-kill "$(pbcopy)"
+  elif [[ "$OSTYPE" == cygwin* ]]; then
+    zle copy-region-as-kill "$(cat /dev/clipboard)"
+  elif (( $+commands[xclip] )); then
     zle copy-region-as-kill "$(xclip -o -selection clipboard)"
-    zle .$WIDGET
+  elif (( $+commands[xsel] )); then
-  }
+    zle copy-region-as-kill "$(xsel --clipboard --output)"
-else
+  fi
-  function putbuffer() {
+  zle .$WIDGET
-    zle copy-region-as-kill "$(pbpaste)"
+}
-    zle .$WIDGET
-  }
-fi
 
 zle_put_widgets=(
   vi-put-after
   vi-put-before
 )
 
-for widget in $zle_put_widgets
+for widget in $zle_put_widgets; do
-do
   zle -N $widget putbuffer
 done