[
Log in
/
Register
]
|
News
|
Chat
|
Pastebin
|
Donations
|
Tutorials
|
Rules
|
Forums
|
|
Maps
|
Skins
|
Icons
|
Models
|
Spells
|
Tools
|
Jass
|
Packs
|
Hosted Projects
|
Starcraft II Modding
|
Starcraft II Resources
|
Galaxy Wiki
|
(Keeps Hive Alive)
The Hive Workshop
>
Spells
Lightning Storm(new)
Remember Me?
Register
FAQ
Members List
Calendar
Mark Forums Read
Page 2 of 2
<
1
2
Thread Tools
09-14-2009, 09:39 PM
#
16
(
permalink
)
Chief-Oblivion
2D Artist In Training
Join Date: Mar 2008
Posts: 557
Hell if I know, I know how to find them not how to fix them. ;D That's why I don't spell make anymore. Plus it's alot of work.
Chief-Oblivion
View Public Profile
Send a private message to Chief-Oblivion
Find More Posts by Chief-Oblivion
09-15-2009, 05:11 PM
#
17
(
permalink
)
deth_lord
Dark Mapper
Join Date: Jun 2009
Posts: 47
true that bro T.T
but i just updated spell. it uses some local vars now (even though there was no problems i'll still do it for you guys *rolls eyes*) :P
__________________
deth_lord
View Public Profile
Send a private message to deth_lord
Find More Posts by deth_lord
09-19-2009, 09:31 PM
#
18
(
permalink
)
Hanky
Are you a bot!?
Join Date: Jan 2007
Posts: 574
*cough* null locals *cough*
Also having a periodic trigger all time active isn't a good thing...
__________________
Quote:
Originally Posted by
cvb1
where can i get GUI
Hanky
View Public Profile
Send a private message to Hanky
Visit Hanky's homepage!
Find More Posts by Hanky
09-19-2009, 11:41 PM
#
19
(
permalink
)
Darkyvm
Noxus Hunter
Join Date: May 2007
Posts: 214
Review
Reviewed by
Darkyvm
Reviewed Version
Lightning Storm(new)
Reviewing Points
[+]
MUI.
[+]
Special effects used fit the spell nicely. The random lightning storms around the map are also a nice bonus.
[+]
Spell runs smoothly and doesn't lag, even when there are many being cast at once.
[+]
Good, simple, and easy to understand readme section.
[+-] The lightning looks a bit weird if you cast it somewhere far away.
[+-] Some BJs used in custom script could've been inlined.
[-]
After you remove locations / destroy groups, the leak is still there, you still need to null the variable.
[-]
Can't modify damage / radius vaules without having to edit the spell's core.
[-]
Not so original, already seen some similar spells somewhere around here.
Suggestions
Inline the BJs to increase efficiency, and if possible, make the spell's damage / radius vaules easier to modify, some people might not be able to identify where are those vaules in the code and not be able to change them (yes, believe me, there are people that can't even understand a simple GUI code, you have to think about everyone that might want to use your spell). Also, null the locals.
Rating
3
/
5
- Good.
Darkyvm
View Public Profile
Send a private message to Darkyvm
Find More Posts by Darkyvm
09-21-2009, 10:40 PM
#
20
(
permalink
)
deth_lord
Dark Mapper
Join Date: Jun 2009
Posts: 47
if you can't figure out simple gui you probably shouldn't be mapping :/ but alas, i will probably change it.
__________________
deth_lord
View Public Profile
Send a private message to deth_lord
Find More Posts by deth_lord
Page 2 of 2
<
1
2
Bookmarks
Digg
del.icio.us
StumbleUpon
Google
«
Previous Thread
|
Next Thread
»
Thread Tools
Show Printable Version
Email this Page
Posting Rules
You
may not
post new threads
You
may not
post replies
You
may not
post attachments
You
may not
edit your posts
BB code
is
On
Smilies
are
On
[IMG]
code is
On
HTML code is
Off
Trackbacks
are
Off
Pingbacks
are
Off
Refbacks
are
Off
Forum Rules
Similar Threads
Thread
Thread Starter
Forum
Replies
Last Post
[
Trigger
]
Lightning Storm Spell
jonhpath
Triggers & Scripts
22
06-27-2009
12:23 AM
Effect wanted: Lightning Storm
Teelo
Requests
7
05-02-2009
07:09 AM
Lightning Storm Model
xXx-Tricky-xXx
Requests
0
10-31-2008
03:40 AM
All times are GMT. The time now is
05:12 PM
.
-- Low Graphics
---- Night Elf
---- Terran (Beta)
Contact Us
-
The Hive Workshop - The Warcraft 3 Modding Site
-
Archive
-
Top
Powered by vBulletin
Copyright 2000 - 2008, Jelsoft Enterprises Ltd.
Search Engine Optimization by
vBSEO
3.5.1 PL2
Copyright © Ralle