Use modern C++ casts instead of current C-style casting approaches

Open
#250 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Refactor
Clarity
Needs clarification
Activity status
Stale
Tech stack
cpp
Domain
api

Research direction

Start by locating the API implementations that use C-style pointer casts or other C-style casting approaches. Replace the relevant casts with modern C++ casts while preserving function signatures and behavior, then verify that the implementations still build and behave identically.

Written by the indexing model from the issue text.

Description

enhancement
Description

Update various API implementations that use c-style pointer casting and other c-style approaches to use modern C++ casts.

Is this a breaking change?

The function signatures would remain unchanged
The behavior of the functions would be identical
User code that calls these functions wouldn't need any modifications

Additional information

Modernizes the code base. Compilers will optimize these the same, but coding standards favor using C++ cast in C++ code.

Dominant language
C++
Stars
306
Forks
150
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from arduino/ArduinoCore-API

All issues in arduino/ArduinoCore-API

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.