Skip to content
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

Improve efficiency in FindRenderizableReferenceNumeric and getReference (#16251) #16255

Merged
merged 2 commits into from
Jun 26, 2021

Conversation

zeripath
Copy link
Contributor

Backport #16251

  • The Fuzzer is running on a non-repo urlprefix which is incorrect for RenderRaw
  • Make FindRenderizableReferenceNumeric and getReferences more efficient

Signed-off-by: Andrew Thornton art27@cantab.net
Co-authored-by: techknowlogick techknowlogick@gitea.io

zeripath and others added 2 commits June 26, 2021 00:43
…ces (go-gitea#16251)

* The Fuzzer is running on a non-repo urlprefix which is incorrect for RenderRaw
* Make FindRenderizableReferenceNumeric and getReferences more efficient

Signed-off-by: Andrew Thornton <art27@cantab.net>
Co-authored-by: techknowlogick <techknowlogick@gitea.io>
Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added this to the 1.14.4 milestone Jun 25, 2021
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Jun 25, 2021
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jun 26, 2021
@zeripath zeripath merged commit b935472 into go-gitea:release/v1.14 Jun 26, 2021
@zeripath zeripath deleted the backport-16251-v1.14 branch June 26, 2021 07:31
@go-gitea go-gitea locked and limited conversation to collaborators Oct 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants