FixCrash 1/3: comments/notes.

svn path=/trunk/kdegraphics/kpdf/; revision=376081

TODO

@@ -7,12 +7,14 @@ Legend:
     (*) - Some parts of this item are already done
 
 In progress:
--> Enhance presentation mode. New preferences dialog. (by Tobias)
+-> FIX: Load performance. Big slowdowns on DocumentSynapsis generation.
+   pdfdoc->findDest( namedDest ) takes too long.
+-> Delay TOC (DocumentSynapsis) generation (and move it on thread)
+-> Make document functions as reentrant as possible!
 -> Fix threaded generation. (crash reports on 02 and 03 Jan 2005).
 -> FIX 1/2: sync Memory Management (in Document) with the Generator (exp. undoing requests) (may Segfault!)
 -> FIX 2/2: check for removing 'canUnloadPixmap' from observers in place of requesting
    a list of visible pixmaps
--> type ahead search in pageview (type '/' then the word to search..) (by JakobS)
 
 Urgent fixes and items to get ready before 3.4 (special high-priority list):
 -> New word highlighting for searches / other highlights (enrico)
@@ -28,6 +30,7 @@ Urgent fixes and items to get ready before 3.4 (special high-priority list):
 -> link thumbnails view with document [first, the 'Viewport' must be defined]
 
 More items (first items will enter 'In progress list' first):
+-> type ahead search in pageview (type '/' then the word to search..) (ready: patch by JakobS)
 -> move toolbar view actions in the PageView instead of the part. maybe.. or not...
 -> usability: layout 2PPV [1 2,3 4,5 6] -> [1,2 3,4 5]. add option for 'ebook' style alignment. (by Mikolaj)
 -> usability: trigger redraw on 'filter text' on current page (by Mikolaj)
@@ -65,12 +68,11 @@ More items (first items will enter 'In progress list' first):
 -> ADD: click over image allows "save image" [60% done (activerect of type image)]
 -> screen editing (annotations): framework (BR67300,BR62793)
 -> screen editing (annotations): tools (BR67300)
--> session support: restoring page location (BR82589)
 -> export all text in plain_text/html
 -> extract(export?) images (have a look at ImageOutputDev.cc and pdfimages.cc from xpdf (not in our xpdf sources))
 -> text selection in wordprocessor style (very hard/impossible)
 -> implement history (mainly for actionNamed)
--> history as a toolbox child (collecting DOs's setPage calls)
+-> history as a toolbox child (collecting Doc's viewport changes notifications)
 -> zoom: fit text (with configurable margin)
 -> open gzipped (.pdf.gz?) files
 -> kttsd output with menu entries. speech{document/page/selection}. (patch available - enrico)
@@ -78,7 +80,7 @@ More items (first items will enter 'In progress list' first):
 -> automatic online dictionaries / translators (BR80338)
 -> add OCR for building TextPages out of pure graphical (aka scanned) pages
 -> rotate the whole document / individual pages
--> presentation: implement transitions
+-> presentation: implement transitions (6/11 done)
 -> presentation: add some gfx tools (like a red pencil)
 -> presentation: save a flag (to the xml) to open a pdf in presentation mode
 -> presentation: link following (difficult due to pagerects related to pageview pixmap only)

core/generator_pdf/gp_outputdev.h

@@ -31,7 +31,6 @@ class KPDFPageRect;
 
 /**
  * @short A SplashOutputDev renderer that grabs text and links.
- * ### MERGE: rename definition/impl to generator_pdf_outputdev.h/.cpp
  *
  * This output device:
  * - renders the page using SplashOutputDev (its parent)

core/link.h

@@ -15,7 +15,7 @@
 
 /**
  * @short Encapsulates data that describes a link.
- * ### TODO: recomment
+ * TODO: comment
  */
 class KPDFLink
 {
@@ -32,7 +32,7 @@ class KPDFLink
 class KPDFLinkGoto : public KPDFLink
 {
     public:
-        // define a 'Viewport' TODO MERGE WITH PAGE DEFINED VIEWPORT
+        // define a 'Viewport' TODO MERGE WITH DOCUMENT VIEWPORT
         struct Viewport {
             int page;
             bool fitWidth, fitHeight;
@@ -104,7 +104,8 @@ class KPDFLinkAction : public KPDFLink
 };
 
 
-class KPDFLinkMovie : public KPDFLink //### REDO THIS ###
+class KPDFLinkMovie : public KPDFLink
+//TODO: this
 {
     public:
         KPDFLinkMovie() {};

core/page.h

@@ -22,14 +22,12 @@ class KPDFPageTransition;
 
 /**
  * @short Collector for all the data belonging to a page.
- * ### MERGE: definition and implementation must be moved to kpdfpage.h/.cpp
- * ### RECOMMENT
  *
- * The KPDFPage class contains pixmaps (referenced using obsedvers id as key),
- * a search page (a class used internally for searching data), link classes
- * (that describe links in the current page) plus graphics overlays and more.
+ * The KPDFPage class contains pixmaps (referenced using observers id as key),
+ * a search page (a class used internally for retrieving text), rect classes
+ * (that describe links or other active areas in the current page) and more.
  *
- * Note: All objects passed to this class will be destoryed on class deletion.
+ * Note: All objects are reparented to this class.
  */
 class KPDFPage
 {
@@ -107,7 +105,7 @@ class KPDFPageRect
 
         // query geometric properties
         bool contains( int x, int y ) const;
-        QRect geometry() const; //TODO add intersects( qrect )
+        QRect geometry() const;
 
         // set a pointer to data associated to this rect
         enum PointerType { NoPointer, Link, Image };
@@ -128,8 +126,7 @@ class KPDFPageRect
 
 /**
  * @short Paints a KPDFPage to an open painter using given flags.
- * ### MERGE: since this file will be empty, we might consider renaming
- * ###        definition and implementation to pagepainter.*
+ * TODO: move definition and implementation to pagepainter.* (..maybe..)
  */
 class PagePainter
 {