Skip to content

[STABLE ABI] Use const/mutable_data_ptr templates instead of reinterpret_cast#4152

Merged
pearu merged 1 commit intopytorch:mainfrom
pearu:pearu/elimnate-data_ptr
Dec 11, 2025
Merged

[STABLE ABI] Use const/mutable_data_ptr templates instead of reinterpret_cast#4152
pearu merged 1 commit intopytorch:mainfrom
pearu:pearu/elimnate-data_ptr

Conversation

@pearu
Copy link
Copy Markdown
Collaborator

@pearu pearu commented Dec 11, 2025

As in the title. The const/mutable_data_ptr templates are safer to use as these implement type check before casting.

PLEASE NOTE THAT THE TORCHAUDIO REPOSITORY IS NO LONGER ACTIVELY MONITORED. You may not get a response. For open discussions, visit https://discuss.pytorch.org/.

@pearu pearu requested a review from a team as a code owner December 11, 2025 15:15
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Dec 11, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/audio/4152

Note: Links to docs will display an error until the docs builds have been completed.

⏳ No Failures, 1 Pending

As of commit ae36bce with merge base c9b37e9 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla Bot added the CLA Signed label Dec 11, 2025
@pearu pearu force-pushed the pearu/elimnate-data_ptr branch from 829a7f9 to ae36bce Compare December 11, 2025 15:17
@pearu pearu merged commit 446a215 into pytorch:main Dec 11, 2025
61 of 65 checks passed
NicolasHug pushed a commit to NicolasHug/audio that referenced this pull request Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants