What's wrong with the definition of UINT32?

Question

0.00/5 (No votes)

See more:

VS2012

I've the following code snippet:

C++

typedef unsigned long UINT32;

typedef struct
{
	char 		str[128];
	UINT32	mSec;
				
} SUB_SEQ_LINK;

I've got compilation error using VS2012/C++ to compile it:

C#

D:\programs\ATP-3\ATP3MMI\MMI\Include\GlobalData.h(294): error C2371: 'UINT32' : redefinition; different basic types
1>          C:\Program Files (x86)\Windows Kits\8.0\Include\shared\basetsd.h(76) : see declaration of 'UINT32'

But if I remove the line of "

C++

typedef unsigned long UINT32;"

I've got error:

C#

1>D:\programs\ATP-3\ATP3MMI\MMI\Include\GlobalData.h(299): error C2146: syntax error : missing ';' before identifier 'mSec'
1>D:\programs\ATP-3\ATP3MMI\MMI\Include\GlobalData.h(299): error C4430: missing type specifier - int assumed. Note: C++ does not support default-int

It's ridiculous. In the former case, it complained that I redefined UINT32, which is already defined at C:\Program Files (x86)\Windows Kits\8.0\Include\shared\basetsd.h, although I didn't find any inclusion in my codes for this header file.

In later case, it seems complaining that UINT32 is undefined.
What confuses me is that if I define UINT32 at my file, it complained I redefined it. If I remove UINT32 definition, it complained UINT32 is undefined term.

Anyone knows why and can help me how to do it?

What I have tried:

I googled the error message and found the explanation does not fit my situation. I examined my codes and I didn't find any inclusion for C:\Program Files (x86)\Windows Kits\8.0\Include\shared\basetsd.h. How come it comes into discussion?

Posted 23-Mar-16 22:54pm

Stan Huang

Updated 24-Mar-16 6:03am

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Jochen Arndt · Answer 1 · 2016-03-23T23:18:00

Solution 1

You are including your file GlobalData.h in different source files. Some of them include basetsd.h (which will throw a redefinition error when including GlobalData.h) and some do not (which will throw the missing type error when UINT32 is not defined in GlobalData.h).

The solution is to include basetsd.h on top of GlobalData.h and remove the type definition.

Posted 23-Mar-16 23:18pm

Jochen Arndt

Updated 24-Mar-16 0:06am

v2

Comments

Stan Huang 24-Mar-16 5:53am

Actually, what I did is to port a VxWorks project into Windows. I still didn't do any significant updates. It means all inclusions must be for standard C or from VxWorks/Tornado. I didn't update code to include basetsd.h. That's the reason why I got confused why basetsd.h was included.

I can't find Global.h & basetsd.h in my files.

Jochen Arndt 24-Mar-16 6:05am

Sorry, I used Global.h instead of GlobalData.h and basedts.h instead of basetsd.h. I will update the solution.

The file basetsd.h may be not included explicitely but by including other header files. AT least it is included accroding to the error message.

PJ Arends · Answer 2 · 2016-03-24T06:03:00

Solution 2

First off, UINT32 is an unsigned int, not unsigned long. That may be why you are getting redefinition errors.

You could put your definition inside a #ifndef protection block.

C++

#ifndef UINT32
#define UINT32 unsigned int
#endif

Posted 24-Mar-16 6:03am

PJ Arends

Updated 24-Mar-16 6:34am

v2

Comments

PJ Arends 24-Mar-16 18:03pm

Thanks Matt T Heffron for updating my answer. I posted from my phone and was unable to fix it myself.

Stan Huang 30-Mar-16 2:05am

What you said is partially right: redefinition error came from defining as 'unsigned long', but it should be 'unsigned int'. UINT32 is defined by typedef, not #define, so it can't be dealt as you said. Instead, I used 'typedef unsigned int UINT32;' to deal with it. At least, it works. I still don't know why it complained 'undefined' if I removed such tyepdef.