Skip to content

Rover: Use name 'destination' (not desired location)#32890

Open
hunt0r wants to merge 1 commit intoArduPilot:masterfrom
hunt0r:issue32836-rover-use-destination-not-desired-location
Open

Rover: Use name 'destination' (not desired location)#32890
hunt0r wants to merge 1 commit intoArduPilot:masterfrom
hunt0r:issue32836-rover-use-destination-not-desired-location

Conversation

@hunt0r
Copy link
Copy Markdown
Contributor

@hunt0r hunt0r commented Apr 22, 2026

Summary

Improve code clarity by aligning Rover on term 'destination' (not "desired location"). This corresponds to PR-set #32883 #32884 #32886 #32887 #32888 which do the same in AR_WPNav.

Classification & Testing (check all that apply and add your own)

  • Checked by a human programmer
  • Non-functional change
  • No-binary change
  • Infrastructure change (e.g. unit tests, helper scripts)
  • Automated test(s) verify changes (e.g. unit test, autotest)
  • Tested manually, description below (e.g. SITL)
  • Tested on hardware
  • Logs attached
  • Logs available on request
./Tools/scripts/size_compare_branches.py --vehicle=rover --board=CubeOrange

Board,rover
CubeOrange,*

I am trusting that Rover's autotests cover all modes, although I have not verified that myself. The fact that it compiles already demonstrates the rename was successful (at least for all compiled-in code paths).

Description

Fixes a few comments which contribute non-obvious information. Removes self-evident ones.

Also removes some unnecessary self-evident comments
@hunt0r hunt0r force-pushed the issue32836-rover-use-destination-not-desired-location branch from f1b7c14 to 38c56da Compare April 22, 2026 19:46
@hunt0r hunt0r marked this pull request as ready for review April 22, 2026 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant