martinvonz added a comment.

  In https://phab.mercurial-scm.org/D2244#37261, @indygreg wrote:
  
  > Now I'm second guessing accepting/committing this...
  
  
  Some thoughts:
  
  - This is probably not very well tested code (because it's an error path)
  - As long as we grep for TypeError and add binascii.Error in all those 
places, we're probably fine
  - Extensions need to be fixed too
  
  I think I'd prefer to define our own node.bin() with the old contract (IOW to 
wrap it)

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D2244

To: durin42, #hg-reviewers, indygreg
Cc: martinvonz, indygreg, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to