Skip to content

changed ptr to pointer - #845

Closed
pavanpannati wants to merge 0 commit into
servo:v2from
pavanpannati:v2
Closed

pavanpannati wants to merge 0 commit into
servo:v2from
pavanpannati:v2

Conversation

@pavanpannati

Copy link
Copy Markdown

closes #843

@alejandro-vaz alejandro-vaz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and so on.

also, why is it called pointernternter??

Comment thread src/allocator/implementations/base.rs Outdated
core::{
alloc::Layout,
ptr::NonNull
pointernternter::NonNull

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this has to stay as it was because it is an import from core

core::{
alloc::Layout,
ptr::NonNull
pointernternter::NonNull

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

Comment thread src/allocator/implementations/native.rs Outdated
core::{
alloc::Layout,
ptr::NonNull
pointernternter::NonNull

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same

Comment thread src/allocator/mod.rs Outdated
use core::{
alloc::Layout,
ptr::NonNull
pointernternter::NonNull

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same

Comment thread src/allocator/proxy.rs Outdated
core::{
alloc::Layout,
ptr::NonNull
pointernternter::NonNull

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same

@alejandro-vaz alejandro-vaz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same thing. all use block should have ptr

all the others should have pointer

@pavanpannati

Copy link
Copy Markdown
Author

once look into the last commit and inform me about the status, i hope i resolved the issue

Comment thread et --hard HEAD~2 Outdated
@@ -0,0 +1,274 @@
3f02b6e (HEAD -> v2, origin/v2, origin/HEAD) closed pointernternter error

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this please

@GauravPawar101

Copy link
Copy Markdown
Collaborator

Also re-request review(rightside of your pr body ) after pushing changes @pavanpannati

Comment thread et --hard HEAD~2 Resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

rename ptr to pointer

3 participants