Ticket #8728 (closed: fixed)
Make a use of EXPORT_OPT_MANTID_ICAT macro
Reported by: | Arturs Bekasovs | Owned by: | Jay Rainey |
---|---|---|---|
Priority: | minor | Milestone: | Release 3.2 |
Component: | Framework | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Martyn Gigg |
Description
EXPORT_OPT_MANTID_ICAT macro defined in ICatExport.h might be used instead of DLLExport for all the ICat classes. This might provide a better optimization potential on Windows and seems to be more consistent.
Change History
comment:1 Changed 7 years ago by Jay Rainey
- Summary changed from [ICAT] Make a use of EXPORT_OPT_MANTID_ICAT macro to Make a use of EXPORT_OPT_MANTID_ICAT macro
comment:3 Changed 7 years ago by Nick Draper
- Status changed from new to assigned
Bulk move of tickets out of triage (new) to assigned at the introduction of the triage state
comment:4 Changed 7 years ago by Jay Rainey
- Status changed from assigned to inprogress
Remove unused DLL Export file. Refs #8728.
Changeset: 774dc6007954568aab8f34b1d36137061e9ea76e
comment:5 Changed 7 years ago by Jay Rainey
Added new DLLConfig file for ICAT. Refs #8728.
Changeset: 3de2e54fc96d2da8bbecfe0d9e1b3ead5f07cfd2
comment:6 Changed 7 years ago by Jay Rainey
Use correct export in catalog algorithms. Refs #8728.
Changeset: ee5bf41e06858b450bdd1049cbe1da903faa8493
comment:7 Changed 7 years ago by Jay Rainey
Merge branch 'feature/8728_icat_import_consistency' into develop
Changeset: f86b83619340c348b8a0cbc653355e2c733f9038
comment:8 Changed 7 years ago by Jay Rainey
- Status changed from inprogress to verify
- Resolution set to fixed
This was a minor consistency change that dealt changing DLLExport in the catalog algorithms to MANTID_ICAT_DLL.
To test
Test by code review to ensure the catalog algorithms (listed here) now make use of the new DLLConfig macro. (see: this commit).
comment:9 Changed 7 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester set to Martyn Gigg
comment:10 Changed 7 years ago by Martyn Gigg
This is me being very picky but could we use lower case Ls in the filename, i.e. DllConfig.h. It's more consistent with all of the others in the framework then.
comment:11 Changed 7 years ago by Jay Rainey
Rename DLLConfig to Dllconfig. Refs #8728.
Changeset: 0bf44a48d4c153e02187660174d9c6f005bd2f64
comment:12 Changed 7 years ago by Jay Rainey
Merge branch 'feature/8728_icat_import_consistency' into develop
- feature/8728_icat_import_consistency: Rename DLLConfig to Dllconfig. Refs #8728.
Changeset: 1719fc71b552cbd8430a23769851e472856b71b3
comment:13 Changed 6 years ago by Nick Draper
- Status changed from verifying to verify
- Tester Martyn Gigg deleted
comment:14 Changed 6 years ago by Martyn Gigg
- Status changed from verify to verifying
- Tester set to Martyn Gigg
comment:15 Changed 6 years ago by Martyn Gigg
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/8728_icat_import_consistency'
Full changeset: a7b0192001d16f87f32e8283134bb1b30cadc8e5
comment:16 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 9572