![]() |
#1 | ||
Procrastination Incarnate
Development Director
|
![]()
![]() Last edited by Anitarf : 08-04-2009 at 11: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?
__________________![]() 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 10: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 |
|
|