Date: prev next · Thread: first prev next last
2012 Archives by date, by thread · List index


On Sun, 2012-01-15 at 09:04 -0800, julien2412 wrote:
First, I thought about just replacing the above line by this one : 
if ( !pCtx->hasType || !pCtx->hasScope )

I have no idea, but http://rfc-ref.org/RFC-TEXTS/2518/chapter23.html
has...

<!ELEMENT lockentry (lockscope, locktype) >
<!ELEMENT lockscope (exclusive | shared) >
<!ELEMENT locktype (write) >

and this is LockEntrySequence_endelement_callback presumably to be
called on processing the close tag, so I would expect that at the end of
a lockentry tag we should have both a locktype and a lockscope

 case STATE_LOCKENTRY:
-   if ( !pCtx->hasType || !pCtx->hasType )
+   if ( !pCtx->hasType || !pCtx->hasScope )
      return 1; // abort

on the basis that presumably if either the type or scope of the lock is
missing then its busted assuming (a big ask) I read the dtd correctly.

C.


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.