wc3campaigns
WC3C Homepage - www.wc3c.netUser Control Panel (Requires Log-In)Engage in discussions with other users and join contests in the WC3C forums!Read one of our many tutorials, ranging in difficulty from beginner to advanced!Show off your artistic talents in the WC3C Gallery!Download quality models, textures, spells (vJASS/JASS), systems, and scripts!Download maps that have passed through our rigorous approval process!

Go Back   Wc3C.net > Warcraft III Modding > Developer's Corner > General Development
User Name
Password
Register Rules Get Hosted! Chat Pastebin FAQ and Rules Members List Calendar



Reply
 
Thread Tools Search this Thread
Old 08-19-2009, 10:44 AM   #1
Anachron
User
 
Anachron's Avatar
 
Join Date: Mar 2009
Posts: 1,079

Anachron will become famous soon enough (51)Anachron will become famous soon enough (51)

Default What to change on Wc3

Hello all. Since the new Patch (1.24) has been released, I think its time
to ask for other fixes.

I am collecting a list of signatures to get blizzard to change their code,
here are some suggestions I made:
  • Camera
    1. SetCameraTargetPositionX
    2. SetCameraTargetPositionY
    3. SetCameraTargetPositionZ
    4. SetCameraEyePositionX
    5. SetCameraEyePositionY
    6. SetCameraEyePositionZ
  • Arrays
    1. Index > 8191
    2. 2D Arrays
    3. Assoziative Indexing
  • Items
    1. GetItemGoldCost()
  • Trackables
    1. add GetTriggerPlayer()
    2. add RemoveTrackable()
    3. add MoveTrackable()
  • Lightning
    1. SetLightningX1()
    2. SetLightningX2()
    3. SetLightningY1()
    4. SetLightningY2()
  • Events
    1. TriggerClearEvents()
    2. EVENT_PLAYER_UNIT_DAMAGED
    3. GetUnitDamage()
  • Location / Coordinates
    1. GetPointZ()
  • Heroes
    1. More then 5 Hero Skills possible
  • Minimap
    1. HideMiniMap()
    2. SetMiniMap(PATH)
    3. DisableCameraReset()
  • Information
    1. about SetUnitLoc()
    2. about the "return-bug"

Other:
Please give other suggestions, and please remember, they have to be REALISTIC!
__________________
CustomInventory [Discussion - Download] - Got Directors Cut!
CustomMissle [Discussion - [Download (not yet)] - In development!
Other systems [Spawn System] [Move System] [CustomBar] [SpellBar]

Last edited by Anachron : 08-24-2009 at 04:13 PM.
Anachron is offline   Reply With Quote
Sponsored Links - Login to hide this ad!
Old 08-19-2009, 10:55 AM   #2
Gorman
.▲▼▲&#9
 
Gorman's Avatar
 
Join Date: Apr 2007
Posts: 1,087

Submissions (2)

Gorman will become famous soon enough (70)Gorman will become famous soon enough (70)Gorman will become famous soon enough (70)

Send a message via MSN to Gorman
Default

RemoveMemoryLeaks()
__________________
Gorman is offline   Reply With Quote
Old 08-19-2009, 11:11 AM   #3
Switch33
User
 
Join Date: Oct 2003
Posts: 150

Switch33 is on a distinguished road (13)

Default

Quote:
Originally posted by Gorman:
RemoveMemoryLeaks()

Wow pure realism. . .



How bout fix hashtable problems? Since they didn't even get that right.

Also. . . it's sort of funny how some maps suddenly don't work that use GUI since the way it was coded was changed.(On blizzard noticing this happened. They will be probably more reluctant to change stuff too much as it may change coding and break mapz like they already did. )

Last edited by Switch33 : 08-19-2009 at 11:14 AM.
Switch33 is offline   Reply With Quote
Old 08-19-2009, 11:41 AM   #4
Anachron
User
 
Anachron's Avatar
 
Join Date: Mar 2009
Posts: 1,079

Anachron will become famous soon enough (51)Anachron will become famous soon enough (51)

Default

Memory leaks can not be cleaned, since you don't know if you need the data or not.

Its good that I didn't use Hashtables, I knew blizzard would create bugs there aswell.
__________________
CustomInventory [Discussion - Download] - Got Directors Cut!
CustomMissle [Discussion - [Download (not yet)] - In development!
Other systems [Spawn System] [Move System] [CustomBar] [SpellBar]
Anachron is offline   Reply With Quote
Old 08-19-2009, 11:41 AM   #5
Toadcop
BuranX
 
Toadcop's Avatar
 
Join Date: Jul 2006
Posts: 1,886

Submissions (4)

Toadcop is just really nice (299)Toadcop is just really nice (299)

Approved Map: TcXSpell Making Session 10 Winner

Send a message via ICQ to Toadcop
Default

afaik today 1.24b will be released O_O so...
__________________
Toadcop is offline   Reply With Quote
Old 08-19-2009, 11:45 AM   #6
0zyx0
Perfectionist noob
 
0zyx0's Avatar
 
Join Date: Mar 2009
Posts: 255

0zyx0 will become famous soon enough (38)0zyx0 will become famous soon enough (38)

Default

SetLightningX1(), SetLightningX2(), SetLightningY1(), etc...
GetItemGoldCost()
TriggerClearEvents()
GetUnitDamage()
EVENT_PLAYER_UNIT_DAMAGED

And most important:
GetPointZ(x, y)
__________________
My new signature - Now easier to understand than ever!
0zyx0 is offline   Reply With Quote
Old 08-19-2009, 11:45 AM   #7
Anachron
User
 
Anachron's Avatar
 
Join Date: Mar 2009
Posts: 1,079

Anachron will become famous soon enough (51)Anachron will become famous soon enough (51)

Default

They surely will NOT fix the Trackables, also there are many other things mappers want.

Edit: That was for Toadcops post.

I updated the list.
__________________
CustomInventory [Discussion - Download] - Got Directors Cut!
CustomMissle [Discussion - [Download (not yet)] - In development!
Other systems [Spawn System] [Move System] [CustomBar] [SpellBar]

Last edited by Anachron : 08-19-2009 at 11:48 AM.
Anachron is offline   Reply With Quote
Old 08-19-2009, 11:50 AM   #8
Pyrogasm
Lackadaisically Absent.
 
Pyrogasm's Avatar


Respected User
 
Join Date: Sep 2006
Posts: 4,523

Submissions (9)

Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)

Hero Contest - Fourth place

Send a message via ICQ to Pyrogasm Send a message via AIM to Pyrogasm Send a message via MSN to Pyrogasm Send a message via Yahoo to Pyrogasm
Default

Quote:
Originally Posted by 0zyx0
SetLightningX1(), SetLightningX2(), SetLightningY1()
Collapse JASS:
native MoveLightning                takes lightning whichBolt, boolean checkVisibility, real x1, real y1, real x2, real y2 returns boolean
native MoveLightningEx              takes lightning whichBolt, boolean checkVisibility, real x1, real y1, real z1, real x2, real y2, real z2 returns boolean
What's wrong with those?
__________________
Quote:
Originally posted by Rising_Dusk
Your spells are mostly ignored because they are not very cool so we aren't very excited to review/approve them, but you are incredibly persistent and won't give us an excuse to graveyard it. That is generally what results in a resource being ignored for a long time.

The Spell Request Thread Done for, unless someone else wants to revive it...
It lasted a damn long time.

Please; Ask for Help Appropriately














Quote:
Originally posted by Kyrbi0
Huh. Almost makes me wish I had a girlfriend, to take advantage of today (wait, no, that's not what I meant... I mean, take advantage of the fact that it is international women's day... gah, never mind).
Quote:
Originally posted by Pyrogasm
Rome may not have been built in a day, but the Romans sure as hell didn't say "look at this great city we built guys!" when they had nothing more than a bit of stone and some cottages.
Pyrogasm is offline   Reply With Quote
Old 08-19-2009, 11:52 AM   #9
Anachron
User
 
Anachron's Avatar
 
Join Date: Mar 2009
Posts: 1,079

Anachron will become famous soon enough (51)Anachron will become famous soon enough (51)

Default

Quote:
Originally Posted by Pyrogasm
Collapse JASS:
native MoveLightning                takes lightning whichBolt, boolean checkVisibility, real x1, real y1, real x2, real y2 returns boolean
native MoveLightningEx              takes lightning whichBolt, boolean checkVisibility, real x1, real y1, real z1, real x2, real y2, real z2 returns boolean
What's wrong with those?
They call to much actions.
(Its the same why you don't use BJs at all, aslong as they are bad)
__________________
CustomInventory [Discussion - Download] - Got Directors Cut!
CustomMissle [Discussion - [Download (not yet)] - In development!
Other systems [Spawn System] [Move System] [CustomBar] [SpellBar]
Anachron is offline   Reply With Quote
Old 08-19-2009, 11:53 AM   #10
Pyrogasm
Lackadaisically Absent.
 
Pyrogasm's Avatar


Respected User
 
Join Date: Sep 2006
Posts: 4,523

Submissions (9)

Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)Pyrogasm is a splendid one to behold (638)

Hero Contest - Fourth place

Send a message via ICQ to Pyrogasm Send a message via AIM to Pyrogasm Send a message via MSN to Pyrogasm Send a message via Yahoo to Pyrogasm
Default

...

That's retarded. Why don't we want a MoveLocationX() or MoveLocationY(), then? Why not break up every function so that we can have every permutation of the possible arguments just for the sake of some arbitrary 'faster'?

Hell, I've never even heard of anyone complaining about MoveLightningEx() being too slow.

Ever.
__________________
Quote:
Originally posted by Rising_Dusk
Your spells are mostly ignored because they are not very cool so we aren't very excited to review/approve them, but you are incredibly persistent and won't give us an excuse to graveyard it. That is generally what results in a resource being ignored for a long time.

The Spell Request Thread Done for, unless someone else wants to revive it...
It lasted a damn long time.

Please; Ask for Help Appropriately














Quote:
Originally posted by Kyrbi0
Huh. Almost makes me wish I had a girlfriend, to take advantage of today (wait, no, that's not what I meant... I mean, take advantage of the fact that it is international women's day... gah, never mind).
Quote:
Originally posted by Pyrogasm
Rome may not have been built in a day, but the Romans sure as hell didn't say "look at this great city we built guys!" when they had nothing more than a bit of stone and some cottages.

Last edited by Pyrogasm : 08-19-2009 at 11:53 AM.
Pyrogasm is offline   Reply With Quote
Old 08-19-2009, 11:59 AM   #11
0zyx0
Perfectionist noob
 
0zyx0's Avatar
 
Join Date: Mar 2009
Posts: 255

0zyx0 will become famous soon enough (38)0zyx0 will become famous soon enough (38)

Default

The problem with MoveLightningEx() is that you have to know the old position of the lightning when only one coordinate has to change. Same problem with SetLightningColor().
__________________
My new signature - Now easier to understand than ever!
0zyx0 is offline   Reply With Quote
Old 08-19-2009, 12:02 PM   #12
Earth-Fury
Two Blue
 
Earth-Fury's Avatar
 
Join Date: Mar 2003
Posts: 2,179

Submissions (7)

Earth-Fury is just really nice (343)Earth-Fury is just really nice (343)Earth-Fury is just really nice (343)Earth-Fury is just really nice (343)

Send a message via MSN to Earth-Fury
Default

Quote:
Originally Posted by 0zyx0
The problem with MoveLightningEx() is that you have to know the old position of the lightning when only one coordinate has to change. Same problem with SetLightningColor().
That would be better solved with:

MoveLightningSource(lit,x,y,z)
MoveLightningDest(lit,x,y,z)

than a function per component...
__________________
Useful Links:
My Resources:
Good Resources:
Search
Tutorials
Ability Guide

UnitMaxState
BonusMod

AutoIndex
ARGB
TimerUtils

Earth-Fury is offline   Reply With Quote
Old 08-19-2009, 01:23 PM   #13
Anachron
User
 
Anachron's Avatar
 
Join Date: Mar 2009
Posts: 1,079

Anachron will become famous soon enough (51)Anachron will become famous soon enough (51)

Default

Quote:
Originally Posted by Pyrogasm
...

That's retarded. Why don't we want a MoveLocationX() or MoveLocationY(), then? Why not break up every function so that we can have every permutation of the possible arguments just for the sake of some arbitrary 'faster'?

Hell, I've never even heard of anyone complaining about MoveLightningEx() being too slow.

Ever.
Hmm, but whats so bad about having one native to change some handle value?
__________________
CustomInventory [Discussion - Download] - Got Directors Cut!
CustomMissle [Discussion - [Download (not yet)] - In development!
Other systems [Spawn System] [Move System] [CustomBar] [SpellBar]
Anachron is offline   Reply With Quote
Old 08-19-2009, 01:48 PM   #14
Gorman
.▲▼▲&#9
 
Gorman's Avatar
 
Join Date: Apr 2007
Posts: 1,087

Submissions (2)

Gorman will become famous soon enough (70)Gorman will become famous soon enough (70)Gorman will become famous soon enough (70)

Send a message via MSN to Gorman
Default

Quote:
Originally Posted by Anachron
Memory leaks can not be cleaned, since you don't know if you need the data or not.
It uses magic.
__________________
Gorman is offline   Reply With Quote
Old 08-19-2009, 01:51 PM   #15
Anachron
User
 
Anachron's Avatar
 
Join Date: Mar 2009
Posts: 1,079

Anachron will become famous soon enough (51)Anachron will become famous soon enough (51)

Default

Really, be serious. I mean I wish that too, but it could never be possible.
So, any serious things you want to have?
__________________
CustomInventory [Discussion - Download] - Got Directors Cut!
CustomMissle [Discussion - [Download (not yet)] - In development!
Other systems [Spawn System] [Move System] [CustomBar] [SpellBar]
Anachron is offline   Reply With Quote
Reply


Thread Tools Search this Thread
Search this Thread:

Advanced Search

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off


All times are GMT. The time now is 08:20 PM.


Affiliates
The Hubb The JASS Vault Clan WEnW Campaign Creations Clan CBS GamesModding Flixreel Videos

Powered by vBulletin (Copyright ©2000 - 2019, Jelsoft Enterprises Ltd).
Hosted by www.OICcam.com
IT Support and Services provided by Executive IT Services