changed it and pushed to master: <a href="http://cgit.freedesktop.org/libreoffice/core/commit/?id=8b8f6835e60e179de616117f59af64d6f2c34b0a">http://cgit.freedesktop.org/libreoffice/core/commit/?id=8b8f6835e60e179de616117f59af64d6f2c34b0a</a> <div class="gmail_quote">2011/8/6 Kohei Yoshida <<a href="mailto:kohei.yoshida@suse.com">kohei.yoshida@suse.com</a>> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"> Hi Markus, <div><div></div><div class="h5"> On Fri, 2011-08-05 at 22:01 +0200, Markus Mohrhard wrote: > Hello, > > this patch should ensure that we write the password algorithm only to > odf 1.2 and later files. > > I'm not quite sure how important it is to be 100% standard compliant > so I don't know if we should include it into 3-4. </div></div>Well, I do know that 100% standard compliance *does* matter to some circle of users, so as long as the fix is a safe fix, I would say we should put it in. I would make one change though. Instead of if ( IsOdfVersionGreaterEqual_1_2() ) we should do if (getDefaultVersion() >= SvtSaveOptions::ODFVER_012) and drop the additional method you added in your patch. Conditionalizing for the ODF version 1.2 or above is common enough that it is done in many other places, and this is how it is done there. Other than that, the patch looks good & safe. Kohei -- Kohei Yoshida, LibreOffice hacker, Calc <<a href="mailto:kohei.yoshida@suse.com">kohei.yoshida@suse.com</a>> </blockquote></div>