Jump to content
XCOMUFO & Xenocide

Fao Nachtwolf - Patches


AndyS

Recommended Posts

Nachtwolf

 

Okay - so I thought about breaking down my patches into more manageable sizes then decided I'd start all over again and do it in much smaller pieces trying to concentrate on one class at a time and be more methodical about all the things I wanted to "improve". So here we have a few dozen small patches each of which changes very few things at any one time - typically one where I can help it. I started with Soldier. I've done Cell but won't post that until I get more feedback.

 

I would have posted to mantis but it seemingly has a habit of falling over....

 

Each is a unified diff of my repository generated using TortoiseSVN, developed in Eclipse with formatting set to K&R (or at least almost K&R seen as the guidelines don't quite match).

 

Also included is the svn log file detailing (briefly) what each diff does and the files it touches. What isn't included anything about why I've done any of the changes I propose. I can do and am happy to do so if in theory at least you're happy with the code. Most of this stuff is based on work by Herb Sutter & Scott Meyer (code quality) or Martin Fowler and other (refactoring). Typically I'm aiming to make the code more readable and more resilient. This is all small stuff to start with to encapsulate the class better - by removing access to internals through public member variables or friendship and to reduce compile dependencies by rationalising the includes.

 

The diff files are based on my repository and so file versions and file locations are not those of the UFO2000 svn repository. A quick search and replace or simple awk/sed script will do the trick to fix them to act as patches for a working copy of UFO2000.

 

Give me feedback and we can take it from there.

 

Andy

diff3.txt

diff4.txt

diff5.txt

diff6.txt

diff7.txt

diff8.txt

diff9.txt

diff10.txt

diff11.txt

diff12.txt

diff13.txt

diff14.txt

diff15.txt

diff16.txt

diff17.txt

diff18.txt

diff19.txt

diff20.txt

diff21.txt

diff22.txt

diff23.txt

LogUpto23_Soldier.txt

Link to comment
Share on other sites

  • 2 weeks later...
I've done Cell but won't post that until I get more feedback.

Well, it would be better if you explain a bit more about your patches. Not the inner things but in common. Like, if i apply these patches can i play with others who haven't applied them? If i'm using the .1086 beta is it possible?

How do these pathes influence on the performance?

Please, provide some info for non-nerd'ish programmers, but common users who just able to apply patches to the source code.

Be so kind =)

Link to comment
Share on other sites

Guest Azrael Strife
non-nerd'ish programmers, but common users who just able to apply patches to the source code.

The common user wouldn't know how to apply them in the first place.

Not to mention he wouldn't have a NEED to do so :P

Link to comment
Share on other sites

he-he... well, the thing is that if one able to google on 'apply patch how-to' then it wouldn't be nerdish, but still, if these patches are there then there were some points to post them, right? Just, please, tell a little bit more about your patches (see my prev post).

Are these patches just the matter of refactoring? or maybe there are some bugfixes and incompatibilities? Just a couple of sentences would be fine, i hope.

Edited by Kraplax
Link to comment
Share on other sites

Guest Azrael Strife
he-he... well, the thing is that if one able to google on 'apply patch how-to' then it wouldn't be nerdish, but still, if these patches are there then there were some points to post them, right? Just, please, tell a little bit more about your patches (see my prev post).

Are these patches just the matter of refactoring? or maybe there are some bugfixes and incompatibilities? Just a couple of sentences would be fine, i hope.

You don't seem to understand. These patches are posted here for Natchwolf, not for the average player. These patches are not the kind of patches you see in commercial games, they are modifications to the code that not necessarily fix bugs, they are modifications, additions.

And there is no "apply patch how-to", every project, every program, has its own way of applying patches (you need to know what kind of compiler they use, dependencies, etc). And there is no "non-nerdish" (whatever that means) way of explaining what they do, I suppose.

Link to comment
Share on other sites

he-he... well, the thing is that if one able to google on 'apply patch how-to' then it wouldn't be nerdish, but still, if these patches are there then there were some points to post them, right? Just, please, tell a little bit more about your patches (see my prev post).

Are these patches just the matter of refactoring? or maybe there are some bugfixes and incompatibilities? Just a couple of sentences would be fine, i hope.

You don't seem to understand. These patches are posted here for Natchwolf, not for the average player. These patches are not the kind of patches you see in commercial games, they are modifications to the code that not necessarily fix bugs, they are modifications, additions.

And there is no "apply patch how-to", every project, every program, has its own way of applying patches (you need to know what kind of compiler they use, dependencies, etc). And there is no "non-nerdish" (whatever that means) way of explaining what they do, I suppose.

 

Azrael is spot on - these patches are only for Nachtwolf to comment on regarding what Nachtwolf would like to see in terms of info as a committer. They were posted here because the bug tracker was playing up.

 

The patches are extremely "nerdish" - the only thing they seek to do is improve code readability (in very very small stages). There is no functional improvement and therefore no interest to gamers. The whole purpose is to start to refactor code to make it more maintenance friendly. These are exteremly small scale internal changes to compilation and class structure.

 

Sorry for the delay in picking this thread up - I've been enjoying the festive season without computer.... How is it possible I hear you cry.... :-)

Link to comment
Share on other sites

They were posted here because the bug tracker was playing up.

 

The patches are extremely "nerdish" - the only thing they seek to do is improve code readability (in very very small stages). There is no functional improvement and therefore no interest to gamers. The whole purpose is to start to refactor code to make it more maintenance friendly.

 

Oh, that was the clearest description. Thanx. So, one may apply or may not apply them - no functionality changes will happen. It's just for programmers. Right.

Thanks again =)

Link to comment
Share on other sites

  • 2 weeks later...

Got them for review. ETA one week along with Cirix new patch.

 

Expect a commit to the source if both are good and possibly a new beta release if there is enough new shiny material to build one.

 

^_^

Link to comment
Share on other sites

Please sign in to comment

You will be able to leave a comment after signing in



Sign In Now
×
×
  • Create New...