See also my proposal in [1] to make a preference (or even a per-package rule) for displaying a pop-up for each shadowed variable. There is already ShadowedVariableNotification which could be caught in the parser to display a dialog window.

Best,
Christoph

---
Sent from Squeak Inbox Talk

On 2022-12-26T17:17:17-06:00, asqueaker at gmail.com wrote:

> >
> > Right now when a variable name is shadowed we only get a line appearing in
> > the Transcript, which is alarmingly easy to miss. Given how many other
> > alerts we get popping up for undeclared or unused variables, maybe we need
> > to add one more for shadowing.
> >
> 
> Shadowed variables are not an issue, technically, just from the
> code-clarity sense, which syntax-highlighting has now largely resolved.  I
> think such fixes like these should be handled by the existing batch process
> in the development cycle where one cleans the Lint out of one's code.  If
> you must add yet another pop up for this, make sure loading a large package
> with a lot of shadowed variables doesn't produce hundreds of such pop ups.
> Which, of course, I guess you'd be back to a bunch of code with shadowed
> variables with no way to identify them except the Transcript notifications
> again.  So, maybe that's another reason to relegate this to the deLint'ing
> stage.
> 
> Best,
>   Chris
> 
> 
> >
> >
> >
> -------------- next part --------------
> An HTML attachment was scrubbed...
> URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20221226/59b8ca45/attachment.html>
> 
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20221227/be39f75e/attachment.html>