google / google/autocxx

Add more tests for array support

Open
#989 3 comments 1 reaction 0 assignees View on GitHub
good first issue
Dominant language
Rust
Stars
2.6k
Forks
175
PR merge metrics
No merged PRs in 30d

Description

#969 introduced some support for C++ arrays. So far, we know this works for fields of structs. We should add more tests to find out what happens if we allow arrays as actual function parameters, etc. Pretty sure it won't work because the support for arrays in `TypeConverter` is incomplete. But, we should find out!

Contributor guide

Open the contributing guide

Research direction

Start by reviewing issue #969 and the incomplete array handling in `TypeConverter`, then inspect the existing tests for arrays used as struct fields. Add tests covering arrays as function parameters and other relevant uses, and use the results to establish which cases currently work and which remain unsupported.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp, rust
Domain
devtools, testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.