Ticket #10507 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Cleanup clang warnings

Reported by: Steven Hahn Owned by: Steven Hahn
Priority: major Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Pete Peterson

Description

Start eliminating as many warnings as I can from AppleClang on OSX.

Change History

comment:1 Changed 6 years ago by Steven Hahn

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Steven Hahn

  • Status changed from assigned to inprogress

Refs #10507. Cleanup clang warning in Mantid Algorithms.

Changeset: ac06f79d61ad336ad2c1588b0f3cc4f3efb4ffd7

comment:3 Changed 6 years ago by Steven Hahn

Refs #10507. fixing more clang warnings

Changeset: e82f714aff0f444eac278dd0f8b16db3832936cf

comment:4 Changed 6 years ago by Steven Hahn

Refs #10507. fix indent

Changeset: dc849f1008816d5d5a92e75a01cd1b714b3c7055

comment:5 Changed 6 years ago by Steven Hahn

Refs #10507. changed single line for loops so clang won't warn

Changeset: 69623d8ee2d6cb6a9552ac58f9c1bf1e7f54675b

comment:6 Changed 6 years ago by Steven Hahn

Refs #10507. cleanup formatting

Changeset: 5eff53d934ba779dd5cde04705abaf224f1cc3c6

comment:7 Changed 6 years ago by Steven Hahn

Refs #10507. Merge branch into develop

Conflicts:

Code/Mantid/MantidQt/SpectrumViewer/src/EModeHandler.cpp

Changeset: a31b4ebda2d607c73d9191809697c822c5ac5a99

comment:8 Changed 6 years ago by Steven Hahn

Refs #10507. fix doxygen warning

Changeset: 82ddece71dc5663ec171d79cbd759ab11422250d

comment:9 Changed 6 years ago by Steven Hahn

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

This is being verified as pull request #63.

comment:10 Changed 6 years ago by Steven Hahn

retest this please

comment:11 Changed 6 years ago by Steven Hahn

Refs #10507. all warnings have been fixed or suppressed

Changeset: 03ecab470abb7f6b4730554a8bb4a18a91cd9b94

comment:12 Changed 6 years ago by Steven Hahn

targets "all" and "AllTests" should now build cleanly.

comment:13 Changed 6 years ago by Steven Hahn

Refs #10507. fix warnings on build servers

Changeset: d653e11670a551b69cc18184519c4f1cd486bb41

comment:14 Changed 6 years ago by Steven Hahn

Refs #10507. Fix merge conflict with master

Changeset: e7c709186e7644518bf39965a64c5d0512ade755

comment:15 Changed 6 years ago by Steven Hahn

Refs #10507. cleaner way to isolate clang

Changeset: 35b93996c91348de2bfdae9ced84e56245615119

comment:16 Changed 6 years ago by Steven Hahn

Revert "Refs #10507. cleaner way to isolate clang"

This reverts commit 35b93996c91348de2bfdae9ced84e56245615119.

Changeset: 2c5eaf4c562058be0feb023b7578bd0f2f68a9c3

comment:17 Changed 6 years ago by Steven Hahn

To fix a merging issue, I created a new branch and am cherry-picking commits to this branch. Therefore I'm closing this pull request without merging and creating a new pull request from the feature/10507_fix_clang_warnings_alt branch.

comment:18 Changed 6 years ago by Steven Hahn

Refs #10507. Cleanup clang warning in Mantid Algorithms.

Changeset: 3972c44e7bc2ed74acbdeb487eeb59bb3c6b3e56

comment:19 Changed 6 years ago by Steven Hahn

Refs #10507. fixing more clang warnings

Conflicts:

Code/Mantid/MantidQt/SpectrumViewer/src/EModeHandler.cpp

Changeset: 90b95afe10180268ae28fa0919e6edc5ce20cddf

comment:20 Changed 6 years ago by Steven Hahn

Refs #10507. fix indent

Conflicts:

Code/Mantid/MantidQt/SpectrumViewer/src/EModeHandler.cpp

Changeset: 421f46a46e7ae65551387f473016991e372503c8

comment:21 Changed 6 years ago by Steven Hahn

Refs #10507. changed single line for loops so clang won't warn

Changeset: 777334308b1c17242aa345504e7a7b8734f00426

comment:22 Changed 6 years ago by Steven Hahn

Refs #10507. cleanup formatting

Changeset: ab0b02863c8c5af52358be1b942c0fe6720a084b

comment:23 Changed 6 years ago by Steven Hahn

Refs #10507. fix doxygen warning

Changeset: 0a1ef20e58ec86d07a62fde2c2acaea682ffb28e

comment:24 Changed 6 years ago by Steven Hahn

Refs #10507. all warnings have been fixed or suppressed

Changeset: a5bf87a270b918cc9fd00366fa8d0ff6be77695a

comment:25 Changed 6 years ago by Steven Hahn

Refs #10507. alt branch now builds with zero warnings

Changeset: 780486490fd67504d05d89eaa27c6e8e5ea006c7

comment:26 Changed 6 years ago by Steven Hahn

This is being verified as pull request #65.

comment:27 Changed 6 years ago by Steven Hahn

Refs #10507. Merge 'feature/10507_fix_clang_warnings_alt' into develop

Conflicts:

Code/Mantid/Framework/API/test/WorkspaceFactoryTest.h Code/Mantid/Framework/Geometry/test/GroupTest.h Code/Mantid/Framework/ISISLiveData/inc/MantidISISLiveData/TCPEventStreamDefs.h Code/Mantid/Framework/LiveData/inc/MantidLiveData/ISIS/TCPEventStreamDefs.h

Changeset: c1ff229b35700d9d69e78a61d775a29af765585c

comment:28 Changed 6 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:29 Changed 6 years ago by Steven Hahn

The branch feature/10507_fix_clang_warnings has been deleted. Changes were merged into develop earlier this week.

comment:30 Changed 6 years ago by Pete Peterson

  • Tester changed from Nick Draper to Pete Peterson

comment:31 Changed 6 years ago by Pete Peterson

  • Status changed from verifying to closed

Merge pull request #65 from mantidproject/feature/10507_fix_clang_warnings_alt

Feature/10507 fix clang warnings

Full changeset: 53b6221671064ec321ed4d0021e38b8af605c442

comment:32 Changed 6 years ago by Steven Hahn

Refs #10507. Cleanup clang warning in Mantid Algorithms.

Changeset: ac06f79d61ad336ad2c1588b0f3cc4f3efb4ffd7

comment:33 Changed 6 years ago by Steven Hahn

Refs #10507. fixing more clang warnings

Changeset: e82f714aff0f444eac278dd0f8b16db3832936cf

comment:34 Changed 6 years ago by Steven Hahn

Refs #10507. fix indent

Changeset: dc849f1008816d5d5a92e75a01cd1b714b3c7055

comment:35 Changed 6 years ago by Steven Hahn

Refs #10507. changed single line for loops so clang won't warn

Changeset: 69623d8ee2d6cb6a9552ac58f9c1bf1e7f54675b

comment:36 Changed 6 years ago by Steven Hahn

Refs #10507. cleanup formatting

Changeset: 5eff53d934ba779dd5cde04705abaf224f1cc3c6

comment:37 Changed 6 years ago by Steven Hahn

Refs #10507. Merge branch into develop

Conflicts:

Code/Mantid/MantidQt/SpectrumViewer/src/EModeHandler.cpp

Changeset: a31b4ebda2d607c73d9191809697c822c5ac5a99

comment:38 Changed 6 years ago by Steven Hahn

Refs #10507. fix doxygen warning

Changeset: 82ddece71dc5663ec171d79cbd759ab11422250d

comment:39 Changed 6 years ago by Steven Hahn

Refs #10507. all warnings have been fixed or suppressed

Changeset: 03ecab470abb7f6b4730554a8bb4a18a91cd9b94

comment:40 Changed 6 years ago by Steven Hahn

Refs #10507. fix warnings on build servers

Changeset: d653e11670a551b69cc18184519c4f1cd486bb41

comment:41 Changed 6 years ago by Steven Hahn

Refs #10507. Fix merge conflict with master

Changeset: e7c709186e7644518bf39965a64c5d0512ade755

comment:42 Changed 6 years ago by Steven Hahn

Refs #10507. cleaner way to isolate clang

Changeset: 35b93996c91348de2bfdae9ced84e56245615119

comment:43 Changed 6 years ago by Steven Hahn

Revert "Refs #10507. cleaner way to isolate clang"

This reverts commit 35b93996c91348de2bfdae9ced84e56245615119.

Changeset: 2c5eaf4c562058be0feb023b7578bd0f2f68a9c3

comment:44 Changed 6 years ago by Steven Hahn

Refs #10507. Merge 'feature/10507_fix_clang_warnings_alt' into develop

Conflicts:

Code/Mantid/Framework/API/test/WorkspaceFactoryTest.h Code/Mantid/Framework/Geometry/test/GroupTest.h Code/Mantid/Framework/ISISLiveData/inc/MantidISISLiveData/TCPEventStreamDefs.h Code/Mantid/Framework/LiveData/inc/MantidLiveData/ISIS/TCPEventStreamDefs.h

Changeset: c1ff229b35700d9d69e78a61d775a29af765585c

comment:45 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11349

Note: See TracTickets for help on using tickets.