KhronosGroup / KhronosGroup/OpenCL-CTS

Replace error-prone `create_random_data` with a reliable implementation

Open
#2,182 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
232
Forks
235
Avg merge
8d 7h
Merged PRs (30d)
18

Description

`create_random_data` is an error-prone function that returns a pointer to allocated memory using malloc.

As mentioned in #2177:
> we really ought to remove use of the create_random_data function entirely given that it's so error-prone. This shouldn't be too bad, since there are only a dozen or so places where create_random_data is called, and about half of the uses are in these two files.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating every call to `create_random_data` and read the related discussion in #2177, paying attention to the two files identified there. Replace or remove the function's uses across the reported call sites, with completion meaning no remaining use of `create_random_data` and the OpenCL CTS tests still passing.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
testing-qa
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.