Forum posts

void CMjolnir::PrimaryAttack()

m_pPlayer->m_rgAmmo[m_iPrimaryAmmoType]--;

If you want to have multiple lines of code depend on an if statement, then you have to surround them with curly braces:

 {
		                 m_flFreeammotime = gpGlobals->time + RANDOM_FLOAT(1, 5);// New code
		                 if (gpGlobals->time >= m_flFreeammotime) //New code
			{
		     		UTIL_ClientPrintAll( HUD_PRINTNOTIFY, "AUTORELOAD\n");//New code
				    m_pPlayer->m_rgAmmo[m_iPrimaryAmmoType] += 99;//Old code
			}
          }

if (FBitSet(m_pPlayer->pev->flags, FL_FAKECLIENT)&& (m_pPlayer->m_rgAmmo[m_iPrimaryAmmoType] <= 0)/*Old Code*/
            && (m_flFreeammotime <= 0)) //New code
 {            m_flFreeammotime = gpGlobals->time + RANDOM_FLOAT(1, 5);       } //New code

if (gpGlobals->time >= m_flFreeammotime) //New code
			{
		     		UTIL_ClientPrintAll( HUD_PRINTNOTIFY, "AUTORELOAD\n");//New code
				    m_pPlayer->m_rgAmmo[m_iPrimaryAmmoType] += 99;//Old code
			}

			{
                    if (gpGlobals->time >= m_flFreeammotime) //New code
		     		UTIL_ClientPrintAll( HUD_PRINTNOTIFY, "AUTORELOAD\n");//New code
				    m_pPlayer->m_rgAmmo[m_iPrimaryAmmoType] += 99;//Old code
			}

           //==================================================================
          // Bots usually cannot reload because trolley_monster cannot reach
          // them, so we give them a little ammo.
          //==================================================================
	     if (FBitSet(m_pPlayer->pev->flags, FL_FAKECLIENT)&& (m_pPlayer->m_rgAmmo[m_iPrimaryAmmoType] <= 0)/*Old Code*/
            && (m_flFreeammotime <= 0)) //New code
	      {
		                 m_flFreeammotime = gpGlobals->time + RANDOM_FLOAT(1, 5);// New code
		                 if (gpGlobals->time >= m_flFreeammotime) //New code
			{
		     		UTIL_ClientPrintAll( HUD_PRINTNOTIFY, "AUTORELOAD\n");//New code
				    m_pPlayer->m_rgAmmo[m_iPrimaryAmmoType] += 99;//Old code
			}
          }

//New code

m_flAutoReloadTime = 0;

I'm not too familiar with the HL SDK

Checking whether m_flstufftime is 0 or less only makes sense if it represents the time that is left until something needs to be done.

m_flstufftime is not a very descriptive name. Something like m_flCountdownTime would be more clear.

	if (condition)
	{
	        while ( m_flstufftime < gpGlobals->time )
				 {
					m_flstufftime += RANDOM_FLOAT (0.025f, 0.01f); // Decrease 1% every fixed random seconds from 1 to 10 if I am not wrong

					if ( m_flstufftime <= 0 )
					{
						dosomething;
		                 	}
				}
}

	if (condition)
	{
	       	m_flstufftime = gpGlobals->time + RANDOM_FLOAT (5, 10); ;
					if ( m_flstufftime <= 0 )
					{
						dosomething;
					}
}

A monster following another monster? Like, actively? I'm not sure that you could. Unless you faked it or the monster was trying to kill the other...

So, if you want better fps, learn graphics programming and rewrite a part of, if not the whole renderer to work more efficiently. Otherwise, don't make high-poly models.

replacing the engine window with a custom one, redirecting engine functions so you handle it in client.dll, only when VAC is disabled of course just like when you use custom opengl32.dll files

compiling dlls is "quite hard"