Skip to content

Comments

topdown: Remove carriage returns (^M)#536

Open
Treece-Burgess wants to merge 1 commit intoicl-utk-edu:masterfrom
Treece-Burgess:01-15-2025-topdown-remove-carriage
Open

topdown: Remove carriage returns (^M)#536
Treece-Burgess wants to merge 1 commit intoicl-utk-edu:masterfrom
Treece-Burgess:01-15-2025-topdown-remove-carriage

Conversation

@Treece-Burgess
Copy link
Contributor

Pull Request Description

The topdown component files:

README.md
Rules.topdown
topdown.c
topdown.h

all had carriage returns (^M). This PR removes them.

Testing

Testing was done on Picard at Oregon with the following setup:

  • PAPI configure: ./configure --prefix=$PWD/test-install --with-components="topdown" --with-debug=yes
  • CPU: Intel Xeon Gold 6430 (SPR)
  • OS: RHEL 8.10

Results are as follows:

  • PAPI build: ✅
  • PAPI utilities*: ✅
  • topdown component tests: ✅

* - papi_component_avail, papi_native_avail, papi_command_line

Author Checklist

  • Description
    Why this PR exists. Reference all relevant information, including background, issues, test failures, etc
  • Commits
    Commits are self contained and only do one thing
    Commits have a header of the form: module: short description
    Commits have a body (whenever relevant) containing a detailed description of the addressed problem and its solution
  • Tests
    The PR needs to pass all the tests

@Treece-Burgess Treece-Burgess added status-ready-for-review PR is ready to be reviewed component-topdown PRs related to the topdown component type-maintenance Update code to keep it compatible, secure, modern. labels Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component-topdown PRs related to the topdown component status-ready-for-review PR is ready to be reviewed type-maintenance Update code to keep it compatible, secure, modern.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant