Ticket #8642 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

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:1 Changed 7 years ago by Peter Peterson

  • Status changed from new to inprogress

Re #8642. Removing Kernel/Strings.h from headers that don't need it.

Changeset: ab049c02ec8c2d7ef89291a46073fe3e1f4fa079

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:

  1. Make sure it still builds
  2. 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

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​9486

Note: See TracTickets for help on using tickets.