On Fri, 2011-09-23 at 22:36 +0400, Ivan Timofeev wrote: > 2011/9/23 Caolán McNamara <caol...@redhat.com>: > > Looks sane to me. A good catch, pushed now. > > He-he, I forgot to consider absolutely crazy cases - if > <mmultiscripts> element is empty at all. Should this be fixed?
Probably, can't ever be too safe on anything which is basically content controlled by the outside world I suppose. There's always something that manages to find its way through the logic cracks eventually. Pushed this now. In that vein - if (rNodeStack.Count() == nElementCount) + if (rNodeStack.Count() <= nElementCount) doesn't hurt either I think. C. _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
