Ticket #1349 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Fix include statements to use <> for system includes.

Reported by: Stuart Campbell Owned by: Stuart Campbell
Priority: minor Milestone: Iteration 28
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description

There are a number of installed header files that are being included using "" rather than <>

Change History

comment:1 Changed 10 years ago by Stuart Campbell

  • Summary changed from Fix import statements to use <> for system includes. to Fix include statements to use <> for system includes.

comment:2 Changed 10 years ago by Nick Draper

  • Owner set to Stuart Campbell
  • Status changed from new to assigned

comment:3 Changed 10 years ago by Stuart Campbell

  • Component set to Mantid
  • Milestone changed from Iteration 25 to Iteration 26

comment:4 Changed 10 years ago by Stuart Campbell

  • Status changed from assigned to accepted
  • Milestone changed from Iteration 26 to Iteration 27

comment:5 Changed 10 years ago by Stuart Campbell

(In [9205]) refs #1349

comment:6 Changed 10 years ago by Stuart Campbell

(In [9206]) refs #1349

comment:7 Changed 10 years ago by Stuart Campbell

(In [9207]) refs #1349

comment:8 Changed 10 years ago by Stuart Campbell

(In [9208]) refs #1349

comment:9 Changed 10 years ago by Stuart Campbell

(In [9210]) refs #1349

comment:10 Changed 10 years ago by Stuart Campbell

(In [9211]) refs #1349

comment:11 Changed 10 years ago by Stuart Campbell

(In [9212]) refs #1349

comment:12 Changed 10 years ago by Stuart Campbell

(In [9215]) refs #1349

comment:13 Changed 10 years ago by Stuart Campbell

(In [9216]) refs #1349

comment:14 Changed 10 years ago by Stuart Campbell

(In [9218]) refs #1349

comment:15 Changed 10 years ago by Stuart Campbell

(In [9221]) refs #1349

comment:16 Changed 10 years ago by Stuart Campbell

(In [9222]) refs #1349

comment:17 Changed 10 years ago by Stuart Campbell

(In [9223]) refs #1349

comment:18 Changed 10 years ago by Stuart Campbell

(In [9224]) refs #1349

comment:19 Changed 10 years ago by Stuart Campbell

(In [9225]) refs #1349

comment:20 Changed 10 years ago by Stuart Campbell

(In [9226]) refs #1349

comment:21 Changed 10 years ago by Stuart Campbell

(In [9227]) refs #1349

comment:22 Changed 10 years ago by Stuart Campbell

(In [9228]) refs #1349

comment:23 Changed 10 years ago by Stuart Campbell

(In [9229]) refs #1349

comment:24 Changed 10 years ago by Stuart Campbell

(In [9230]) refs #1349

comment:25 Changed 10 years ago by Stuart Campbell

  • Status changed from accepted to verify
  • Resolution set to fixed

comment:26 Changed 10 years ago by Janik Zikovsky

  • Status changed from verify to verifying
  • Tester set to Janik Zikovsky

comment:27 Changed 10 years ago by Janik Zikovsky

  • Status changed from verifying to reopened
  • Resolution fixed deleted
  • Milestone changed from Iteration 27 to Iteration 28

Search for 'include "Poco' and 'include "Boost' reveals 52 and 70 matches, respectively; but this can wait until the next iteration.

comment:28 Changed 10 years ago by Stuart Campbell

  • Status changed from reopened to accepted

comment:29 Changed 10 years ago by Stuart Campbell

(In [9847]) refs #1349

comment:30 Changed 10 years ago by Stuart Campbell

(In [9849]) refs #1349

comment:31 Changed 10 years ago by Stuart Campbell

(In [9850]) refs #1349

comment:32 Changed 10 years ago by Stuart Campbell

(In [9856]) refs #1349

comment:33 Changed 10 years ago by Stuart Campbell

(In [9857]) refs #1349

comment:34 Changed 10 years ago by Stuart Campbell

(In [9858]) refs #1349

comment:35 Changed 10 years ago by Stuart Campbell

(In [9859]) refs #1349

comment:36 Changed 10 years ago by Russell Taylor

(In [9861]) Re #1349.

comment:37 Changed 10 years ago by Stuart Campbell

(In [10447]) More changes. refs #1349

comment:38 Changed 10 years ago by Stuart Campbell

(In [10448]) All Poco done. refs #1349

comment:39 Changed 10 years ago by Stuart Campbell

(In [10449]) All Poco done. refs #1349

comment:40 Changed 10 years ago by Stuart Campbell

(In [10450]) change include for boost headers in vates to use <>. refs #1349

comment:41 Changed 10 years ago by Stuart Campbell

(In [10451]) change include for boost headers in MantidPlot to use <>. refs #1349

comment:42 Changed 10 years ago by Stuart Campbell

(In [10452]) change include for boost headers in NeXus to use <>. refs #1349

comment:43 Changed 10 years ago by Stuart Campbell

(In [10453]) change include for boost headers in MDAlgorithms to use <>. refs #1349

comment:44 Changed 10 years ago by Stuart Campbell

(In [10454]) change include for boost headers in Kernel to use <>. refs #1349

comment:45 Changed 10 years ago by Stuart Campbell

(In [10455]) change include for boost headers in DataObjects to use <>. refs #1349

comment:46 Changed 10 years ago by Stuart Campbell

(In [10456]) change include for boost headers in API to use <>. refs #1349

comment:47 Changed 10 years ago by Stuart Campbell

  • Status changed from accepted to verify
  • Resolution set to fixed

As of now, there are no Poco and boost headers included using ""...

...but I am sure that by the time this is tested, there will be! :-)

comment:48 Changed 9 years ago by Michael Whitty

  • Status changed from verify to verifying
  • Tester changed from Janik Zikovsky to Michael Whitty

comment:49 Changed 9 years ago by Michael Whitty

(In [11105]) refs #1349 - Fix include statements to use <> for system includes.

comment:50 Changed 9 years ago by Michael Whitty

  • Status changed from verifying to closed

now no files under \Code\Mantid have "" instead of <> for boost or poco.

comment:51 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2196

Note: See TracTickets for help on using tickets.