GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — bpo-27122: Fix comment to point to correct issue # by njsmith · Pull Request #47 · python/cpython

Conversation

@njsmith

@njsmith njsmith commented Feb 12, 2017

Copy link
Copy Markdown
Contributor

It took me quite a bit to figure out what this was referring to, since the given issue number is wrong, and the original commit message I found through git blame lists a different, also wrong issue number... see https://bugs.python.org/issue27122#msg279449

Here's the correct issue link to confirm: https://bugs.python.org/issue27122

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
deronnax reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
@njsmith
Fix comment to point to correct issue #
a4f3401 
It took me quite a bit to figure out what this was referring to, since the given issue number is wrong, and the original commit message I found through git blame lists a different, also wrong issue number... see https://bugs.python.org/issue27122#msg279449
@the-knights-who-say-ni the-knights-who-say-ni added the CLA signed label Feb 12, 2017
@codecov

codecov Bot commented Feb 12, 2017

Copy link
Copy Markdown

Codecov Report

Merging #47 into master will decrease coverage by -0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #47      +/-   ##
==========================================
- Coverage   82.37%   82.37%   -0.01%     
==========================================
  Files        1427     1427              
  Lines      350948   350948              
==========================================
- Hits       289089   289083       -6     
- Misses      61859    61865       +6

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2294f3a...a4f3401. Read the comment docs.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
berkerpeksag
berkerpeksag approved these changes Feb 12, 2017
View reviewed changes
@berkerpeksag
berkerpeksag merged commit af88e7e into python:master Feb 12, 2017
@berkerpeksag berkerpeksag added backport to 3.5 labels Feb 12, 2017
berkerpeksag pushed a commit that referenced this pull request Feb 12, 2017
@njsmith @berkerpeksag
bpo-27122: Fix comment to point to correct issue number (#47)
79f292f 
It took me quite a bit to figure out what this was referring to,
since the given issue number is wrong, and the original commit
message I found through git blame lists a different, also wrong
issue number... see https://bugs.python.org/issue27122#msg279449(cherry picked from commit af88e7e)
berkerpeksag pushed a commit that referenced this pull request Feb 12, 2017
@njsmith @berkerpeksag
bpo-27122: Fix comment to point to correct issue number (#47)
df948fd 
It took me quite a bit to figure out what this was referring to,
since the given issue number is wrong, and the original commit
message I found through git blame lists a different, also wrong
issue number... see https://bugs.python.org/issue27122#msg279449

(cherry picked from commit af88e7e)
@berkerpeksag

berkerpeksag commented Feb 12, 2017

Copy link
Copy Markdown
Member

Thanks! I've created #48 and #50 to backport this to maintenance branches.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@Mariatta Mariatta changed the title Fix comment to point to correct issue # bpo-27122: Fix comment to point to correct issue # Feb 13, 2017
@Mariatta Mariatta removed backport to 3.5 labels Feb 13, 2017
@ghost ghost mentioned this pull request Sep 24, 2017
slpmodule_new() doesn't initialize extra members added by type_new() stackless-dev/stackless#22
Closed
iritkatriel referenced this pull request in iritkatriel/cpython Jul 23, 2021
@iritkatriel
Make hydration context a PyObject (#47)
e692fee
jaraco pushed a commit that referenced this pull request Dec 2, 2022
@pyup-bot @Mariatta
Update pytest from 3.3.2 to 3.4.0 (GH-47)
f14f205
isidentical referenced this pull request in isidentical/cpython Mar 22, 2023
@pablogsal
Merge pull request #47 from Eclips4/fstring-grammar-rebased-after-sprint
0fc633c
oraluben pushed a commit to oraluben/cpython that referenced this pull request Jun 27, 2023
@Fidget-Spinner
docs: move trying out README to the top (python#47)
837a1ea
@gvanrossum gvanrossum mentioned this pull request Aug 22, 2023
heap-use-after-free in _PyFunction_LookupByVersion #108253
Closed
@LinanV LinanV mentioned this pull request Oct 23, 2023
Objects/typeobject.c: No such file or directory. #111203
Closed
@kcatss kcatss mentioned this pull request Feb 20, 2024
Use After Free at _heapreplace_max #115706
Closed
@ngoldbaum ngoldbaum mentioned this pull request Sep 23, 2024
Crash running PyO3 tests with --test-threads=1000 #124375
Closed
This was referenced Feb 11, 2025
segmentfault when pip installing setuptools #129992
Closed
segmentfault when pip installing setuptools #129993
Closed
@sergey-miryanov sergey-miryanov mentioned this pull request Dec 18, 2025
Observed memory leak in ssl library: Python 3.14 GC issue #142516
Closed
@ngoldbaum ngoldbaum mentioned this pull request Mar 6, 2026
Remove deprecated and undocumented function ctypes.SetPointerType #133866
Closed
@johng johng mentioned this pull request Jul 19, 2026
use-after-free: functools partial #154189
Closed
@djarfluka djarfluka mentioned this pull request Sep 12, 2026
Heap UAF in io.TextIOWrapper read path via reentrant detach() #157363
Open
Kronuz added a commit to Kronuz/cpython that referenced this pull request Oct 1, 2026
@Kronuz
Record the pythongh-157757 + pythonGH-158282-behaviors backport (clea…
418fded 
…npython314 PR python#47)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@berkerpeksag berkerpeksag berkerpeksag approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

4 participants

@njsmith @berkerpeksag @Mariatta @the-knights-who-say-ni
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.