[lldb-dev] [PATCH] r124716 - make dirent usage more portable

[kirk.beitz at nokia.com]

Jai ++

I am experiencing another problem in relation to this same code:  the d_type is also not mentioned in the URL you site, and the build i am attempting under MinGW fails because of this.

This patch looks like a reasonable start.  I may have to build onto it to make it more cross-platform friendly with windows.

++ Kirk

On 2011-Feb-4, at 7:46 PM, ext Jai Menon wrote:

Hello,

Revision 124716 introduced some code to enumerate directories which
isn't quite POSIX compatible. Specifically, the 'd_namlen' entry isn't
guaranteed to be present in POSIX compatible environments [1] (for
example, on linux/glibc, the entry is 'd_reclen'). Attached patch
fixes this.


1. http://pubs.opengroup.org/onlinepubs/009695399/basedefs/dirent.h.html

--
Jai Menon
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20110205/66531a93/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lldb_dirent_posix.patch
Type: text/x-patch
Size: 1000 bytes
Desc: lldb_dirent_posix.patch
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20110205/66531a93/attachment.bin>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20110205/66531a93/attachment.htm>