-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
struct Rav1dIntraPredDspContext::cfl_ac
: Make an EnumMap
over Rav1dPixelLayoutSubSampled
#585
struct Rav1dIntraPredDspContext::cfl_ac
: Make an EnumMap
over Rav1dPixelLayoutSubSampled
#585
Conversation
a37dba0
to
4a60b90
Compare
8e3b4df
to
61e77a0
Compare
4a60b90
to
eccc29a
Compare
61e77a0
to
f54bf19
Compare
eccc29a
to
86d472d
Compare
f54bf19
to
d18b3a5
Compare
86d472d
to
91c1922
Compare
d18b3a5
to
0e08885
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change itself looks fine.
However, what does this actually enable with regards to safety improvements? Was this actually blocking anything or is this a readability improvement? We need to focus on only what is blocking memory safety improvements.
This was to remove the bounds check, since |
…v1dPixelLayoutSubSampled`.
91c1922
to
dab1eef
Compare
0e08885
to
f383eac
Compare
I'll do the same for the other array fields, but I need to make the
enum
s for those first; this one already exists from before.