Skip to content

Reformats and use odr_strdup_null to simplify - #35

Merged
adamdickmeiss merged 1 commit into
masterfrom
reformatting-space-left-paran
Nov 9, 2026
Merged

adamdickmeiss merged 1 commit into
masterfrom
reformatting-space-left-paran

Conversation

@adamdickmeiss

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues are reported for this change.

No open issues 🎉

What changed in this PR

This PR simplifies nullable database-name copying and cleans up formatting across the yazpp toolkit’s Z39.50 server, client association, and sample code.

Changes:

  • Replaces conditional odr_strdup usage with odr_strdup_null in database-record creation.
  • Standardizes function spacing, indentation, and line wrapping.
  • Removes trailing whitespace from copyright comments.
File Description
src/​yaz-z-server.cpp Simplifies nullable database-name copying and normalizes formatting.
src/​yaz-z-server-update.cpp Adjusts update-service call formatting and indentation.
src/​yaz-z-server-sr.cpp Reformats retrieval signatures and function calls.
src/​yaz-z-server-ill.cpp Normalizes logging-call spacing and comment whitespace.
src/​yaz-z-query.cpp Removes trailing whitespace from the copyright comment only.
src/​yaz-z-databases.cpp Normalizes database-helper signatures and call spacing.
src/​yaz-my-server.cpp Reformats sample-server declarations, definitions, and calls.
src/​yaz-marc-sample.cpp Removes trailing whitespace from the copyright comment only.
src/​yaz-ir-assoc.cpp Standardizes association-method formatting and indentation.

Review effort: Balanced

@adamdickmeiss
adamdickmeiss merged commit 4aef54a into master Nov 9, 2026
3 checks passed
@adamdickmeiss
adamdickmeiss deleted the reformatting-space-left-paran branch November 9, 2026 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants