From c8c0e9bbcc4cefce33df3bf9fe37744e81822f1b Mon Sep 17 00:00:00 2001
From: Jonathan Kamens <jik@kamens.us>
Date: Tue, 12 Mar 2019 08:06:18 -0400
Subject: [PATCH] End times should be outside of %"...%"

Although the remind documentation claims that remind doesn't actually
use `DURATION` for anything, that's not entirely accurate. It turns
out that both `rem -c` and `rem2ps` display reminder end times when
both `AT` and `DURATION` are both specified. Therefore, it's not
necessary to put the end time explicitly inside the %"...%" calendar
markers.
---
 ical2rem.pl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/ical2rem.pl b/ical2rem.pl
index d759f1a..736b559 100644
--- a/ical2rem.pl
+++ b/ical2rem.pl
@@ -332,6 +332,7 @@ sub _process_todos {
                 print "%\"", &quote($event->{'SUMMARY'});
                 print(" at ", &quote($event->{'LOCATION'}))
                     if ($do_location and $event->{'LOCATION'});
+                print "\%\"";
                 if ($do_end_times and ($start->hour or $start->minute or
                                        $end->hour or $end->minute)) {
                     my $start_date = $start->strftime("%F");
@@ -352,7 +353,7 @@ sub _process_todos {
                     }
                     print " (-", join(" ", @pieces), ")";
                 }
-                print "\%\"%\n";
+                print "%\n";
             }
         }
     }