Caching icon theme information
Matthias Clasen
mclasen at redhat.com
Thu Oct 14 08:48:35 EEST 2004
On Wed, 2004-10-13 at 17:22 -0400, Owen Taylor wrote:
> On Wed, 2004-10-13 at 16:17 -0400, Matthias Clasen wrote:
>
> > Header:
> > 2 CARD16 MAJOR_VERSION 1
> > 2 CARD16 MINOR_VERSION 0
> > 4 CARD32 HASH_OFFSET
> > 4 CARD32 DIRECTORY_LIST_OFFSET
> >
> > DirectoryList:
> > 4 CARD32 N_DIRECTORIES
> > 4*N_DIRECTORIES CARD32 DIRECTORY_OFFSET
>
> Hmm, a definition of Directory seems to be missing here,
> or alternatively there should be some clarification that
> what is pointed to is the relative path of the subdirectory
> as a string.
>
Ah, yes. I cut out that extra indirection. Maybe I should have
renamed that offset to PATH_OFFSET to make it clear that it directly
points to the string holding the relative path of the directory.
> > ImageList:
> > 4 CARD32 N_IMAGES
> > 4*N_IMAGES Image IMAGES
>
> This looks like it should be 8*N_IMAGES with the change to
> include IMAGE_DATA_OFFSET.
Good catch. The code already uses 8.
Matthias
More information about the xdg
mailing list