08ad7d7f4e
penev discovered that the RulesetLoaded functions of projectiles were never being called, meaning that their blocking calculations were not properly accounting for actors with large hitboxes. The best fix for this is to change FindActorsOnLine to always account for the largest actor's hit radius, rather than forcing callers to pass the largest radius. Per the comment in Util.cs, as a result, move this computation to ActorMap. I decided to simplify by not making a separate calculation for actors that block projectiles only; this may cause a small performance degradation as the search space is a bit larger. Similarly to this, I've removed the ability to specify a search radius manually. Because this is only a search radius, setting a value smaller than the largest eligible actor makes no sense; that would lead to completely inconsistent blocking. Setting a larger value, on the other hand, would make no difference. CreateEffectWarhead was the only place in core code any of these search radii were set, and that's because 0 was a mysterious magic value that made the warhead incapable of hitting actors. I replaced it with a boolean flag that more clearly indicates the actual behaviour. Fixes #14151.
86 lines
2.6 KiB
C#
86 lines
2.6 KiB
C#
#region Copyright & License Information
|
|
/*
|
|
* Copyright 2007-2018 The OpenRA Developers (see AUTHORS)
|
|
* This file is part of OpenRA, which is free software. It is made
|
|
* available to you under the terms of the GNU General Public License
|
|
* as published by the Free Software Foundation, either version 3 of
|
|
* the License, or (at your option) any later version. For more
|
|
* information, see COPYING.
|
|
*/
|
|
#endregion
|
|
|
|
using System.Collections.Generic;
|
|
using System.Linq;
|
|
using OpenRA.GameRules;
|
|
using OpenRA.Graphics;
|
|
using OpenRA.Mods.Common.Traits;
|
|
using OpenRA.Traits;
|
|
|
|
namespace OpenRA.Mods.Common.Projectiles
|
|
{
|
|
[Desc("Simple, invisible, usually direct-on-target projectile.")]
|
|
public class InstantHitInfo : IProjectileInfo
|
|
{
|
|
[Desc("Maximum offset at the maximum range.")]
|
|
public readonly WDist Inaccuracy = WDist.Zero;
|
|
|
|
[Desc("Projectile can be blocked.")]
|
|
public readonly bool Blockable = false;
|
|
|
|
[Desc("The width of the projectile.")]
|
|
public readonly WDist Width = new WDist(1);
|
|
|
|
[Desc("Scan radius for actors with projectile-blocking trait. If set to a negative value (default), it will automatically scale",
|
|
"to the blocker with the largest health shape. Only set custom values if you know what you're doing.")]
|
|
public WDist BlockerScanRadius = new WDist(-1);
|
|
|
|
public IProjectile Create(ProjectileArgs args) { return new InstantHit(this, args); }
|
|
}
|
|
|
|
public class InstantHit : IProjectile
|
|
{
|
|
readonly ProjectileArgs args;
|
|
readonly InstantHitInfo info;
|
|
|
|
Target target;
|
|
WPos source;
|
|
|
|
public InstantHit(InstantHitInfo info, ProjectileArgs args)
|
|
{
|
|
this.args = args;
|
|
this.info = info;
|
|
source = args.Source;
|
|
|
|
if (args.Weapon.TargetActorCenter)
|
|
target = args.GuidedTarget;
|
|
else if (info.Inaccuracy.Length > 0)
|
|
{
|
|
var inaccuracy = Util.ApplyPercentageModifiers(info.Inaccuracy.Length, args.InaccuracyModifiers);
|
|
var maxOffset = inaccuracy * (args.PassiveTarget - source).Length / args.Weapon.Range.Length;
|
|
target = Target.FromPos(args.PassiveTarget + WVec.FromPDF(args.SourceActor.World.SharedRandom, 2) * maxOffset / 1024);
|
|
}
|
|
else
|
|
target = Target.FromPos(args.PassiveTarget);
|
|
}
|
|
|
|
public void Tick(World world)
|
|
{
|
|
// Check for blocking actors
|
|
WPos blockedPos;
|
|
if (info.Blockable && BlocksProjectiles.AnyBlockingActorsBetween(world, source, target.CenterPosition,
|
|
info.Width, out blockedPos))
|
|
{
|
|
target = Target.FromPos(blockedPos);
|
|
}
|
|
|
|
args.Weapon.Impact(target, args.SourceActor, args.DamageModifiers);
|
|
world.AddFrameEndTask(w => w.Remove(this));
|
|
}
|
|
|
|
public IEnumerable<IRenderable> Render(WorldRenderer wr)
|
|
{
|
|
return Enumerable.Empty<IRenderable>();
|
|
}
|
|
}
|
|
}
|