patchWarzone 2100 Project - Patches: patch #1104, New turret management code

 
 
Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

patch #1104: New turret management code

Submitted by:  Per I. Mathisen <per>
Submitted on:  Thu 11 Sep 2008 07:09:10 PM UTC  
 
Category: FeaturePriority: 5 - Normal
Status: Ready For TestPrivacy: Public
Assigned to: NoneOpen/Closed: Open
Planned Release: None

(Jump to the original submission Jump to the original submission)

Tue 23 Sep 2008 08:40:15 PM UTC, SVN revision 6070:

patch #1104: New turret management code: Part 2: Cleanup actionTargetTurret(). Reduces number of input parameters from 7 to 3.

(Browse SVN revision 6070)

Per I. Mathisen <per>
Project Administrator
Mon 22 Sep 2008 03:33:47 AM UTC, comment #5:

I didn't test vtols, but I tested a bunch of other turrets, and it seems to work fine in skirmish games.

btw, I rarely check gna for patches, I like trac much better, since you can see a quick diff of the code with no real effort.

Bugs Buggy <buginator>
Project Administrator
Sat 20 Sep 2008 07:58:40 PM UTC, comment #4:

Part 2: Cleanup actionTargetTurret(). Reduces number of input parameters from 7 to 3.

(file #4805)

Per I. Mathisen <per>
Project Administrator
Tue 16 Sep 2008 07:21:26 PM UTC, SVN revision 6033:

From patch #1104: New turret management code: Minor change the API for actionInsideMinRange() to pass pointer
to weapon stats directly.

(Browse SVN revision 6033)

Per I. Mathisen <per>
Project Administrator
Sun 14 Sep 2008 10:28:01 AM UTC, comment #2:

Turns out I need to change a lot of other stuff for this to work properly. Part 1 of this work attached - just changes the API for actionInsideMinRange(), as I want to remove a lot of the hardcoded weapon slot indexing.

(file #4795)

Per I. Mathisen <per>
Project Administrator
Sat 13 Sep 2008 12:16:21 PM UTC, comment #1:

afai tested, vtols do not shoot

would it be easy to add support for "turret rotation boundaries"?

Elio Gubser <ohyeh>
Project Member
Thu 11 Sep 2008 07:09:10 PM UTC, original submission:

I rewrote the turret management code to fix problems with self-repair, but mostly because the existing code was hideously ugly and hard to maintain. It should now be easier to extend to support multiple turrets, where one or more are non-weapon turrets.

Per I. Mathisen <per>
Project Administrator

 

Attached Files
file #4805:  actionTargetTurret2.diff added by per (34kB - text/x-patch)
file #4795:  actionInsideMinRange.diff added by per (6kB - text/x-patch)
file #4787:  attack1b.diff added by per (20kB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by per
  • -unavailable- added by ohyeh (Posted a comment)
  • -unavailable- added by per (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 20 Sep 2008 07:58:40 PM UTCperAttached File-=>Added actionTargetTurret2.diff, #4805
    Sun 14 Sep 2008 10:28:01 AM UTCperAttached File-=>Added actionInsideMinRange.diff, #4795
      Carbon-Copy-=>Added -unavailable-
    Thu 11 Sep 2008 07:09:10 PM UTCperAttached File-=>Added attack1b.diff, #4787
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup