Fix for ticket #3: Better end-of-talk display. We remove the CounterView when the timer has expired on the last slide.

Signed-off-by: Jonathan 'Wolf' Rentzsch <[email protected]>

Author: Philippe Casgrain

BlitzPDFView.h

@@ -11,6 +11,7 @@
 
 @property uint16_t secondsElapsed;
 
+- (void)atLastPage;
 - (IBAction)updateSecondsElapsed:(id)sender;
 
 @end

BlitzPDFView.m

@@ -163,9 +163,8 @@ - (void)drawPagePost:(PDFPage*)page {
 		[counterView setNeedsDisplay: YES];
 }
 
-- (void)dealloc
-{
-    [counterView release];
+- (void)dealloc {
+    [self atLastPage];
     [super dealloc];
 }
 
@@ -175,7 +174,12 @@ - (uint16_t) secondsElapsed {
 
 - (void)setSecondsElapsed:(uint16_t)secs {
     secondsElapsed = secs;
-		counterView.secondsElapsed = secs;
+    counterView.secondsElapsed = secs;
+}
+
+- (void)atLastPage {
+    [[counterView animator] removeFromSuperview];
+    counterView = nil;
 }
 
 - (IBAction)updateSecondsElapsed:(id)sender {

MyDocument.m

@@ -52,7 +52,13 @@ - (BOOL)readFromURL:(NSURL *)initWithURL ofType:(NSString *)typeName error:(NSEr
 
 - (void)updateElapsedTimer:(NSTimer*)timer_ {
     if (self.pdfView.secondsElapsed != 0 && (self.pdfView.secondsElapsed % 15 == 0)) {
-        [self.pdfView goToNextPage:nil];
+        if ([self.pdfView canGoToNextPage]) {
+            [self.pdfView goToNextPage:nil];
+        }
+        else {
+            [self.pdfView atLastPage];
+            [timer_ invalidate];
+        }
     }
     self.pdfView.secondsElapsed += 1;
     [self.pdfView setNeedsDisplay:YES];