Re: [Libreoffice] [PATCH] format clipboard

2012-03-28 Thread Cedric Bosdonnat
Hi Maxime, On Tue, 2012-03-27 at 18:57 +0200, Maxime de Roucy wrote: I hope this clarified a bit my choice to remove the nSelectionType == nsSelectionType::SEL_TBL part from the lcl_CreateEmptyItemSet function. Thanks for the details, I pushed the patch and the others that were needed (and

Re: [Libreoffice] [PATCH] format clipboard

2012-03-27 Thread Maxime de Roucy
Hello, Following my discussion with Cedric on IRC here is some other explanations on why I chose to remove the nSelectionType == nsSelectionType::SEL_TBL case from the lcl_CreateEmptyItemSet function in formatclipboard.cxx. Without the patch : lcl_CreateEmptyItemSet is called only in the

Re: [Libreoffice] [PATCH] format clipboard

2012-03-16 Thread Maxime de Roucy
Here a copy of the mail I send to Cedric Bosdonnat… forgot to add the list as Cc:. Hello Thanks you for asking me… When you select a cell in Writer the selection type is a mix of nsSelectionType::SEL_TBL_CELLS, nsSelectionType::SEL_TBL and nsSelectionType::SEL_TXT. So when the

Re: [Libreoffice] [PATCH] format clipboard

2012-03-15 Thread Cedric Bosdonnat
Hi Maxime, On Wed, 2012-03-14 at 16:50 +0100, Maxime de Roucy wrote: Here is some new patchs on the format clipboard. The first just add some comment in the formatclipboard.hxx file. The second one depend on modifications I made in the previous patch

[Libreoffice] [PATCH] format clipboard

2012-03-14 Thread Maxime de Roucy
Hello Here is some new patchs on the format clipboard. The first just add some comment in the formatclipboard.hxx file. The second one depend on modifications I made in the previous patch 0001-add-GetCurParAttr-and-GetPaMParAttr-in-SwEditShell.patch which is itself dependant of the patch