Just a few comments.
+The Error module will provide
It is much better to speak in present, like "The Error module
provides". The same applies to the architecture pages.
[EMAIL PROTECTED] Status types
[EMAIL PROTECTED] Status types
+
[EMAIL PROTECTED] {Data Type} pdf_status_t
+A status variable.
[EMAIL PROTECTED] deftp
I would also comment in the data type description that pdf_status_t
variables are returned by many library functions in order to
communicate the status of the performed operation.
+For each layer there is a macro procedure defined. The file name and
+the line number at which the error ocurred is printed to stderr
+followed by @var{message}.
+
[EMAIL PROTECTED] PDF_DEBUG_BASE (message, ...)
[EMAIL PROTECTED] defmac
[EMAIL PROTECTED] PDF_DEBUG_OBJECT (message, ...)
[EMAIL PROTECTED] defmac
[EMAIL PROTECTED] PDF_DEBUG_DOCUMENT (message, ...)
[EMAIL PROTECTED] defmac
[EMAIL PROTECTED] PDF_DEBUG_PAGE (message, ...)
[EMAIL PROTECTED] defmac
Would be good to document the output format used by these macros in
the reference manual. It is useful if users want to automatically
process the output.
If you agree with those suggestions please fix it, commit the patch
and ACK.
Thanks :)
