Skip to content

Conversation

@konard
Copy link
Member

@konard konard commented Sep 13, 2025

Summary

  • Update friend std::ostream operator to use more descriptive parameter names as requested in Maybe use luxury style  #65
  • Replace 'out' with 'stream' for std::ostream parameter
  • Replace 'obj' with 'self' for object parameter
  • Update both C# and Python transformers for consistency

Changes Made

  1. C# Transformer (CSharpToCppTransformer.cs):

    • Updated regex replacement pattern to generate friend std::ostream & operator <<(std::ostream &stream, const TYPE &self)
    • Updated comment to reflect the new luxury style
  2. Python Transformer (cs2cpp.py):

    • Updated corresponding SubRule to maintain consistency between C# and Python implementations
    • Updated comment to reflect the new luxury style

Test Results

  • All existing tests continue to pass ✅
  • Changes maintain backward compatibility
  • No breaking changes to existing functionality

Fixes #65

🤖 Generated with Claude Code

Adding CLAUDE.md with task information for AI processing.
This file will be removed when the task is complete.

Issue: #65
@konard konard self-assigned this Sep 13, 2025
Update friend std::ostream operator to use more descriptive parameter names:
- Replace 'out' with 'stream' for std::ostream parameter
- Replace 'obj' with 'self' for object parameter
- Update both C# and Python transformers for consistency

Fixes #65

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@konard konard changed the title [WIP] Maybe use luxury style Implement luxury style for ostream operator parameters Sep 13, 2025
@konard konard marked this pull request as ready for review September 13, 2025 08:17
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.

Maybe use luxury style

2 participants