Jump to content

[Dev] remove DBC files and make life easily for everyone


antz

Recommended Posts

Posted
But there is problem, not all dbc files/params are known, so devs will often turn to dbc for advice.

Only the known fields are used by the core, so these fields will always be imported. The unknown fields can still be imported by the devs if they need to.

Posted

Keyword "unknown fields can still be imported" = pain in the butt for devs... At now for them all good. But if implement only part, everything becomes bad... That's where I'm going. For those who just clone repositories & compile core it would be awesome of course, because it decreases the amount of things to do. Developers all the way around, if you make only a part of it.

The way i'm designing the system, the default is to include every field from all DBC's plus all the data. If the fields are unknown then Mangos will not be using them. Once the use is figured out, they can be easily wired in.

Also nothing is hardcoded in AD.exe, all settings are controlled via a config file.

  • 2 months later...
Posted
if your interested the latest version of my AD.exe modifications is located at :-

git://github.com/billy1arm/server.git

Milestone One of this mini-project is complete.

There are extra options added to ad.exe for extraction to SQL and CSV files.

My question is this, should I submit this to master at this point since it adds additional functionality or wait until i've completed the next 2 milestones?

Posted

Most definitely submit this! As TheLuda has said, "Commit early, commit often." is how development cycles in MaNGOS should be done, even if it means the code is still a work-in-progress.

This is done for two reasons...

To keep code from being unecessarily delayed or withheld.

To let developers more easily maintain their projects by having core changes made more accessible.

At the very least, your work could be forked into a branch for testing if you submit it.

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. Privacy Policy Terms of Use