Go Back   EQEmulator Home > EQEmulator Forums > Development > Development::Server Code Submissions

Reply
 
Thread Tools Display Modes
  #1  
Old 07-25-2008, 03:28 PM
Derision
Developer
 
Join Date: Feb 2004
Location: UK
Posts: 1,540
Default

After looking over this again, the call to SendPetBuffsToClient in Mob::BuffFadeByEffect is redundant, since BuffFadeByEffect calls BuffFadeBySlot, and the pet window buff packet is already sent in that routine.

So, after applying the patch, you can delete these lines:

spells.cpp, Line 2614:
Code:
void Mob::BuffFadeByEffect(int effectid, int skipslot)
{
        int i;

        for(i = 0; i < BUFF_COUNT; i++)
        {
                if(buffs[i].spellid == SPELL_UNKNOWN)
                        continue;
                if(IsEffectInSpell(buffs[i].spellid, effectid) && i != skipslot)
                        BuffFadeBySlot(i, false);
        }
        // Delete the following 3 lines as they are redundant.
        if(IsPet() && GetOwner() && GetOwner()->IsClient()) {    
                SendPetBuffsToClient();                                     
        }                                                                           
        //we tell BuffFadeBySlot not to recalc, so we can do it only once when were done
        CalcBonuses();
}
Reply With Quote
  #2  
Old 07-25-2008, 04:07 PM
So_1337
Dragon
 
Join Date: May 2006
Location: Cincinnati, OH
Posts: 689
Default

Another great submission. If you keep this up, we're going to have a whole slew of functionality that wasn't there before just by connecting a few missing op_codes. Keep up the great work!
Reply With Quote
  #3  
Old 07-25-2008, 04:17 PM
KLS
Administrator
 
Join Date: Sep 2006
Posts: 1,348
Default

Nice, I'll get this all sorted out and in submission in a few days.
Reply With Quote
  #4  
Old 07-25-2008, 04:27 PM
Angelox
AX Classic Developer
 
Join Date: May 2006
Location: filler
Posts: 2,049
Default

Wow, thought i'd never see the buffs on pets again - Thanks again!
Reply With Quote
  #5  
Old 07-25-2008, 08:06 PM
joligario's Avatar
joligario
Developer
 
Join Date: Mar 2003
Posts: 1,505
Default

I remember on live not being able to click off beneficial spells from pet. So its all good. Good work.
Reply With Quote
  #6  
Old 07-27-2008, 05:55 PM
KLS
Administrator
 
Join Date: Sep 2006
Posts: 1,348
Default

Something I'm changing: not sending a pet buff update every client buff tic. I don't see the point really, also adding a pet buff update at connect finishing since without the buff tic update it wont show up on zoning without it.

Save roughly 2k/min per client connected this way which may not be a lot but if we keep adding things like it because they're "not a lot" they add up.
Reply With Quote
  #7  
Old 07-27-2008, 06:22 PM
trevius's Avatar
trevius
Developer
 
Join Date: Aug 2006
Location: USA
Posts: 5,946
Default

Thanks KLS! Every bit helps! Sounds like some really nice adjustments.
__________________
Trevazar/Trevius Owner of: Storm Haven
Everquest Emulator FAQ (Frequently Asked Questions) - Read It!
Reply With Quote
Reply


Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

   

All times are GMT -4. The time now is 01:58 PM.


 

Everquest is a registered trademark of Daybreak Game Company LLC.
EQEmulator is not associated or affiliated in any way with Daybreak Game Company LLC.
Except where otherwise noted, this site is licensed under a Creative Commons License.
       
Powered by vBulletin®, Copyright ©2000 - 2026, Jelsoft Enterprises Ltd.
Template by Bluepearl Design and vBulletin Templates - Ver3.3