Re: [PATCH] fdo#39999: Changing spelling preferences requires application restart

2012-04-22 Thread Tommy
On Sat, 21 Apr 2012 15:34:09 +0200, Dézsi Szabolcs  
dezsisz...@hotmail.com wrote:




Hi!

Bug's page

After applying this patch, checking 'Check uppercase words' in options  
works without restarting LO.
It's very simple, a single line, but it took me a while to find the  
right place to start... :)

I hope that the patch is OK.

Szabolcs



another nice one from you!!!

and I still remember you fixed also OOo Bug 101726 - Speed up autocorrect  
replacement table loading time (that guys at OOo/Apache never fixed it)

https://issues.apache.org/ooo/show_bug.cgi?id=101726

since it seems that you are very good with spellchecking and autocorrect  
issue, would you please take a look at these?


LibO Bug 44580 - EasyHack: share autocorrect replacement table for misc.  
language subgroups

https://bugs.freedesktop.org/show_bug.cgi?id=44580

LibO Bug 46805 - large autocorrect database make LibO freeze when start  
typing

https://www.libreoffice.org/bugzilla/show_bug.cgi?id=46765

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] fdo#39999: Changing spelling preferences requires application restart

2012-04-21 Thread Dézsi Szabolcs

Hi!

Bug's page

After applying this patch, checking 'Check uppercase words' in options works 
without restarting LO.
It's very simple, a single line, but it took me a while to find the right place 
to start... :)
I hope that the patch is OK.

Szabolcs
  From 4918f04e6f7e57314b58061860bc5b7a6f3ea60c Mon Sep 17 00:00:00 2001
From: Szabolcs Dezsi dezsisz...@hotmail.com
Date: Sat, 21 Apr 2012 15:28:50 +0200
Subject: [PATCH] Changing 'Check uppercase words' works without restart

---
 linguistic/source/lngprophelp.cxx |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/linguistic/source/lngprophelp.cxx b/linguistic/source/lngprophelp.cxx
index b93971f..82a7c4d 100644
--- a/linguistic/source/lngprophelp.cxx
+++ b/linguistic/source/lngprophelp.cxx
@@ -522,6 +522,7 @@ void PropertyHelper_Spell::SetTmpPropVals( const PropertyValues rPropVals )
 nResMaxNumberOfSuggestions  = GetDefaultNumberOfSuggestions();
 bResIsSpellWithDigits   = bIsSpellWithDigits;
 bResIsSpellCapitalization   = bIsSpellCapitalization;
+bResIsSpellUpperCase= bIsSpellUpperCase;
 
 sal_Int32 nLen = rPropVals.getLength();
 if (nLen)
-- 
1.7.7

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


Re: [PUSHED][PATCH] fdo#39999: Changing spelling preferences requires application restart

2012-04-21 Thread Caolán McNamara
On Sat, 2012-04-21 at 15:34 +0200, Dézsi Szabolcs wrote:
 After applying this patch, checking 'Check uppercase words' in options
 works without restarting LO.
 It's very simple, a single line, but it took me a while to find the
 right place to start... :)

That's great, looks very good to me, pushed. Much appreciated.

C.



___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice