marmoute added inline comments.

INLINE COMMENTS

> martinvonz wrote in nodemap.py:47-49
> The actual semantic of this code (the Python code) is to serialize the data. 
> To me, "persist" implies writing it to disk, which is not what this code 
> does. I think it's just misleading to call it that even if the Rust code will 
> actually be writing directly to disk.

On my side, I feel like `serialize` is misleading because this is not meant to 
have a "deserialization" phase taht reconstruct the data in memory.

Can we wait for the bulk of the code to be in before we decide on a name and do 
a (simpler) changeset to align things to the naming scheme?

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D7834/new/

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

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

Reply via email to