Re: Missing "--" in the "age" function

[Peter Stephenson <p.w.stephenson@xxxxxxxxxxxx>]

On Wed, 22 May 2013 09:46:01 +0100
Peter Stephenson <p.stephenson@xxxxxxxxxxx> wrote:
> Maybe the most convenient thing is if you give one :file argument it
> uses the date, and if there is a second argument it uses the date and
> time for both?

diff --git a/Doc/Zsh/calsys.yo b/Doc/Zsh/calsys.yo
index b3eb454..7dc51ab 100644
--- a/Doc/Zsh/calsys.yo
+++ b/Doc/Zsh/calsys.yo
@@ -665,10 +665,15 @@ Instead of an explicit date and time, it's possible to use the
 modification time of a file as the date and time for either argument
 by introducing the file name with a colon:
 
+example(print *+LPAR()e-age :file1-+RPAR())
+
+matches all files created on the same day (24 hours starting from
+midnight) as tt(file1).
+
 example(print *+LPAR()e-age :file1 :file2-+RPAR())
 
-This matches all files modified no earlier than tt(file1) and
-no later than tt(file2).
+matches all files modified no earlier than tt(file1) and
+no later than tt(file2); precision here is to the nearest second.
 
 texinode(Calendar Styles)(Calendar Utility Functions)(Calendar System User Functions)(Calendar Function System)
 sect(Styles)
diff --git a/Functions/Calendar/age b/Functions/Calendar/age
index c636bdf..17cf4d1 100644
--- a/Functions/Calendar/age
+++ b/Functions/Calendar/age
@@ -33,6 +33,7 @@ zmodload -i zsh/parameter
 
 autoload -Uz calendar_scandate
 
+local timefmt
 local -a vals tmp
 
 [[ -e $REPLY ]] || return 1
@@ -40,14 +41,19 @@ zstat -A vals +mtime -- $REPLY || return 1
 
 if (( $# >= 1 )); then
   if [[ $1 = :* ]]; then
-    zstat -A tmp -F "%Y/%m/%d" +mtime -- ${1#:} || return 1
+      if (( $# > 1 )); then
+	  timefmt="%Y/%m/%d:%H:%M:%S"
+      else
+	  timefmt="%Y/%m/%d"
+      fi
+      zstat -A tmp -F $timefmt +mtime -- ${1#:} || return 1
     local AGEREF=$tmp[1]
   else
     local AGEREF=$1
   fi
   # if 1 argument given, never use globally defined AGEREF2
   if [[ $2 = :* ]]; then
-    zstat -A tmp -F "%Y/%m/%d" +mtime -- ${2#:} || return 1
+    zstat -A tmp -F "%Y/%m/%d:%H:%M:%S" +mtime -- ${2#:} || return 1
     local AGEREF2=$tmp[1]
   else
     local AGEREF2=$2

-- 
Peter Stephenson <p.w.stephenson@xxxxxxxxxxxx>
Web page now at http://homepage.ntlworld.com/p.w.stephenson/