Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Process <repname> as described in draft documents #233

Merged

Conversation

bobjacobsen
Copy link
Contributor

This adds <repname> element as described in the draft S&TN:

  • Allow multiple <repname> elements
  • Extend the last one as needed
    • By incrementing integer suffix is present
    • Otherwise numbering 1..N as now
    • And respecting whitespace

This is in use in JMRI 5.5.4 and initial feedback is good.

Contains small update to test/org/openlcb/cdi/swing/CdiPanelDemo.java which was used during development

@bobjacobsen
Copy link
Contributor Author

Could we get this (and other OpenLCB/OpenLCB_Java PRs) merged so I can use an official openlcb.jar version in the next JMRI production version please?

@dpharris dpharris merged commit 0eaf235 into openlcb:master Nov 4, 2023
@dpharris
Copy link
Contributor

dpharris commented Nov 4, 2023

Merged

@dpharris
Copy link
Contributor

dpharris commented Nov 4, 2023 via email

@bobjacobsen bobjacobsen deleted the bobjacobsen-repname-update-from-candp branch November 4, 2023 13:36
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.

2 participants