From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eli Zaretskii To: gdb@sources.redhat.com Cc: ac131313@cygnus.com Subject: DOS/Windows-specific code: source.c Date: Tue, 08 May 2001 04:47:00 -0000 Message-id: <200105081149.OAA06252@is.elta.co.il> References: <20010503211502.21716.qmail@web6401.mail.yahoo.com> <3AF1DAA0.3060702@cygnus.com> <200105071609.TAA24129@is.elta.co.il> X-SW-Source: 2001-05/msg00106.html * source.c:openp() #ifdef _WIN32 mode |= O_BINARY; #endif I think this is obsolete and should be removed: defining CRLF_SOURCE_FILES in xm-whatever should solve the underlying problem. For those who don't know: the problem here is that find_source_lines creates a table which records the byte position of each line in a source file, but many DOS/Windows libraries don't DTRT with byte offsets unless the file is open in binary mode. So this issue is limited to source files, and CRLF_SOURCE_FILES should provide the solution there for Windows systems. (Code which wants to open binary files, like in exec.c, already uses O_BINARY when it calls openp. Hmm, perhaps solib.c should be fixed to use O_BINARY when it looks for the libraries.)