|
|
#1 | ||
|
Procrastination Incarnate
Development Director
|
Spell code:Last edited by Anitarf : 08-04-2009 at 10:16 PM. |
||
|
|
|
| Sponsored Links - Login to hide this ad! |
|
|
|
|
#3 |
|
Free Software Terrorist
Technical Director
|
Moyack, you should already have had WarCiTy in your system.
__________________ |
|
|
|
|
|
#4 | |
|
Procrastination Incarnate
Development Director
|
Quote:
|
|
|
|
|
|
|
#5 |
|
Superuser
|
Very impressive.
__________________ |
|
|
|
|
|
#6 | |
|
I blink, therefore I am.
Join Date: Sep 2006
Posts: 1,812
![]() ![]() ![]()
|
Quote:
Ps: for the record, you have permission :P |
|
|
|
|
|
|
#7 | |
|
Procrastination Incarnate
Development Director
|
Quote:
|
|
|
|
|
|
|
#9 |
|
Procrastination Incarnate
Development Director
|
I can't use it because his GroupEnumUnitsInRangeOfSeg function doesn't take unit's collision sizes into account and if I wanted to do that without inlining his code I'd have to deal with locations; so I inlined his code instead for the sake of efficiency.
__________________ |
|
|
|
|
|
#10 |
|
I blink, therefore I am.
Join Date: Sep 2006
Posts: 1,812
![]() ![]() ![]()
|
Not asking for you to not in-line it, but wouldn't adding the correction factor to the range accomplish the exact same thing?
__________________ JASS:GetEnumUnitsInRangeOfSeg(arguments..., distance + collisionCorrection) In-lining is totally fine with me as performance with spells is important. |
|
|
|
|
|
#11 |
|
Obscurity, the Art
|
CollisionCorrection as you have it would be unit-type dependent, something that can't be accessed in JASS without hardcoding it or using a processing-hog function. Performing IsUnitInRange..(...) checks guarantees that a unit's collision is properly factored.
__________________Anyways, you should probably update LineSegments to include collision checks and I think Anitarf should then update this to use LineSegments. It seems counter-productive to shoot for modularity as a site scripting goal if we're just inlining other peoples' codes because they lack a very specific feature that they should have anyways. |
|
|
|
|
|
#12 |
|
retired coder | real ilfe
|
Impressive in deed. However I fail to understand why it doesn't work sometimes ....
When I tested the spell, there was a time I had a unit in the wire and yet nothing happened. Problem is that this "there was a time" gets repeated quite often ... here is the replay, I hope it helps: Last edited by Flame_Phoenix : 12-15-2008 at 09:42 PM. |
|
|
|
|
|
#13 |
|
Procrastination Incarnate
Development Director
|
Updated, the spell now uses the LineSegment library which also fixes the bug that Flame_Phoenix reported.
__________________ |
|
|
|
|
|
#15 |
|
Procrastination Incarnate
Development Director
|
Updated the test map so it works in 1.24 (had to import the new Table and TimerUtils, everything else remains the same).
__________________ |
|
|
|
![]() |
| Thread Tools | Search this Thread |
|
|
|
Donate |