As far as I know:

aStream
������ nextPutAll: ' with value: ���;
������ print: count.

Works with most streams, does not create intermediary streams, and is readable.

On Sun, 27 Dec 2020 at 05:59, St��phane Ducasse <stephane.ducasse@inria.fr> wrote:


On 26 Dec 2020, at 20:41, Sean P. DeNigris <sean@clipperadams.com> wrote:

Sven Van Caekenberghe-2 wrote
Maybe his question is (also) why the automatic refactoring did it wrong,
the rules warned about the wrong use of #printString, suggested a fix, but
the solution is still using #printString, hence the same problem.

I might be reading it wrong, but I thought everything worked as advertised,
and the limitation was that there was no option to automatically fix the
other warning about the printString. The "Use cascaded nextPutAll:���s instead
of #, in #nextPutAll:" transformation did change:

aStream nextPutAll: ' with value: ', count printString.

to:

aStream
��nextPutAll: ' with value: ';
��nextPutAll: count printString




Yes but with such approach you still create an extra intermediary��
stream.��

The solution should be��


aStream
�� ��nextPutAll: ' with value: ���.
�� ��count printOn: aStream

Now this is a bit more tricky to do.





--------------------------------------------
St��phane Ducasse
03 59 35 87 52
Assistant: Aurore Dalle��
FAX 03 59 57 78 50
TEL 03 59 35 86 16
S. Ducasse - Inria
40, avenue Halley,��
Parc Scientifique de la Haute Borne, B��t.A, Park Plaza
Villeneuve d'Ascq 59650
France