ZDoom

[url=http://forum.zdoom.org/viewtopic.php?f=2&t=52893]And that's what happens whenever SourceTree eats some files.[/url]

constants.txt wasn't updated to include the new flags.

[url=https://github.com/rheit/zdoom/pull/729]Commit screwiness fixed.[/url]

Okay, that's done now. The P_LineAttack will only spawn if both a projectile name is supplied and the missile actually spawns.

Anything else?

No. But you call that stuff even when you do not need it. Do it inside a check for spawning a projectile.

So back to the old requirement of +ALWAYSPUFF then?

			if (!puff)
			{
				temp = true;
				puff = P_LineAttack(self, bangle, range, bslope, 0, NAME_Hitscan, pufftype, laflags | LAF_NOINTERACT);
			}

I think this is mostly ok, but there's one problem:

This code
[code]
if (!puff)
{
temp = true;
puff = P_LineAttack(self, bangle, range, bslope, 0, NAME_Hitscan, pufftype, laflags | LAF_NOINTERACT);
}
[/code]

which appears to be exclusively for the bullet tracer parts is called unconditionally. This needs to be changed. P_LineAttack is a very expensive call that should be avoided if not needed.

No, but the last change was just before I left for vacation. Will have to review this a bit before merging.

Is this still waiting by any chance?

Since I hadn't actually responded yet: code seems good to me after the latest commit, FWIW.

Ah, keen eyes Xaser. Thanks!

And fixed.

Finally got a chance to sit down play around with this, and I noticed a couple of issues:

[list]
[*]thingdef_codeptr.cpp line 1652: "bangle" and "bslope" should be "angle" and "slope"
[*]thingdef_codeptr.cpp line 1932: "bangle" and "bslope" should be "angle" and "slope"
[*]p_map.cpp line 4099: should use single &, not double &&[/list]

Here's a [url=http://static.angryscience.net/pub/doom/xatest/zd_bullet-tracer.wad]quick test wad[/url] showcasing the oddities that result. There's a Chaingun and Rocket Launcher replacement in slots 4 and 5 (summon "ChaingunTest" and "WeirdGun", respectively"). Both weapons' primary fires exhibit aim issues (point at an enemy or the sky), and the chaingun's secondary fire deals no damage since the double && accidentally sets the NOINTERACT flag on the real puff.

I fixed the 3 issues in my local environment, and after that, the test wad (and the rest of my mucking about) all works as expected.

Nothing else seems awry, and the feature's fun to play with, so "whoop!" there. :P

Puffs no longer require +ALWAYSPUFF. Also cleaned up the code quite a bit in A_FireBullets and A_CustomBulletAttack.

@StrikerMan780: This is meant to replace that kind of stuff indeed.

I actually made clever, and quite clean tracer code a while back, no ACS required. Just A_Warp and some switching of pointers. Works with weapons that have spread. However, a native implementation would be nice, since there's two downsides to my simple implementation: Offsets don't consider floor clipping, nor crouching. It also doesn't adjust if different player classes have different shooting heights. However, if you don't need to worry about that last one, it's fine in most cases.

If anyone wants to know how I did it, let me know and I'll post examples later. (Currently very busy).

Not to mention you'd have to call A_CheckSight just to kill off the tracer properly, too. A_CheckLOF cannot return the distance from something it hits unfortunately...