mode code cleanup (100c / l, comments)

experimental/PdfViewer/SkPdfGraphicsState.h

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkPdfGraphicsState_DEFINED
 #define SkPdfGraphicsState_DEFINED
 
 #include "SkCanvas.h"
 #include "SkPaint.h"
 #include "SkPdfConfig.h"
 #include "SkPdfUtils.h"
 
-//#include "SkTDStack.h"
-
 class SkPdfFont;
 class SkPdfDoc;
 class SkPdfNativeObject;
 class SkPdfResourceDictionary;
 class SkPdfSoftMaskDictionary;
 
 class SkPdfNativeDoc;
 class SkPdfAllocator;
 
 // TODO(edisonn): move this class in include/core?
@@ skipping 426 matching lines @@
                                           dered on the output device (see Section 6.5.2, “Smoothness Toler-
                                           ance”). The value of this parameter gives the maximum error
                                           tolerance, expressed as a fraction of the range of each color compo-
                                           nent; smaller numbers give smoother color transitions at the
                                           expense of more computation and memory use. Initial value:
                                           installation-dependent.
  */
 
 
 
-
-
-
+    // TODO(edisonn): some defaults are contextual, they could on colorspace, pdf version, ...
 
     SkPdfGraphicsState() {
         fCurPosX      = 0.0;
         fCurPosY      = 0.0;
         fCurFontSize  = 0.0;
         fTextBlock    = false;
         fCTM          = SkMatrix::I();
         fMatrixTm     = SkMatrix::I();
         fMatrixTlm    = SkMatrix::I();
         fPathClosed   = true;
         fLineWidth    = 0;
         fTextLeading  = 0;
         fWordSpace    = 0;
         fCharSpace    = 0;
         fHasClipPathToApply = false;
         fResources    = NULL;
         fSkFont       = NULL;
         fLineCap      = 0;
         fLineJoin     = 0;
         fMiterLimit   = 10.0;
         fAphaConstant = 1.0;
         fAlphaSource  = false;
         fDashArrayLength = 0;
         fDashPhase    = 0;
         fBlendModesLength = 1;
         fBlendModes[0] = SkXfermode::kSrc_Mode;  // PDF: Normal Blend mode
         fSMask        = NULL;
     }
 
-    // TODO(edisonn): make two functons instead, stroking and non stoking, avoid branching
+    // TODO(edisonn): make two functions instead, stroking and non stoking, avoid branching
     void applyGraphicsState(SkPaint* paint, bool stroking);
 };
 
-// TODO(edisonn): better class design.
-// TODO(edisonn): rename to SkPdfContext
 class SkPdfContext {
 public:
     SkTDStackNew<SkPdfNativeObject*>  fObjectStack;
     SkTDStackNew<SkPdfGraphicsState>  fStateStack;
     SkPdfGraphicsState              fGraphicsState;
     SkPdfNativeDoc*                 fPdfDoc;
-    // TODO(edisonn): the allocator, could be freed after the page is done drawing.
+    // TODO(edisonn): the allocator, could be freed after the page is done drawing, so we have the
+    // pixels on the screen asap.
     SkPdfAllocator*                 fTmpPageAllocator;
     SkMatrix                        fOriginalMatrix;
 
     SkPdfContext(SkPdfNativeDoc* doc);
     ~SkPdfContext();
 };
 
 #endif  // SkPdfGraphicsState_DEFINED