[PATCH] Support for animated cursors

[Juan Lang]

> Ok, attached, and fixed an issue with setting delay on the last frame.

+static ANIHEADER *anih;
+static LPBYTE *ani_frames;
+static DWORD ani_frame_idx;
+
+static void decodeRIFF(LPBYTE bytes, DWORD size)
+{
+    CK *chunk = (CK*)bytes;
+    CKSIZE sizeWithPad = chunk->ckSize % 2 ? chunk->ckSize + 1 : chunk->ckSize;
+    switch(chunk->ckID)
+    {
+    case 0x46464952:  /* RIFF */
+        decodeRIFF(chunk->ckData + 4, chunk->ckSize - 4);

Like Henri already said, you really can't use globals for anih,
ani_frames, and ani_frame_idx.  These should be in a structure, and
decodeRIFF should write into that structure, not into globals.  The
structure should then be associated with the icon.

+                    HICON hNextIcon = CreateIconFromResourceEx(
&bits[entry->dwDIBOffset], entry->dwDIBSize, TRUE, 0x00030000, 0, 0, 0
);

This may be obvious to other, but certainly not to me:  what does
0x00030000 mean?  You need to use symbolic constants here.

+        /*if (CURSORICONINFO_NEXT(info) && !animate_timer_started) {
+            animate_timer_started = TRUE;
+            SetTimer(NULL, 0, 55, animate_cursor);
+        }*/

Please don't include dead code.
--Juan