ivakyb / minini

Automatically exported from code.google.com/p/minini
Other
0 stars 0 forks source link

INI_ANSIONLY has no effect #30

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Use glue-stdio
2. Compile with C project in Qt qmake mingw Windows 7x32
3. #define INI_ANSIONLY in glue-stdio.

What is the expected output? 
Sizeof(TCHAR) == 1
What do you see instead?
Sizeof(TCHAR) == 2 for minini.c, resulting in wide char reads from file.

What version of the product are you using? On what operating system?
12b, from homepage.

Please provide any additional information below.
INI_ANSIONLY has no effect. There is a reference to MININI_ANSI, which makes it 
work.

Original issue reported on code.google.com by lodd...@gmail.com on 14 Aug 2014 at 2:36

GoogleCodeExporter commented 9 years ago
Fix committed. Thanks for reporting it.

Original comment by thiadmer...@gmail.com on 15 Aug 2014 at 7:40

GoogleCodeExporter commented 9 years ago
Let me make a suggestion.
With the "#define OPTION TRUE" format all options can be listed in the glue 
header file, and you don't have to rely on documentation for details.
Using them doesn't change much, instead of #ifdef, only #if suffices.

Example (from chibios):
#if !defined(CH_NO_IDLE_THREAD) || defined(__DOXYGEN__)
#define CH_NO_IDLE_THREAD               FALSE
#endif

Original comment by lodd...@gmail.com on 15 Aug 2014 at 1:04