Thread (8 messages) flat view 8 messages, 2 authors, 2016-06-15

Re: [PATCH 2/2] decorate: add "clear_decoration()"

From: Jeff King <hidden>
Date: 2016-06-15 22:56:44

On Sun, Apr 07, 2013 at 11:14:17PM -0700, Junio C Hamano wrote:
So far, all the users of the decoration API used decoration that
only grows and discarded at the end of the program execution.

Introduce for_each_decoration() that lets the caller iterate over
all defined decorations and use it to implement clear_decoration()
function.

Signed-off-by: Junio C Hamano <redacted>
Both this and the first patch look good to me. I had documented
api-decorate long ago as part of the cache-metadata-on-disk series (it
was built on generic maps, which I also refactored decorate on top of).
But I didn't see anything in my version that was missing from yours (in
fact, it was much sparser, because it just referred to the generic map
api, which we never merged).

Out of curiosity: where is the patch 3 that presumably led you to
wanting these?

-Peff
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help