Array instantiation in WorkPlan.processContainedInAndPrepareExecution is misleading

Description

The code:

The first array instantiation is useless, since it is erased on the very next line.

Environment

None

Activity

Show:
Yoann Rodière
January 9, 2017, 5:08 PM

Actually, no, I was completely and utterly wrong. Let's say it's getting late...

Guillaume Smet
January 9, 2017, 8:03 PM

I think this code is worth a change as it's quite unclear.

Usually, the pattern is the following:

Having it in 2 lines is quite misleading and it would avoid creating another issue for the same problem in 2 months .

Yoann Rodière
January 10, 2017, 8:28 AM

Assignee

Yoann Rodière

Reporter

Yoann Rodière

Labels

None

Suitable for new contributors

None

Feedback Requested

None

Components

Fix versions

Affects versions

Priority

Trivial
Configure