Tidy public fpdfview.h and fpdf_flatten.h.

public/fpdf_flatten.h

Index: public/fpdf_flatten.h
diff --git a/public/fpdf_flatten.h b/public/fpdf_flatten.h
index 0c8322e6972a1edc4580f41519a45e8baa856a02..9197c57648a3463ee0182a34624d53f885e750b7 100644
--- a/public/fpdf_flatten.h
+++ b/public/fpdf_flatten.h
@@ -9,31 +9,33 @@
 
 #include "fpdfview.h"
 
+// Result codes.
 #define FLATTEN_FAIL            0   // Flatten operation failed.
 #define FLATTEN_SUCCESS         1   // Flatten operation succeed.
-#define FLATTEN_NOTINGTODO      2   // There is nothing can be flatten.
+#define FLATTEN_NOTHINGTODO     2   // There is nothing to be flattened.
 
-#ifdef __cplusplus
-extern "C" {
-#endif
+// Backwards compatibility.
+#define FLATTEN_NOTINGTODO FLATTEN_NOTHINGTODO  // [sic].

[Lei Zhang, 2015/05/19 19:11:29]

We should fix the typos in fpdfsdk/src/fpdf_flatten.cpp.

Can we just delete this? It should be really trivial for users, if any, to
update when they roll DEPS.

[Tom Sepez, 2015/05/19 19:40:01]

SGTM.

 
+// Flags.
 #define FLAT_NORMALDISPLAY     0
 #define FLAT_PRINT             1
-    //Function: FPDFPage_Flatten
-
-    //          Flat a pdf page,annotations or form fields will become part of the page contents.
-    //Parameters:
 
-    //          page  - Handle to the page. Returned by FPDF_LoadPage function.
-    //          nFlag - the flag for the use of flatten result. Zero for normal display, 1 for print.
-    //Return value:
-    //          The result flag of the function, See flags above ( FLATTEN_FAIL, FLATTEN_SUCCESS, FLATTEN_NOTINGTODO ).
-    //
-    // Comments: Current version all fails return zero. If necessary we will assign different value
-    //          to indicate different fail reason.
-    //
-    DLLEXPORT int STDCALL FPDFPage_Flatten( FPDF_PAGE page, int nFlag);
+#ifdef __cplusplus
+extern "C" {
+#endif
 
+//Function: FPDFPage_Flatten

[Lei Zhang, 2015/05/19 19:11:29]

nit: space after //

[Tom Sepez, 2015/05/19 19:40:01]

Done.

+//          Make annotations and form fields become part of the page contents itself.
+//Parameters:
+//          page  - Handle to the page, as returned by FPDF_LoadPage().
+//          nFlag - Intended use of the flattened result: 0 for normal display, 1 for printing.
+//Return value:
+//          Either FLATTEN_FAIL, FLATTEN_SUCCESS, or FLATTEN_NOTHINGTODO (see above).
+// Comments:
+//          Currently, all failures return FLATTEN_FAIL, with no indication for the reason
+//          for the failure.
+DLLEXPORT int STDCALL FPDFPage_Flatten(FPDF_PAGE page, int nFlag);
 
 #ifdef __cplusplus
 }