New guard-range sexp - #7606
Conversation
New guard-range sexp which allows different ranges for different guarders
TheForce172
added
feature
|
Can this not be done as an expansion of the old sexp instead of adding yet another -new sexp? I know there's convention for that but it's not really a user friendly convention. |
|
Not without changeing the old sexp's behaviour which were not allowed to do as it's been in a stable release. |
|
I suspect I'm the only dev who has used the 'old' set-guard-range Sexp since 26 is so new and I have no issue with adding this to the existing Sexp 🙂 |
|
I did a poll to ask if anyone had used the set-guard-range Sexp and it looks like no one has except myself, and most people didn't even know it was 'a thing', so I think we can safely add to it without upsetting anyone. |
Goober5000
left a comment
Goober5000
left a comment
There was a problem hiding this comment.
Got a review for this which I'll send over Discord.
|
Ok, Is clang having a nervous breakdown today or something? This one makes no sense either? |
The codebase in general prefers EDIT: Oh I see.. it's saying there's a NULL on that line when there's clearly not. Uh. That's neat. |
Goober5000
left a comment
Goober5000
left a comment
There was a problem hiding this comment.
Had some time to look at this more closely now. A few more comments inline. Also:
- The
set-guard-rangedefinition in theOperatorsarray still has 2 minimum arguments. It should have 3. - Since you've made extensive modifications to the SEXP, you could add your name as a co-author.
- The
warp_shipp = NULL;checks are clang checking unrelated lines that weren't included in the diff. They are fromship_render. Annoying, but just two lines to tweak and the warnings should go away.
|
Hi guys. I don't want to sound pushy but do we have an ETA or target date for this? If it's going to be a while I'll need to plan around it. Also if I were to use the 'old' set-guard-range Sexp in a mission will I need to go back and change it for the new one once it's ready? Cheers 👍 |
|
Sorry for the delay, my life got super busy suddenly out of nowhere and I hadn't had the chance to look at this. Baring any issues the version I just pushed should be ready to go in. I did have to rip out the scripting support as I simply don't have time right now to learn how to rewrite it for how we now do the sexp. If you have used the old sexp you will have to update but it should be obvious where as it should fail on running the sexp as it wont have enough parameters. |
Goober5000
left a comment
Goober5000
left a comment
There was a problem hiding this comment.
Nice work! This is nearly ready to go. Just a couple more tweaks.
| "\t1:\tShip the escorts won't leave the range of if guarding (Ship must be in mission)\r\n" | ||
| "\t2:\tGuard range cap in meters (<= 0 disables cap)\r\n" | ||
| "\t3+:\tEscort ships and wings that the limit appies too" }, | ||
| "\t3+:\tEscort ships and wings that the limit applies too" }, |
There was a problem hiding this comment.
The "too" -> "to" typo should still be fixed, but this is close enough to approve
4 participants
New guard-range sexp which allows different ranges for different guarders. As requested by Iain baker.