[not a bug] GMCP tables

User avatar
Heiko
Site Admin
Posts: 1548
Joined: Wed Mar 11, 2009 6:26 pm

Re: [not a bug] GMCP tables

Post by Heiko »

Verified the problem. Going to fix it.

Sanaki
Posts: 110
Joined: Wed Mar 09, 2011 1:30 am

Re: [not a bug] GMCP tables

Post by Sanaki »

Attempted to compile the newest git snapshot to check the fix, failed on this:

tmp/TCommandLine.o: In function `TCommandLine::spellCheck()':
/home/shared/bin/mudlet/src/TCommandLine.cpp:405: undefined reference to `Hunspell_spell'
tmp/TCommandLine.o: In function `TCommandLine::slot_popupMenu()':
/home/shared/bin/mudlet/src/TCommandLine.cpp:435: undefined reference to `Hunspell_free_list'
tmp/TCommandLine.o: In function `TCommandLine::mousePressEvent(QMouseEvent*)':
/home/shared/bin/mudlet/src/TCommandLine.cpp:445: undefined reference to `Hunspell_spell'
/home/shared/bin/mudlet/src/TCommandLine.cpp:448: undefined reference to `Hunspell_suggest'
tmp/TCommandLine.o: In function `TCommandLine':
/home/shared/bin/mudlet/src/TCommandLine.cpp:38: undefined reference to `Hunspell_create'
collect2: ld returned 1 exit status
make: *** [mudlet] Error 1

User avatar
Vadi
Posts: 5050
Joined: Sat Mar 14, 2009 3:13 pm

Re: [not a bug] GMCP tables

Post by Vadi »

You need to mod src.pro:


diff --git a/src/src.pro b/src/src.pro
index 1c23f91..5eb4769 100755
--- a/src/src.pro
+++ b/src/src.pro
@@ -13,7 +13,8 @@ exists(/usr/local/lib/libluajit-5.1.a):LIBLUA = -L/usr/local/l

unix:LIBS += -lpcre \
$$LIBLUA \
- -lyajl
+ -lyajl \
+ -lhunspell
win32:LIBS += -Lc:\mudlet_package \
-llua51 \
-lpcre \

User avatar
Heiko
Site Admin
Posts: 1548
Joined: Wed Mar 11, 2009 6:26 pm

Re: [not a bug] GMCP tables

Post by Heiko »

Linux build should be fixed now (as well as this gmcp issue).
To use the spell checker you need to copy your dictionary + aff file into mudlet/src (en_US.aff + en_US.dic) othwise they cannot be loaded at this point.

Sanaki
Posts: 110
Joined: Wed Mar 09, 2011 1:30 am

Re: [not a bug] GMCP tables

Post by Sanaki »

Didn't copy my .dic or .aff into the src directory (though en_US.aff was already in there), spellcheck works fine. GMCP also works fine. Was erroring the compile left and right until I realized hunspell wasn't installed at all somehow. Installed it, everything's clean now.

EDIT: I lied, I did copy the .dic in there. Ignore my rambling.
Last edited by Sanaki on Sun May 08, 2011 1:17 am, edited 1 time in total.

Sanaki
Posts: 110
Joined: Wed Mar 09, 2011 1:30 am

Re: [not a bug] GMCP tables

Post by Sanaki »

Right, something I hadn't considered before. Is there any way to tell what value was updated if the value sent and the prior value are identical? In this case, I'd like to be able to tell when I gain XP from quests, kills, etc, even if the xp gained doesn't increase my current total by any measurable amount. Previously, I could tell because the server would send only that value. A bit of a glamour feature rather than anything majorly functional either way, so I won't be heartbroken if the answer's no.

User avatar
Vadi
Posts: 5050
Joined: Sat Mar 14, 2009 3:13 pm

Re: [not a bug] GMCP tables

Post by Vadi »

The gmcp event is still raised

Sanaki
Posts: 110
Joined: Wed Mar 09, 2011 1:30 am

Re: [not a bug] GMCP tables

Post by Sanaki »

Aye, the gmcp.Char.Status event is raised. Was just wondering if with the table updating there would still be a way to identify which individual fields were updated. No sub-events are raised beyond gmcp.Char.Status, and a secondary argument on the function just returns the event name in a string again, unlike how it works with atcp.

User avatar
Vadi
Posts: 5050
Joined: Sat Mar 14, 2009 3:13 pm

Re: [not a bug] GMCP tables

Post by Vadi »

Oh. No, it doesn't say which field was updated then.

User avatar
Heiko
Site Admin
Posts: 1548
Joined: Wed Mar 11, 2009 6:26 pm

Re: [not a bug] GMCP tables

Post by Heiko »

The relevant sever sent event key is being raised just as with any other gmcp events. Adding the changed fields to the events would be inconsistent as it's not supported in non-merge events either.

Post Reply