Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-07-08 Thread Cedric Bosdonnat
Hi John, Korrawit, On Wed, 2011-07-06 at 22:51 +0700, Korrawit Pruegsanusak wrote: On Fri, Jun 24, 2011 at 14:38, John LeMoyne Castle lemoyne.cas...@gmail.com wrote: -- you (and others) cannot reproduce fdo#37584 (redlined text disappears) in 3-3 or 3-3-3 because my patch referenced in

Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-07-06 Thread Korrawit Pruegsanusak
Hello John, all, First, sorry for my late reply. On Fri, Jun 24, 2011 at 14:38, John LeMoyne Castle lemoyne.cas...@gmail.com wrote:  -- you (and others) cannot reproduce fdo#37584 (redlined text disappears) in 3-3 or 3-3-3 because my patch referenced in your original post [OP] is not in the

Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-06-24 Thread John LeMoyne Castle
Hi Korrawit, My take is: -- you (and others) cannot reproduce fdo#37584 (redlined text disappears) in 3-3 or 3-3-3 because my patch referenced in your original post [OP] is not in the 3-3 branches. -- you and others cannot reproduce 'leading quote as word' in your 3-3 build because the

Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-06-23 Thread Korrawit Pruegsanusak
Hello Cedric, Tor, John, all, Yesterday, @vilpan (in irc) and I tested on official 3.3.3 build and can reproduce fdo#33774 ('leading quote as word') But we both can't reproduce fdo#37584. So we should fix only fdo#33774. However, I've found two patches about this in -3-4 and master:

Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-06-22 Thread Tor Lillqvist
Did you backport also the fixes mentioned by John? No, the mail thread was otherwise too confusing to me... --tml ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice

Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-06-22 Thread Korrawit Pruegsanusak
Hello all, On Wed, Jun 22, 2011 at 12:59, Tor Lillqvist tlillqv...@novell.com wrote: Thanks, committed and pushed to 3-3. (With a slightly improved commit message.) Thanks :-) On Wed, Jun 22, 2011 at 19:48, Tor Lillqvist tlillqv...@novell.com wrote: Did you backport also the fixes mentioned

Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-06-22 Thread Cedric Bosdonnat
On Wed, 2011-06-22 at 20:58 +0700, Korrawit Pruegsanusak wrote: Two issues John mentioned are: 1. fdo#37584: redline text disappear I would be really surprised if that one didn't appear... -- Cédric Bosdonnat LibreOffice hacker http://documentfoundation.org OOo Eclipse Integration developer

Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-06-21 Thread John LeMoyne Castle
Hi Korrawit, Cedric, all ... A first, quick reply is that this patch of mine contains the dread redline wipeout: + // make a copy of the text + String rTextCopy = const_castlt;Stringamp;gt;(m_Text); The latter needs to be modified to include (or be immediately followed by) Cedric's fix: -

Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-06-21 Thread John LeMoyne Castle
Attached is another patch (in libs-gui repo) that is in 3.4 but not in 3.3 found with lo-commit-stat using: root/bin $perl lo-commit-stat --log-dir='./' --log-suffix='mjWCRL-3.3' --rev-list ../ --until='2010-12-01' --author='Johnsson' origin/libreoffice-3-4 ^origin/libreoffice-3-3 and shown

Re: [Libreoffice] [PATCH] Fix for fdo#30550 Character count without spaces

2011-06-21 Thread Tor Lillqvist
Attached patch is a fix for fdo#30550 in comment 11-12 Thanks, committed and pushed to 3-3. (With a slightly improved commit message.) --tml ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org