Re: [PATCH v1 2/2] oxenstored: make Quota.t pure

2024-02-23 Thread Andrew Cooper
On 23/02/2024 11:35 am, Edwin Torok wrote: >> On 31 Jan 2024, at 16:27, Edwin Torok wrote: >>> On 31 Jan 2024, at 11:17, Christian Lindig >>> wrote: On 31 Jan 2024, at 10:52, Edwin Török wrote: Now that we no longer have a hashtable inside we can make Quota.t pure, and push

Re: [PATCH v1 2/2] oxenstored: make Quota.t pure

2024-02-23 Thread Edwin Torok
> On 31 Jan 2024, at 16:27, Edwin Torok wrote: > > > >> On 31 Jan 2024, at 11:17, Christian Lindig >> wrote: >> >> >> >>> On 31 Jan 2024, at 10:52, Edwin Török wrote: >>> >>> Now that we no longer have a hashtable inside we can make Quota.t pure, >>> and push the mutable update to its

Re: [PATCH v1 2/2] oxenstored: make Quota.t pure

2024-01-31 Thread Edwin Torok
> On 31 Jan 2024, at 11:17, Christian Lindig wrote: > > > >> On 31 Jan 2024, at 10:52, Edwin Török wrote: >> >> Now that we no longer have a hashtable inside we can make Quota.t pure, >> and push the mutable update to its callers. >> Store.t already had a mutable Quota.t field. >> >> No

Re: [PATCH v1 2/2] oxenstored: make Quota.t pure

2024-01-31 Thread Christian Lindig
> On 31 Jan 2024, at 10:52, Edwin Török wrote: > > Now that we no longer have a hashtable inside we can make Quota.t pure, > and push the mutable update to its callers. > Store.t already had a mutable Quota.t field. > > No functional change. Acked-by: Christian Lindig This is shifting

[PATCH v1 2/2] oxenstored: make Quota.t pure

2024-01-31 Thread Edwin Török
Now that we no longer have a hashtable inside we can make Quota.t pure, and push the mutable update to its callers. Store.t already had a mutable Quota.t field. No functional change. Signed-off-by: Edwin Török --- tools/ocaml/xenstored/quota.ml | 8 tools/ocaml/xenstored/store.ml |