Ticket #8642 (closed: fixed)
Remove unnecessary includes from header files
Reported by: | Peter Peterson | Owned by: | Peter Peterson |
---|---|---|---|
Priority: | major | Milestone: | Release 3.1 |
Component: | Framework | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Andrei Savici |
Description
This is just a time-box for the work concentrating on Kernel. Since having less includes in the headers means that less will be brought in by the pre-processor before compiling and should speed up compilation.
Change History
comment:2 Changed 7 years ago by Peter Peterson
Re #8642. Removing iostream includes from headers.
Changeset: fde53aee35f2b8c51ed66609019f693eaa7891d6
comment:3 Changed 7 years ago by Peter Peterson
Re #8642. Cleaned up DateAndTime.h
Changeset: f2dcba025017cc24943b0ab00bd98b914683ffc7
comment:4 Changed 7 years ago by Peter Peterson
Re #8642. Forgot windoze export macro.
Changeset: e0ebdca041686b012cfa1193598aab7d7cec1278
comment:5 Changed 7 years ago by Peter Peterson
- Status changed from inprogress to verify
- Resolution set to fixed
This is on feature/8642_header_work.
To test:
- Make sure it still builds
- Look at the changeset to see that there is less in the headers that were edited.
Since this was a time-boxing ticket there is a lot left to do.
comment:6 Changed 7 years ago by Andrei Savici
- Status changed from verify to verifying
- Tester set to Andrei Savici
comment:7 Changed 7 years ago by Andrei Savici
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/8642_header_work'
Full changeset: c52d00d7ccdafba700cc7146ab2c4a38f97ee199
Re #8642. Removing Kernel/Strings.h from headers that don't need it.
Changeset: ab049c02ec8c2d7ef89291a46073fe3e1f4fa079