Ticket #11314 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Clean up line breaks in SNSLiveEventDataListener

Reported by: Ross Miller Owned by: Ross Miller
Priority: minor Milestone: Release 3.4
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Harry Jeffery

Description

Some time back, someone ran an automated tool to shorten the lines in the source code to 80 characters. That tool made some less-than-optimal decisions about where to break the lines. Clean up the most awkward line breaks manually.

No code or functionality changes.

Change History

comment:1 Changed 6 years ago by Ross Miller

  • Status changed from new to inprogress

Clean up line breaks

At some point, an automated tool was used to limit line lengths to 80 characters. This tool made some less-than-optimal choices about where to break the lines. This change cleans that up. No actual code or functionality changes.

Refs #11314

Changeset: b3939b16f41da356e3228054e071f7d8a9e374cc

comment:2 Changed 6 years ago by Ross Miller

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

This is being verified as pull request #376.

comment:3 Changed 6 years ago by anonymous

Can one of the admins verify this patch?

comment:4 Changed 6 years ago by Federico Montesino Pouzols

Jenkins, test this please

comment:5 Changed 6 years ago by Pete Peterson

Jenkins, Test this please

comment:6 Changed 6 years ago by Pete Peterson

Test this please

comment:7 Changed 6 years ago by Pete Peterson

test this please

comment:8 Changed 6 years ago by Federico Montesino Pouzols

Jenkins, retest this please

comment:9 Changed 6 years ago by Harry Jeffery

  • Status changed from verify to verifying
  • Tester set to Harry Jeffery

comment:10 Changed 6 years ago by Harry Jeffery

comment:11 Changed 6 years ago by Harry Jeffery

  • Status changed from verifying to closed

Merge pull request #376 from mantidproject/11314_clean_up_live_listener_comments

Clean up line breaks

Full changeset: 03dbf266d660c1d7a0548eacdfd3599c13651f79

comment:12 Changed 5 years ago by Nick Draper

Somehow these slipped through without a resolution. Set to Fixed.

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 12153

Note: See TracTickets for help on using tickets.