Ticket #3302 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Iteration 30 Warning Fixes.

Reported by: Janik Zikovsky Owned by: Nick Draper
Priority: major Milestone: Iteration 30
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description (last modified by Stuart Campbell) (diff)

General ticket for compiler warning fixes

Change History

comment:1 Changed 9 years ago by Stuart Campbell

  • Description modified (diff)

comment:2 Changed 9 years ago by Martyn Gigg

(In [12992]) Minor tweak to CMAKE_BUILD_TYPE so that the options in the GUI are listed rather than requiring a string. Re #3302

comment:3 Changed 9 years ago by Janik Zikovsky

(In [13051]) Fixes #3321: deprecated DiffractionFocussing1. Fix tests in release version (refs #3203). Fix a few warnings Refs #3302.

comment:4 Changed 9 years ago by Janik Zikovsky

(In [13055]) Refs #3302: Fix several warnings.

comment:5 Changed 9 years ago by Russell Taylor

In [13131]:

Eliminate a ubiquitous Intel compiler warning. Re #3302.

comment:6 Changed 9 years ago by Russell Taylor

In [13139]:

Fix Intel compiler warnings. Re #3302.

comment:7 Changed 9 years ago by Janik Zikovsky

In [13152]:

Refs #3302: Fix warning.

comment:8 Changed 9 years ago by Janik Zikovsky

In [13153]:

Refs #3302: Fix last 2 warnings.

comment:9 Changed 9 years ago by Janik Zikovsky

In [13154]:

Refs #3302: Fix warning without breaking test.

comment:10 Changed 9 years ago by Russell Taylor

In [13159]:

Clear Intel warnings. Re #3302.

comment:11 Changed 9 years ago by Janik Zikovsky

In [13192]:

Refs #3302: Warning fix.

comment:12 Changed 9 years ago by Russell Taylor

In [13202]:

Kill off more Intel compiler warnings. Re #3302.

comment:13 Changed 9 years ago by Russell Taylor

In [13232]:

Remove Intel compiler warnings coming out of Boost by using the -isystem flag when including the headers. Have to enable this possibility in CMake as by default it doesn't use it on the Mac. Re #3302.

comment:14 Changed 9 years ago by Janik Zikovsky

In [13244]:

Refs #3302: Warnings

comment:15 Changed 9 years ago by Russell Taylor

In [13277]:

Clear last remaining Intel warnings related to CurveFitting. Fixes #3352. Clear some other Intel warnings as well. Re #3302.

comment:16 Changed 9 years ago by Roman Tolchenov

In [13406]:

Clearing some warnings. re #3302

comment:17 Changed 9 years ago by Nick Draper

In [13525]:

re #3302
Version number change

comment:18 Changed 9 years ago by Russell Taylor

In [13532]:

Clean namespace pollution. Re #3302.

comment:19 Changed 9 years ago by Michael Reuter

In [13641]:

Subduing weird compiler warning. This refs #3302.

comment:20 Changed 9 years ago by Martyn Gigg

In [13675]:

Refs #3302. Avoid a warning with CMake on the Mac.

comment:21 Changed 9 years ago by Andrei Savici

In [13683]:

Fixes some warnings. Refs #3302

comment:22 Changed 9 years ago by Russell Taylor

In [13772]:

Clear a few Intel compiler warnings. Re #3302.

comment:23 Changed 9 years ago by Janik Zikovsky

In [13787]:

Refs #3302: Fix some compiler warnings on the Mac build.

comment:24 Changed 9 years ago by Russell Taylor

In [13794]:

Chipping away at intel compiler warnings. Re #3302.

comment:25 Changed 9 years ago by Russell Taylor

In [13822]:

Chipping away at intel compiler warnings. Re #3302.

comment:26 Changed 9 years ago by Russell Taylor

In [13843]:

More Intel compiler warnings gone. Should only be 1 left now (excluding tests). Re #3302.

comment:27 Changed 9 years ago by Russell Taylor

In [13844]:

Fix typo to squash last Intel compiler warning. Re #3302.

comment:28 Changed 9 years ago by Russell Taylor

In [13859]:

Remove an ld warning on the Mac & also downgrade a LoadInstrument logging message that was too prominent. Re #3302.

comment:29 Changed 9 years ago by Martyn Gigg

In [13885]:

Refs #3302. Trying to generate Python tests again that don't seem to update

comment:30 Changed 9 years ago by Martyn Gigg

In [13887]:

Refs #3302. Fix a dependency issue when building the python unit tests.

comment:31 Changed 9 years ago by Martyn Gigg

In [13889]:

Refs #3302. Will this do it?

comment:32 Changed 9 years ago by Russell Taylor

In [13895]:

Starting on Intel warnings in the tests. This should immediately get rid of most of them. Re #3302.

comment:33 Changed 9 years ago by Russell Taylor

In [13896]:

This should be most of the other Intel warnings in the tests. Re #3302.

comment:34 Changed 9 years ago by Janik Zikovsky

In [13901]:

Refs #3302: Fix warnings

comment:35 Changed 9 years ago by Janik Zikovsky

In [13902]:

Refs #3532: Track box total signal whenever you save it to disk. Refs #3302: Hope to really fix warnings now.

comment:17 Changed 9 years ago by Russell Taylor

In [13903]:

Re #3302.

comment:18 Changed 9 years ago by Russell Taylor

In [13909]:

Should only be 1 Intel warning left now. Re #3302.

comment:19 Changed 9 years ago by Roman Tolchenov

In [13917]:

Removing a warning. re #3302

comment:20 Changed 9 years ago by Martyn Gigg

In [13955]:

Refs #3302. Clear MSVC compiler warnings regarding bit-shift operators.

comment:21 Changed 9 years ago by Martyn Gigg

In [13956]:

Refs #3302. Fix tests

comment:22 Changed 9 years ago by Martyn Gigg

In [13957]:

Refs #3302. Okay, leave that warning for now.

comment:23 Changed 9 years ago by Martyn Gigg

In [13958]:

Refs #3302. Fix warning properly.

comment:24 Changed 9 years ago by Russell Taylor

In [13998]:

Try to fix Mac test failures. Looks to be code in the destructor of another test causing the problem. Re #3302.

comment:25 Changed 9 years ago by Michael Reuter

In [14154]:

Taking care of some VATES warnings. This refs #3302.

comment:26 Changed 9 years ago by Russell Taylor

In [14216]:

Clear new Intel compiler warnings. Re #3302.

comment:27 Changed 9 years ago by Nick Draper

In [14299]:

re #3302
dropping a const to remove warnings

comment:28 Changed 9 years ago by Nick Draper

In [14301]:

re #3302
quashing some unused variable warnings from the tests

comment:29 Changed 9 years ago by Peter Peterson

In [14304]:

Quieting implicit cast warning. Refs #3302.

comment:30 Changed 9 years ago by Michael Reuter

In [14311]:

Windows warning fixes. This refs #3302.

comment:31 Changed 9 years ago by Nick Draper

  • Owner set to Anyone
  • Status changed from new to assigned

comment:32 Changed 9 years ago by Michael Reuter

In [14349]:

Using MS suggestion to fix warnings. This refs #3302.

comment:33 Changed 9 years ago by Michael Reuter

In [14360]:

More Windows warnings. This refs #3302.

comment:34 Changed 9 years ago by Janik Zikovsky

In [14363]:

Refs #3302: class_maker test improvement

comment:35 Changed 9 years ago by Russell Taylor

In [14364]:

eol. Re #3302.

comment:36 Changed 9 years ago by Martyn Gigg

In [14417]:

Refs #3302. Activate windows compiler warning that should make MSVC treat signed/unsigned comparisons in the same manner as gcc.

comment:37 Changed 9 years ago by Roman Tolchenov

In [14455]:

A file counter hit a limit. A quick fix to extend the limit. re #3302

comment:38 Changed 9 years ago by Stuart Campbell

In [14472]:

cppcheck fixes. refs #3302

comment:39 Changed 9 years ago by Stuart Campbell

In [14473]:

Change to catch by reference. refs #3302

comment:40 Changed 9 years ago by Stuart Campbell

In [14474]:

Change to catch by reference. refs #3302

comment:41 Changed 9 years ago by Stuart Campbell

In [14475]:

Change to catch by reference. refs #3302

comment:42 Changed 9 years ago by Peter Peterson

In [14476]:

Fixing postfixOperator warnings. Refs #3302.

comment:43 Changed 9 years ago by Stuart Campbell

In [14477]:

Change to catch by reference. refs #3302

comment:44 Changed 9 years ago by Peter Peterson

In [14479]:

Fixing postfixOperator warnings. Refs #3302.

comment:45 Changed 9 years ago by Martyn Gigg

In [14485]:

Export the access functions for the oriented lattice from the sample class. Fixes #3680. Also cleared some warnings on windows. Refs #3302

comment:2 Changed 9 years ago by Martyn Gigg

In [14488]:

Kill some prevalent warnings on Windows. Refs #3302.

comment:3 Changed 9 years ago by Peter Peterson

In [14495]:

Fixing cppcheck postfix warnings. Refs #3302.

comment:4 Changed 9 years ago by Peter Peterson

In [14497]:

Refs #3302

comment:5 Changed 9 years ago by Peter Peterson

In [14498]:

Postfix operator again. Refs #3302.

comment:6 Changed 9 years ago by Martyn Gigg

In [14523]:

Kill remaining gcc compiler warnings. Refs #3302

comment:7 Changed 9 years ago by Martyn Gigg

In [14526]:

Further windows warnings cull Refs #3302

comment:8 Changed 9 years ago by Martyn Gigg

In [14527]:

Fix the win32 build problem. Refs #3302

comment:9 Changed 9 years ago by Martyn Gigg

In [14590]:

Refs #3302. Fix build.

comment:10 Changed 9 years ago by Martyn Gigg

In [14591]:

Refs #3302. Fix build.

comment:11 Changed 9 years ago by Martyn Gigg

In [14607]:

Fix for PYTHON_EXECUTABLE variable on Windows that seems to disappear sometimes. Most notably on the clean windows 7 builds. Refs #3302

comment:12 Changed 9 years ago by Michael Reuter

In [14650]:

Removing file from MOC list to fix natty warning. This refs #3302.

comment:13 Changed 9 years ago by Martyn Gigg

In [14691]:

Fix a couple of warnings Refs #3302

comment:14 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 30 to Iteration 31

Bulk move of tickets to iteration 31 at the iteration 30 code freeze

comment:15 Changed 9 years ago by Martyn Gigg

In [14756]:

Add a couple of Qt Dlls to the windows installer that are required. Refs #3302.

comment:16 Changed 9 years ago by Russell Taylor

In [14766]:

Remove debug cout output. Re #3302.

comment:17 Changed 9 years ago by Russell Taylor

In [14792]:

Update category to be consistent, if equally meaningless. Re #3302.

comment:18 Changed 9 years ago by Russell Taylor

In [14793]:

Update algorithm categories for consistency. Re #3302.

comment:19 Changed 9 years ago by Janik Zikovsky

In [14800]:

Refs #3302: Minor script tweaks

comment:20 Changed 9 years ago by Russell Taylor

In [14805]:

Fix spelling error. Re #3302.

comment:21 Changed 9 years ago by Nick Draper

  • Owner changed from Anyone to Nick Draper
  • Status changed from assigned to accepted
  • Milestone changed from Iteration 31 to Iteration 30

comment:22 Changed 9 years ago by Nick Draper

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

comment:23 Changed 9 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

Pinching an easy one!

comment:24 Changed 9 years ago by Russell Taylor

  • Status changed from verifying to closed

The current state of play on the ORNL clean builds (which include the tests):

  • ornl_clean_rhel6_vates - 0 warnings
  • ornl_clean_lucid - 0 warnings
  • ornl_clean_snowleopard - 11 warnings - but these are all 'known' and will be addressed after the release
  • ornl_clean_windows7 - oddly, sometimes says 53 warnings, sometimes 0

comment:25 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4149

Note: See TracTickets for help on using tickets.