bpo-36817: Fix reference leak for expr_text in f-string = parsing by pablogsal · Pull Request #13249 · python/cpython

@pablogsal

@pablogsal

❯ ./python -m test test_future -m test_annotations -R :
Run tests sequentially
0:00:00 load avg: 0.66 [1/1] test_future
beginning 9 repetitions
123456789
.........
test_future passed

== Tests result: SUCCESS ==

1 test OK.

Total duration: 324 ms
Tests result: SUCCESS

ilevkivskyi

Choose a reason for hiding this comment

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

Thanks for the fix! LGTM.

ericvsmith

Choose a reason for hiding this comment

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

Thanks for catching this. I kept thinking "I need to remember to make sure that memory gets freed", and then I forgot to do it.

@pablogsal

@pablogsal

I am restarting the CI because appveyor failed with some random asyncio error.