Skip to content

Move Job_* script constants and @jobchange help text to source - #3422

Open
MrKeiKun wants to merge 4 commits into
HerculesWS:masterfrom
MrKeiKun:fix-165-centralize-job-constants
Open

Move Job_* script constants and @jobchange help text to source#3422
MrKeiKun wants to merge 4 commits into
HerculesWS:masterfrom
MrKeiKun:fix-165-centralize-job-constants

Conversation

@MrKeiKun

Copy link
Copy Markdown
Contributor

Pull Request Prelude

Changes Proposed

  • Register Job_* script constants (previously duplicated by hand in db/constants.conf) from the JOB_ENUM_VALUE X-macro already used in class.h/class_hidden.h/class_special.h, so they can never drift from the JOB_* enum.
  • Generate the @jobchange / @help jobchange job listing at boot from that same source data plus pc->job_name(), instead of a hand-maintained, hand-aligned block in conf/map/help.txt.

Issues addressed: #165

…py's skip list, matching the existing atcommand_help_string precedent.
…(comparing a string literal against NULL) under /WX.
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