Should nbtime.h have #include <time.h>?

Discussion to talk about software related topics only.
Post Reply
joepasquariello
Posts: 85
Joined: Mon Apr 28, 2008 7:32 am

Should nbtime.h have #include <time.h>?

Post by joepasquariello »

I'm just now moving up from 2.6.7 to 2.7.5, and found new files nbtime.cpp and nbtime.h, which is nice. nbtime.h references type time_t, but has no #include <time.h> to define it, so files that #include <nbtime.h> must also #include <time.h>. Should nbtime.h #include <time.h>, or was it left out for a reason?

Joe
User avatar
dciliske
Posts: 624
Joined: Mon Feb 06, 2012 9:37 am
Location: San Diego, CA
Contact:

Re: Should nbtime.h have #include <time.h>?

Post by dciliske »

I believe you are correct...
Dan Ciliske
Project Engineer
Netburner, Inc
joepasquariello
Posts: 85
Joined: Mon Apr 28, 2008 7:32 am

Re: Should nbtime.h have #include <time.h>?

Post by joepasquariello »

Thanks for the reply, Dan. Obviously a minor thing. Is there anything I should do to get it on a low-priority to-do list?

Joe
User avatar
dciliske
Posts: 624
Joined: Mon Feb 06, 2012 9:37 am
Location: San Diego, CA
Contact:

Re: Should nbtime.h have #include <time.h>?

Post by dciliske »

Exactly what you did. It'll be in the next release...
Dan Ciliske
Project Engineer
Netburner, Inc
Post Reply