Bug 71939 - Lao Spellcheck
Summary: Lao Spellcheck
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Extensions (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: Other All
: medium normal
Assignee: Andras Timar
URL:
Whiteboard: target:4.3.0 target:4.2.0.0.beta2
Keywords:
Depends on:
Blocks:
 
Reported: 2013-11-23 15:29 UTC by Robert M Campbell
Modified: 2013-11-24 19:55 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Robert M Campbell 2013-11-23 15:29:36 UTC
Just started working on Spellcheck dictionary extension for Lao. Here's what I have so far: https://gerrit.libreoffice.org/#/c/6774/

I've not been able to get it to build for some reason (it may be my local setup, so may not be an issue for anyone else).
Comment 1 tommy27 2013-11-24 08:17:43 UTC
I add Andras Timar to CC list. maybe he can help.
Comment 2 Robert M Campbell 2013-11-24 11:52:59 UTC
Ignore changeset 5 (still learning git). Changeset 6 should include the right data (file encoding problem in vim). I'll build it and see if that gets it working.
Comment 3 tommy27 2013-11-24 12:44:54 UTC
Ok. we will wait 4 your feedback.
Comment 4 Robert M Campbell 2013-11-24 13:35:47 UTC
Looks good. Built and seems to work fine on this end. Like the Khmer and Thai spellcheckers, it'll be limited to how ICU detects the words (might give false negatives and/or false-positives), but at least it's a start! Thanks to each of you for your help!
Comment 5 Robert M Campbell 2013-11-24 13:38:49 UTC
Lol, actually, got one more change - needs a single 'word' added to the list. Will comment once it's ready.
Comment 6 Robert M Campbell 2013-11-24 13:44:04 UTC
Changeset 7 (need to add the repeat symbol as a correctly spelled word)
Comment 7 Robert M Campbell 2013-11-24 14:22:18 UTC
Works great now! Should be good to go.
Comment 8 bfoman (inactive) 2013-11-24 19:34:17 UTC
I think that as you are actively working on the bug please assign it to yourself and change its status to ASSIGNED.
Comment 9 Commit Notification 2013-11-24 19:40:36 UTC
Robert M Campbell committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/dictionaries/commit/?id=8e87e3eb166047de9359e13d3472fbe4248545d1

Add lo_LA (Lao) dictionary fdo#71939



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 10 Commit Notification 2013-11-24 19:43:58 UTC
Andras Timar committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=f0d3fa4ad08ba0f9975f4a6f9f96ab7ac946871d

Add lo_LA (Lao) dictionary fdo#71939



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 11 Commit Notification 2013-11-24 19:45:15 UTC
Robert M Campbell committed a patch related to this issue.
It has been pushed to "libreoffice-4-2":

http://cgit.freedesktop.org/libreoffice/dictionaries/commit/?id=c4659a9bbbe9a35cedcf29c11631d78ec09ebd17&h=libreoffice-4-2

Add lo_LA (Lao) dictionary fdo#71939


It will be available in LibreOffice 4.2.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 12 Commit Notification 2013-11-24 19:47:34 UTC
Andras Timar committed a patch related to this issue.
It has been pushed to "libreoffice-4-2":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=565b5ab16aa6e3b7cfaccf0391d2d6f4c13269a3&h=libreoffice-4-2

Add lo_LA (Lao) dictionary fdo#71939


It will be available in LibreOffice 4.2.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.