AcademySoftwareFoundation / AcademySoftwareFoundation/openexr

restrict type qualifier

Open
#1,996 7 comments 0 reactions 0 assignees View on GitHub
needs attention
Dominant language
C
Stars
1.8k
Forks
700
Avg merge
3d 12h
Merged PRs (30d)
29

Description

In 22cbb7966fa16dae2a1dde14c3a7d9966661516c the `restrict` type qualifier was added to some variables, but Visual Studio doesn't like it. Microsoft uses [`__restrict`](https://learn.microsoft.com/en-us/cpp/cpp/extension-restrict) instead.

I guess we should make a `RESTRICT` macro to use in its place. I'd provide a pull request, but not sure where such a thing should live.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing commit 22cbb7966fa16dae2a1dde14c3a7d9966661516c to locate the variables using restrict, then check how compiler-specific definitions are organized. Confirm the replacement accommodates Visual Studio's __restrict and preserves the existing use of restrict elsewhere.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
computer-graphics
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.