Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compile issues with GCC4.4.3 on linux. #2

Open
GoogleCodeExporter opened this issue Jan 20, 2016 · 2 comments
Open

Compile issues with GCC4.4.3 on linux. #2

GoogleCodeExporter opened this issue Jan 20, 2016 · 2 comments

Comments

@GoogleCodeExporter
Copy link

1. Compile ptp2d with on a gcc 4.4.3 on Linux.  Fails compile due to INT_MAC 
not being defined.  

I added this statement in constants_dep.h to fix the problem.  

#ifndef INT_MAX 
#define INT_MAX __INT_MAX__
#endif

2. A compile also causes a warning with a */ included in a comment.  

A */ should be added at the end of the 2nd line.  

/* System Dependent (AKB: Moved from constants.h to constants_dep.h */
/* Defines for MPC831X implementation with Hardware timestampinng

Original issue reported on code.google.com by msab...@gmail.com on 19 May 2011 at 6:49

@GoogleCodeExporter
Copy link
Author

1. Using the C standard library, instead of #ifndef INT_MAX, you could just

#include <limits.h>

Original comment by czing...@gmail.com on 31 May 2011 at 9:47

@GoogleCodeExporter
Copy link
Author

Thanks for the change.  I'll take a look at this.

Of the 2 suggestions, it appears the include limits.h option appears to be the 
best.

Before making the change I want to look at what Windows supports for printf as 
I'm trying to keep as much as possible common.

Apologies for not responding sooner as I've been mostly focusing comments and 
responses from the ptpv2d google code group.  To join, go to the page, enter 
the info and please make sure you give a short description that you are working 
on ptpv2d (so I can filter out junk mailers).

Thanks,

Alan

Original comment by a...@bartky.net on 9 Jul 2011 at 11:59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant