Looks good, but the definition for the canonical text signature is
still wrong. (Also I note you didn't include the note about PGP 2.6.x
being unable to hande signature and pubkey packets of length type 0.)
| 0x01: Signature of a canonical text document.
| Typically, this means the signer owns it, created it, or
| certifies that it has not been modified. The signature is
| calculated over the text data with its line endings converted to
| <CR><LF> and trailing blanks removed.
Trailing blanks are removed _only_ for cleartext signatures, in the
transformation described in section 7.1. Otherwise (for signatures
contained in a PGP message, and for detached signatures) they are part
of the signature.