Skip to content

Fix bug where includer never got added#17

Open
Oipo wants to merge 1 commit intoemilk:masterfrom
Oipo:master
Open

Fix bug where includer never got added#17
Oipo wants to merge 1 commit intoemilk:masterfrom
Oipo:master

Conversation

@Oipo
Copy link

@Oipo Oipo commented Sep 3, 2021

No description provided.

@Oipo
Copy link
Author

Oipo commented Sep 3, 2021

Brainfart moment, the signature of the function is const DocInfo_SP& doc()... which returns a reference anyway.

@Oipo Oipo closed this Sep 3, 2021
@Oipo
Copy link
Author

Oipo commented Sep 3, 2021

Nope, was right after all: https://godbolt.org/z/sszYbWYrb

@Oipo Oipo reopened this Sep 3, 2021
@amiteshsingh-cpi
Copy link

Nope, was right after all: https://godbolt.org/z/sszYbWYrb

not required, if the object is of type shared_ptr.

https://godbolt.org/z/jnoPYddEq

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.

2 participants

Comments