mirrors/jj
1
0
Fork 1
mirror of https://github.com/martinvonz/jj.git synced 2025-01-07 21:26:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 14

templater: store type-erased version of commit/change id

Browse source 
It's unlikely we'll need to discriminate commit/change id types while
templating, so let's unify them. Since ObjectId trait isn't object safe,
I decided to simply store bytes instead of Box<dyn ObjectId>.
This commit is contained in:
Yuya Nishihara 2023-01-19 13:25:18 +09:00
parent 0f4269a141
commit d6c6cdb45c
1 changed files with 6 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
src/templater.rs
View file

@ -18,7 +18,7 @@ use std::io;
use std::ops::AddAssign;
use itertools::Itertools;
use jujutsu_lib::backend::{ChangeId, CommitId, ObjectId, Signature, Timestamp};
use jujutsu_lib::backend::{ChangeId, ObjectId, Signature, Timestamp};
use jujutsu_lib::commit::Commit;
use jujutsu_lib::op_store::WorkspaceId;
use jujutsu_lib::repo::RepoRef;
@ -407,18 +407,13 @@ impl<'a, C, I, O> TemplateProperty<C, O> for TemplateFunction<'a, C, I, O> {
}
}
/// Type-erased `CommitId`/`ChangeId`.
#[derive(Debug, Clone)]
pub enum CommitOrChangeId {
CommitId(CommitId),
ChangeId(ChangeId),
}
pub struct CommitOrChangeId(Vec<u8>);
impl CommitOrChangeId {
pub fn hex(&self) -> String {
match self {
CommitOrChangeId::CommitId(id) => id.hex(),
CommitOrChangeId::ChangeId(id) => id.hex(),
}
hex::encode(&self.0)
}
pub fn short(&self) -> String {
@ -470,7 +465,7 @@ pub struct CommitIdProperty;
impl TemplateProperty<Commit, CommitOrChangeId> for CommitIdProperty {
fn extract(&self, context: &Commit) -> CommitOrChangeId {
CommitOrChangeId::CommitId(context.id().clone())
CommitOrChangeId(context.id().to_bytes())
}
}
@ -478,7 +473,7 @@ pub struct ChangeIdProperty;
impl TemplateProperty<Commit, CommitOrChangeId> for ChangeIdProperty {
fn extract(&self, context: &Commit) -> CommitOrChangeId {
CommitOrChangeId::ChangeId(context.change_id().clone())
CommitOrChangeId(context.change_id().to_bytes())
}
}