Skip to content

Standardize naming conventions in C code - #21

Closed
matapatos wants to merge 1 commit into
mainfrom
vibe/standardize-naming-conventions-a54fe0
Closed

Standardize naming conventions in C code#21
matapatos wants to merge 1 commit into
mainfrom
vibe/standardize-naming-conventions-a54fe0

Conversation

@matapatos

Copy link
Copy Markdown
Contributor

Standardize naming conventions in C code

Changes

  • Add av_ prefix to all internal functions in validate_function.c:
    • transform_property_nameav_transform_property_name
    • get_property_nameav_get_property_name
    • get_property_valueav_get_property_value
    • add_field_errorav_add_field_error
    • validate_field_valueav_validate_field_value
  • Standardize all functions to use zend_always_inline
  • Update const correctness for error_message parameter
  • Fix add_field_error to use ZVAL_STRINGL and remove redundant dtor
  • Update all function calls to use new names
  • Update header file declarations to match

Impact

  • Consistent naming convention across the codebase
  • Easier to identify extension-specific functions
  • Improved code maintainability
  • Better code organization

Closes #8

- Add av_ prefix to all internal functions in validate_function.c:
  - transform_property_name -> av_transform_property_name
  - get_property_name -> av_get_property_name
  - get_property_value -> av_get_property_value
  - add_field_error -> av_add_field_error
  - validate_field_value -> av_validate_field_value
- Standardize all functions to use zend_always_inline
- Update const correctness for error_message parameter
- Fix add_field_error to use ZVAL_STRINGL and remove redundant dtor
- Update all function calls to use new names
- Update header file declarations to match

Closes #8

Co-authored-by: matapatos <matapatos@users.noreply.github.com>
@matapatos matapatos closed this Aug 7, 2026
@matapatos
matapatos deleted the vibe/standardize-naming-conventions-a54fe0 branch August 7, 2026 15:50
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