Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Aug 7, 2025

This PR standardizes all references to the C and C++ programming languages by converting lowercase instances to the proper uppercase format throughout the project documentation and code comments.

Changes Made

Documentation Updates:

  • README.md: Changed "c/c++ lib" to "C/C++ lib" in the usage section

Code Comment Updates:

  • _xtool/internal/clang/clang.go: Updated comment from "c/c++ standard" to "C/C++ standard" and "c++:gnu++17" to "C++:gnu++17"
  • _xtool/internal/parser/parser.go: Changed "non cpp mode" to "non C++ mode" in developer note
  • cmd/llcppcfg/llcppcfg.go: Updated flag description from "c++ lib" to "C++ lib"
  • parser/iparser.go: Changed comment from "parser with c++" to "parser with C++"

What Was Not Changed

The changes were surgical and focused only on language name references. Technical terms, command names, file extensions, and JSON field names were preserved:

  • Command line flags like -cpp and clang++
  • File extensions like .cpp
  • JSON field names like "c++": in symbol tables
  • Code examples showing actual C/C++ syntax

All tests continue to pass and the build remains successful after these changes.

Fixes #536.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits August 7, 2025 15:21
Co-authored-by: luoliwoshang <51194195+luoliwoshang@users.noreply.github.com>
…de comments

Co-authored-by: luoliwoshang <51194195+luoliwoshang@users.noreply.github.com>
Copilot AI changed the title [WIP] docs:standardize C/C++ Language Names to Uppercase in docs docs: standardize C/C++ language names to uppercase in documentation and code comments Aug 7, 2025
Copilot AI requested a review from luoliwoshang August 7, 2025 15:31
Co-authored-by: luoliwoshang <51194195+luoliwoshang@users.noreply.github.com>
…age standardization

Co-authored-by: luoliwoshang <51194195+luoliwoshang@users.noreply.github.com>
Copy link
Member

@luoliwoshang luoliwoshang left a comment

Choose a reason for hiding this comment

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

LGTM

@luoliwoshang luoliwoshang marked this pull request as ready for review August 7, 2025 15:51
@luoliwoshang luoliwoshang requested a review from MeteorsLiu August 7, 2025 15:52
@codecov
Copy link

codecov bot commented Aug 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.20%. Comparing base (30c4497) to head (83b6836).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #537   +/-   ##
=======================================
  Coverage   84.20%   84.20%           
=======================================
  Files          27       27           
  Lines        2747     2747           
=======================================
  Hits         2313     2313           
  Misses        391      391           
  Partials       43       43           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@luoliwoshang
Copy link
Member

@MeteorsLiu CC

@MeteorsLiu MeteorsLiu merged commit 9d19b74 into main Aug 8, 2025
22 checks passed
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.

docs:standardize C/C++ Language Names to Uppercase in docs

3 participants