On Tue, Aug 13, 2013 at 08:51:30AM -0400, Alon Levy wrote:
> > On Tue, Aug 13, 2013 at 10:47:29AM +0300, Alon Levy wrote:
> > > diff --git a/server/spice_image_cache.h b/server/spice_image_cache.h
> > > new file mode 100644
> > > index 0000000..f11cebc
> > > --- /dev/null
> > > +++ b/server/spice_image_cache.h
> > > @@ -0,0 +1,39 @@
> > > +#ifndef H_SPICE_IMAGE_CACHE
> > > +#define H_SPICE_IMAGE_CACHE
> > > +
> > > +#include <inttypes.h>
> > > +
> > > +#include "common/pixman_utils.h"
> > > +#include "common/canvas_base.h"
> > > +
> > > +#include "common/ring.h"
> > > +
> > > +typedef struct ImageCacheItem {
> > > + RingItem lru_link;
> > > + uint64_t id;
> > > +#ifdef IMAGE_CACHE_AGE
> > > + uint32_t age;
> > > +#endif
> > > + struct ImageCacheItem *next;
> > > + pixman_image_t *image;
> > > +} ImageCacheItem;
> > > +
> > > +#define IMAGE_CACHE_HASH_SIZE 1024
> > > +
> > > +typedef struct ImageCache {
> > > + SpiceImageCache base;
> > > + ImageCacheItem *hash_table[IMAGE_CACHE_HASH_SIZE];
> > > + Ring lru;
> > > +#ifdef IMAGE_CACHE_AGE
> > > + uint32_t age;
> > > +#else
> > > + uint32_t num_items;
> > > +#endif
> > > +} ImageCache;
> >
> > It seems it could even be possible to have an image_cache_new() function
> > which allocates memory for a new ImageCache, which would allow to make the
> > ImageCache/ImageCacheItem structures private (I only took a quick look so I
> > may have missed fundamental issues that prevent that from being possible ;)
>
> Probably, do you mind if I leave that for a further patch? this was purely a mechanical move.
Sure, fine with me to have that in an additional patch.
Christophe
Attachment:
pgpnU6gLsT5Fy.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel