Skip to content

Pointer Reform: proper slicing and indexing #1053

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 5, 2018
Merged

Pointer Reform: proper slicing and indexing #1053

merged 1 commit into from
Jun 5, 2018
+268 −79

Conversation

@andrewrk
Copy link
Member

@andrewrk andrewrk commented Jun 5, 2018

  • enable slicing for single-item ptr to arrays
  • disable slicing for other single-item pointers
  • enable indexing for single-item ptr to arrays
  • disable indexing for other single-item pointers

see #770
closes #386

@andrewrk andrewrk added the breaking Implementing this issue could cause existing code to no longer compile or have different behavior. label Jun 5, 2018
@andrewrk
Pointer Reform: proper slicing and indexing
38c62a3 
 * enable slicing for single-item ptr to arrays
 * disable slicing for other single-item pointers
 * enable indexing for single-item ptr to arrays
 * disable indexing for other single-item pointers

see #770
closes #386
@andrewrk andrewrk force-pushed the pointer-reform-slicing branch from 4823614 to 38c62a3 Compare June 5, 2018 00:36
@andrewrk andrewrk merged commit e53b683 into master Jun 5, 2018
@andrewrk andrewrk deleted the pointer-reform-slicing branch June 5, 2018 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

breaking Implementing this issue could cause existing code to no longer compile or have different behavior.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

design flaw: Access out of bounds on pointer to array

1 participant

@andrewrk