RFR: 1452: Can never notify on first commit [v8]
Erik Joelsson
erikj at openjdk.org
Fri Oct 7 17:41:05 UTC 2022
On Fri, 7 Oct 2022 16:34:32 GMT, Zhao Song <zsong at openjdk.org> wrote:
>> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
>>
>> A better way to represent NULL_REVISION
>
> vcs/src/main/java/org/openjdk/skara/vcs/ReadOnlyRepository.java line 173:
>
>> 171: int commitCount() throws IOException;
>> 172:
>> 173: Hash initialHash();
>
> Since the initial hash is called differently in Git Repo and Hg Repo, I named the method initialHash. I don't know if this is appropriate.
I think the name is ok. I suggest adding a comment describing what it is though.
Suggestion:
/**
* Returns the special hash that references the virtual commit before the first real commit in a repository.
*/
Hash initialHash();
-------------
PR: https://git.openjdk.org/skara/pull/1385
More information about the skara-dev
mailing list