Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: RE: dreamlayers: r31089 - trunk/apps

RE: dreamlayers: r31089 - trunk/apps

From: Thomas Martitz <kugel_at_rockbox.org>
Date: Tue, 29 Nov 2011 09:54:52 +0100

> Date: 2011-11-29 05:56:42 +0100 (Tue, 29 Nov 2011)
> New Revision: 31089
>
> Log Message:
> FS#12414 : Fix directory functions in plugins on targets which HAVE_DIRCACHE. In rockbox_api, PREFIX( ) is removed around directory functions because that's now handled in directory header files. Thanks to Fred Bauer for reporting this.
>
> Modified:
> trunk/apps/plugin.c
>
> Modified: trunk/apps/plugin.c
> ===================================================================
> --- trunk/apps/plugin.c 2011-11-29 00:42:27 UTC (rev 31088)
> +++ trunk/apps/plugin.c 2011-11-29 04:56:42 UTC (rev 31089)
> _at__at_ -98,16 +98,6 _at__at_
> {
> return filesize(fd);
> }
> -
> -static int app_closedir(DIR *dirp)
> -{
> - return closedir(dirp);
> -}
> -
> -static struct dirent *app_readdir(DIR *dirp)
> -{
> - return readdir(dirp);
> -}
> #endif
>
> #if defined(HAVE_PLUGIN_CHECK_OPEN_CLOSE)&& (MAX_OPEN_FILES>32)
> _at__at_ -385,11 +375,11 _at__at_
> filetype_get_attr,
>
> /* dir */
> - (opendir_func)PREFIX(opendir),
> - (closedir_func)PREFIX(closedir),
> - (readdir_func)PREFIX(readdir),
> - PREFIX(mkdir),
> - PREFIX(rmdir),
> + (opendir_func)opendir,
> + (closedir_func)closedir,
> + (readdir_func)readdir,
> + mkdir,
> + rmdir,
> dir_exists,
> dir_get_info,
>

Okay, I admit this fix seemed obvious. But this bug is actually known
for longer (see FS#12032) and I fear it's not that easy.

When I looked into it I found that the above is the culprit indeed.
However, one needs to look into svn history (revision r28929, and more
importantly r28927) to see that the PREFIX() have been introduced for a
reason (to fix FS#11844).

That said, I haven't looked any further, so I don't know what the
correct fix is or if the above is incorrect. But I wonder if the reason
it was introduced was considered, as it bascally reverts most of r28929.

I'm sorry if I missed something obvious. I'm not asking for reversal but
for clarification.

Best regards.
Received on 2011-11-29

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy