0007770: cascading does not work with super

(0014363 - 197 - 257 - 257 - 257 - 257 - 257)
FrankShearar
06-19-13 09:54

What do you expect this to mean? A cascade says "send this sequence of messages to the same object", so do you expect this to be equivalent to

super initialize.
self setListProperties.

?

(0014567 - 238 - 354 - 354 - 354 - 354 - 354)
Patrickr
11-18-15 13:57
edited on: 11-20-15 09:56

For consistency reasons, I'd actually expect this to be equivalent to:

super initialize.
super setListProperties.

corresponding to:

self
initialize;
setListProperties.

and:

self initialize.
self setListProperties.

(0014577 - 241 - 253 - 253 - 253 - 253 - 253)
marceltaeumel
11-20-15 11:10

I would rather not support cascading for super sends because this might easily be overlooked when debugging. It is not only about sending messages to the same object but also modifying message lookup again and again.

Such code smells. :-/

(0014578 - 30 - 30 - 30 - 30 - 30 - 30)
krono
11-20-15 11:13

+1 Expected behavior, I'd say.

Issue History
Date Modified	Username	Field	Change
06-19-13 09:49	vaidasd	New Issue
06-19-13 09:54	FrankShearar	Note Added: 0014363
12-03-13 09:49	FrankShearar	Summary	cascading does ont work with super => cascading does not work with super
11-18-15 13:57	Patrickr	Note Added: 0014567
11-20-15 09:56	Patrickr	Note Edited: 0014567
11-20-15 11:10	marceltaeumel	Note Added: 0014577
11-20-15 11:13	krono	Note Added: 0014578

Notes
(0014363 - 197 - 257 - 257 - 257 - 257 - 257) FrankShearar 06-19-13 09:54	What do you expect this to mean? A cascade says "send this sequence of messages to the same object", so do you expect this to be equivalent to super initialize. self setListProperties. ?

(0014567 - 238 - 354 - 354 - 354 - 354 - 354) Patrickr 11-18-15 13:57 edited on: 11-20-15 09:56	For consistency reasons, I'd actually expect this to be equivalent to: super initialize. super setListProperties. corresponding to: self initialize; setListProperties. and: self initialize. self setListProperties.

(0014577 - 241 - 253 - 253 - 253 - 253 - 253) marceltaeumel 11-20-15 11:10	I would rather not support cascading for super sends because this might easily be overlooked when debugging. It is not only about sending messages to the same object but also modifying message lookup again and again. Such code smells. :-/

(0014578 - 30 - 30 - 30 - 30 - 30 - 30) krono 11-20-15 11:13	+1 Expected behavior, I'd say.

Relationships