This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author Carlo_Bramini
Recipients Carlo_Bramini
Date 2010-11-28.11:17:56
SpamBayes Score 5.824563e-12
Marked as misclassified No
Message-id <1290943077.95.0.556815872588.issue10560@psf.upfronthosting.co.za>
In-reply-to
Content
Hello,
I was able to compile latest python sources with MSVC5, by importing and fixing a bit the project workspaces of MSVC6.
During this action I received some messages, so I gave a look to the sources and I would like to suggest the following fixes:

1) in posixmodule.c, I had a warning of constant out of range for secs_between_epochs.
The problem has been solved by applying "i64" suffix as suggested in MSDN for doing correct declaration of 64 bits constants.
While doing this fix, I took this chance for adding support for all currently supported platforms: MSVC, GCC and Watcom.

2) in posixmodule.c, I fixed a bit check_gfax() function:
- Function declaration has been fixed with 'void' in its parameter section (fixes a warning).
- I removed the variable 'checked' and I placed instead the handle to kernel32 module.
- I removed the L-value assignment: all other source codes in Python are already corrected in that manner, this is the last one that needs to be fixed.

3) in posixmodule.c, there is a bug in function Py_GetFileAttributesExW(): it tests for 'gfaxa' but later it uses 'gfaxw'.

4) in dl_nt.c, I fixed function prototypes with "void" (solves a warning).

5) in dl_nt.c, in function _LoadActCtxPointers(), I do not see a good reason for using widechar version of GetModuleHandle, since it just need to retrieve the instance of kernel32.

6) in dynload_win.c, I fixed _Py_ActivateActCtx prototype declaration (solves a warning).

7) in dynload_win.c, I suggest to rename local strcasecmp() to something less dangerous to prototype conflicts.

8) in dynload_win.c, in function _PyImport_GetDynLoadFunc() the first 'cookie' variable is not used, so its declaration has been removed.

I hope you will find this useful.

Sincerely,

Carlo Bramini.
History
Date User Action Args
2010-11-28 11:17:58Carlo_Braminisetrecipients: + Carlo_Bramini
2010-11-28 11:17:57Carlo_Braminisetmessageid: <1290943077.95.0.556815872588.issue10560@psf.upfronthosting.co.za>
2010-11-28 11:17:56Carlo_Braminilinkissue10560 messages
2010-11-28 11:17:56Carlo_Braminicreate