Skip to content

Commit

Permalink
End times should be outside of %"...%"
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
jikamens committed Mar 12, 2019
1 parent f847007 commit c8c0e9b
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion ical2rem.pl
Original file line number Diff line number Diff line change
Expand Up @@ -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");
Expand All @@ -352,7 +353,7 @@ sub _process_todos {
}
print " (-", join(" ", @pieces), ")";
}
print "\%\"%\n";
print "%\n";
}
}
}
Expand Down

0 comments on commit c8c0e9b

Please sign in to comment.