Skip to content

Add getID function to the simplefile implementation - #26003

Merged
MorrisJobke merged 1 commit into
masterfrom
enh/simplefile/getid
Mar 15, 2021
Merged

Add getID function to the simplefile implementation#26003
MorrisJobke merged 1 commit into
masterfrom
enh/simplefile/getid

Conversation

@rullzer

Copy link
Copy Markdown
Member

Sometimes it is useful to get the actual fileid

Signed-off-by: Roeland Jago Douma roeland@famdouma.nl

Sometimes it is useful to get the actual fileid
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

@juliusknorrjuliusknorr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Ah wait, should be added to the public interface as well then I guess 😉

@rullzer

Copy link
Copy Markdown
MemberAuthor

Ah wait, should be added to the public interface as well then I guess wink

Not for now. I'm not sure yet how to do it with the other implementations.
But also would like to backport this part. It is mainly for internal use this getId anyway.

@MorrisJobke
MorrisJobke merged commit f512705 into masterMar 15, 2021
@MorrisJobke
MorrisJobke deleted the enh/simplefile/getid branch March 15, 2021 13:41
@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable21

@MorrisJobke

Copy link
Copy Markdown
Member

/backport to stable20

@kesselb

Copy link
Copy Markdown
Contributor

/backport to stable20

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsenhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@rullzer@MorrisJobke@kesselb@ChristophWurst@juliusknorr