Hi Noel, On Tue, Apr 16, 2013 at 01:04:46PM -0700, Noel Grandin <n...@peralex.com> wrote: > sfx2/inc/sfx2/dispatch.hxx | 4 + > sfx2/source/control/dispatch.cxx | 99 > ++++++++++++++++++--------------------- > 2 files changed, 49 insertions(+), 54 deletions(-) > > New commits: > commit 7e4db91a3900fd0bbce0f0592a045458a37862b3 > Author: Noel Grandin <n...@peralex.com> > Date: Mon Apr 15 11:21:16 2013 +0200 > > Convert SfxShellStack_Impl from SfxPtrArr to std::vector > > Change-Id: I1ffd615a505029242a87337c9ef42c8156c42d70 > Reviewed-on: https://gerrit.libreoffice.org/3394
After this commit, I see some new warnings on startup: warn:legacy.tools:27529:1:sfx2/source/control/dispatch.cxx:1592: pushed SfxShell already on stack Reverting this commit (to test, you need to also revert 187a592df9bc9c1ef4b53fe6dce40a3c74f6bb67 and e2ce3538943c510efa8041fca3c83728acf3a155, but they are unlikely to be reason for this) makes the warning go away. I did not look into this deeply, is it possible that the old container checked if the given SfxShell is already on the stack, and if so, it didn't add it? Miklos
signature.asc
Description: Digital signature
_______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
