[Libreoffice] [PUSHED][PATCH 1/4] fdo38832 Replace ImplAccelTable with std::map

2011-12-12 Thread Caolán McNamara
On Sat, 2011-12-10 at 19:23 +0100, Marcel Metz wrote: Hello lo-devs, This patch series replaces some DECLARE_TABLE macros with the std::map equivalent. Looks good to me. Thanks for this, pushed now. C. ___ LibreOffice mailing list

Re: [Libreoffice] [PUSHED][PATCH 1/4] fdo38832 Replace ImplAccelTable with std::map

2011-12-12 Thread Ivan Timofeev
12.12.2011 20:05, Caolán McNamara пишет: On Sat, 2011-12-10 at 19:23 +0100, Marcel Metz wrote: Hello lo-devs, This patch series replaces some DECLARE_TABLE macros with the std::map equivalent. Looks good to me. Thanks for this, pushed now. Hmm... looking at this I think that - else if (

Re: [Libreoffice] [PUSHED][PATCH 1/4] fdo38832 Replace ImplAccelTable with std::map

2011-12-12 Thread Caolán McNamara
On Mon, 2011-12-12 at 16:05 +, Caolán McNamara wrote: On Sat, 2011-12-10 at 19:23 +0100, Marcel Metz wrote: Hello lo-devs, This patch series replaces some DECLARE_TABLE macros with the std::map equivalent. Looks good to me. Thanks for this, pushed now. oh wait... - else if (

Re: [Libreoffice] [PUSHED][PATCH 1/4] fdo38832 Replace ImplAccelTable with std::map

2011-12-12 Thread Caolán McNamara
On Mon, 2011-12-12 at 20:10 +0400, Ivan Timofeev wrote: i.e. ! is missed in replacement. Indeed, fixed up now. C. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice