A forum for feature requests/discussions and user submitted patches that improve MQ2
Moderator: MacroQuest Developers
-
renagade8
- orc pawn

- Posts: 28
- Joined: Sat Oct 19, 2002 9:17 pm
Post
by renagade8 » Mon Jul 07, 2003 3:00 pm
With the newest cvs version $char(buff,<buff name>) crashes me. It seems when the ini file stuff was implimented thats when it got messed up, because i have an older eqlib file prior to the ini file implimentation that does not crash when using this. I just wanna know if anyone else is having this trouble or if its just me.
- Ren
-
BlueSkies
- a ghoul

- Posts: 132
- Joined: Tue Oct 01, 2002 6:22 pm
Post
by BlueSkies » Tue Jul 08, 2003 1:42 am
I can verify the INI stuff didn't mess up the $char(buff) stuff
Val messed with the GetArg function -- perhaps he can shed some light on the subject. I certainly hope that's all it is...
Live your dreams! Blue Skies everyone
-
dont_know_at_all
- Developer

- Posts: 5450
- Joined: Sun Dec 01, 2002 4:15 am
- Location: Florida, USA
-
Contact:
Post
by dont_know_at_all » Tue Jul 08, 2003 4:40 am
Fixed in CVS. Here's the diffs:
Code: Select all
Index: EQLib_MacroParser.cpp
===================================================================
RCS file: /cvsroot/macroquest/macroquest/EQLib/EQLib_MacroParser.cpp,v
retrieving revision 1.5
diff -w -r1.5 EQLib_MacroParser.cpp
2186c2186
< if (pCharInfo->Buff[bf].SpellID != 0xFFFFFFFF) {
---
> if (pCharInfo->Buff[bf].SpellID != 0x0000FFFF) {
2195c2195
< if (pCharInfo->Buff[Buff-1].SpellID != 0xFFFFFFFF) {
---
> if (pCharInfo->Buff[Buff-1].SpellID != 0x0000FFFF) {
SpellID changed to from DWORD to WORD and no one changed the constants.
-
YKW-28983
- a hill giant

- Posts: 252
- Joined: Sun Dec 01, 2002 11:37 pm
Post
by YKW-28983 » Tue Jul 08, 2003 6:29 am
ah. was wondering why I was crashing 24/7 lately lol. thx for the notice and fast fix.
willl have to check this soon as I get home today got the new cvs recompiled but no time to test.
had some odd behavior with it telling me the EQLib file was not there had to open it in windows and then open up the cmd prompt so it would already be in the folder and worked after that..
-
YKW-28983
- a hill giant

- Posts: 252
- Joined: Sun Dec 01, 2002 11:37 pm
Post
by YKW-28983 » Tue Jul 08, 2003 11:57 am
not fixed. if the buff is not there it will crash otherwise it returns 15 again.. it's broke yet again.. wish there was two cvs's.. untouched and modded.. that way the mods can't screw with working version =\
and I agree things were working fine till the ini project got implemented.
-
BlueSkies
- a ghoul

- Posts: 132
- Joined: Tue Oct 01, 2002 6:22 pm
Post
by BlueSkies » Tue Jul 08, 2003 11:59 am
How bout next time you get what you consider to be a 'working copy,' just save it in another directory. That way, when changes come down the CVS pipeline, you can just look through the code and decide which changes you want to implement in your copy?
That's what I do. :)
Live your dreams! Blue Skies everyone
-
YKW-28983
- a hill giant

- Posts: 252
- Joined: Sun Dec 01, 2002 11:37 pm
Post
by YKW-28983 » Tue Jul 08, 2003 12:09 pm
Well aren't you special..
-
BlueSkies
- a ghoul

- Posts: 132
- Joined: Tue Oct 01, 2002 6:22 pm
Post
by BlueSkies » Tue Jul 08, 2003 12:36 pm
Er...
It was just a suggestion, dude, take it or leave it...
Live your dreams! Blue Skies everyone
-
dont_know_at_all
- Developer

- Posts: 5450
- Joined: Sun Dec 01, 2002 4:15 am
- Location: Florida, USA
-
Contact:
Post
by dont_know_at_all » Tue Jul 08, 2003 2:36 pm
Learn the ways of the -D.
Regular cvs is fixed but it will not be updated in the anonymous cvs until sometime later today, which is why I posted the diffs.
-
fryfrog
- a hill giant

- Posts: 271
- Joined: Fri Jun 20, 2003 5:37 am
Post
by fryfrog » Tue Jul 08, 2003 2:45 pm
by "-D" dkaa is suggesting you learn this "option" which allows you to check out any version of the mq source by date. so if you feel that mq from a week ago worked much better, just figure out the right -D option and get the source from 1 week ago. you can go back to the beginning if you like.
-
dont_know_at_all
- Developer

- Posts: 5450
- Joined: Sun Dec 01, 2002 4:15 am
- Location: Florida, USA
-
Contact:
Post
by dont_know_at_all » Tue Jul 08, 2003 4:46 pm
YKW-28983 wrote:not fixed. if the buff is not there it will crash otherwise it returns 15 again.. it's broke yet again.
I didn't see this behavior before or after I fixed my crash. Can you provide a specific example that returns 15?
-
renagade8
- orc pawn

- Posts: 28
- Joined: Sat Oct 19, 2002 9:17 pm
Post
by renagade8 » Tue Jul 08, 2003 4:51 pm
cool thanks dkaa works great with the fix
-ren
-
YKW-28983
- a hill giant

- Posts: 252
- Joined: Sun Dec 01, 2002 11:37 pm
Post
by YKW-28983 » Wed Jul 09, 2003 11:42 am
If the spell defined in quotations was on my toon it would return 15 however if the spell was no where to be found it would just crash me.