Ticket #1349 (closed: fixed)
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:10 Changed 10 years ago by Stuart Campbell
comment:11 Changed 10 years ago by Stuart Campbell
comment:12 Changed 10 years ago by Stuart Campbell
comment:13 Changed 10 years ago by Stuart Campbell
comment:14 Changed 10 years ago by Stuart Campbell
comment:15 Changed 10 years ago by Stuart Campbell
comment:16 Changed 10 years ago by Stuart Campbell
comment:17 Changed 10 years ago by Stuart Campbell
comment:18 Changed 10 years ago by Stuart Campbell
comment:19 Changed 10 years ago by Stuart Campbell
comment:20 Changed 10 years ago by Stuart Campbell
comment:21 Changed 10 years ago by Stuart Campbell
comment:22 Changed 10 years ago by Stuart Campbell
comment:23 Changed 10 years ago by Stuart Campbell
comment:24 Changed 10 years ago by Stuart Campbell
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:29 Changed 10 years ago by Stuart Campbell
comment:30 Changed 10 years ago by Stuart Campbell
comment:31 Changed 10 years ago by Stuart Campbell
comment:32 Changed 10 years ago by Stuart Campbell
comment:33 Changed 10 years ago by Stuart Campbell
comment:34 Changed 10 years ago by Stuart Campbell
comment:35 Changed 10 years ago by Stuart Campbell
comment:36 Changed 10 years ago by Russell Taylor
comment:37 Changed 10 years ago by Stuart Campbell
comment:38 Changed 10 years ago by Stuart Campbell
comment:39 Changed 10 years ago by Stuart Campbell
comment:40 Changed 10 years ago by Stuart Campbell
comment:41 Changed 10 years ago by Stuart Campbell
comment:42 Changed 10 years ago by Stuart Campbell
comment:43 Changed 10 years ago by Stuart Campbell
comment:44 Changed 10 years ago by Stuart Campbell
comment:45 Changed 10 years ago by Stuart Campbell
comment:46 Changed 10 years ago by Stuart Campbell
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
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.