D7964: cext: change two more vars to Py_ssize_t in manifest.c

2020-01-22 Thread durin42 (Augie Fackler)
durin42 added a comment. Yes, I'll move this to stable. REPOSITORY rHG Mercurial CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7964/new/ REVISION DETAIL https://phab.mercurial-scm.org/D7964 To: spectral, #hg-reviewers Cc: pulkit, durin42, martinvonz, mercurial-devel

D7964: cext: change two more vars to Py_ssize_t in manifest.c

2020-01-22 Thread pulkit (Pulkit Goyal)
pulkit added a comment. pulkit added subscribers: martinvonz, durin42, pulkit. This should go to stable branch right? cc: @martinvonz @durin42 REPOSITORY rHG Mercurial CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D7964/new/ REVISION DETAIL

D7964: cext: change two more vars to Py_ssize_t in manifest.c

2020-01-21 Thread spectral (Kyle Lippincott)
Closed by commit rHG0e198f1f276a: cext: change two more vars to Py_ssize_t in manifest.c (authored by spectral). This revision was automatically updated to reflect the committed changes. This revision was not accepted when it landed; it landed in state "Needs Review". REPOSITORY rHG Mercurial

D7964: cext: change two more vars to Py_ssize_t in manifest.c

2020-01-21 Thread spectral (Kyle Lippincott)
spectral created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY D7913 fixed a compiler warning with a signedness conflict in a ternary operator by changing the types of some variables