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