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 — PyObject_New breaks by the commit 7ad6f74fcf9db1ccfeaf0986064870d8d3887300 · Issue #94185 · python/cpython

PyObject_New breaks by the commit 7ad6f74fcf9db1ccfeaf0986064870d8d3887300 #94185

New issue
Copy link
New issue
Copy link
Closed
Closed
PyObject_New breaks by the commit 7ad6f74fcf9db1ccfeaf0986064870d8d3887300#94185
Copy link
Labels
type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

Description

@trcrsired
trcrsired
opened on Jun 23, 2022
Issue body actions

/usr/local/include/python3.12/objimpl.h:134:46: error: expected ')' before '_PyObject_New'
134 | #define PyObject_New(type, typeobj) ((type *)_PyObject_New(typeobj))
| ~ ^~~~~~~~~~~~~
7ad6f74

This is not working for g++ 13

Activity

  1. trcrsired
    added
    type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error
    on Jun 23, 2022

  2. vstinner commented on Jun 24, 2022

    @vstinner
    vstinner
    on Jun 24, 2022
    Member
    More actions

    Oh no, writing correct macros is so complicated :-( I wrote PR #94234 to fix this issue.

    cc @erlend-aasland

    Reacted by Erlend E. Aasland

  3. kumaraditya303 commented on Jun 26, 2022

    @kumaraditya303
    kumaraditya303
    on Jun 26, 2022
    Contributor
    More actions

    Fixed by #94234

  4. kumaraditya303
    closed this as completedon Jun 26, 2022

  5. vstinner commented on Jun 26, 2022

    @vstinner
    vstinner
    on Jun 26, 2022
    Member
    More actions

    Thanks for reporting the issue! It's great that you catched the issue early, so it could be fixed quickly ;-) I added unit tests to prevent future similar regressions. Sadly, the C API is still not fully tested, as you can see.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions
      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.