Re: Patch to remove redundant code

[Owen Taylor <otaylor redhat com>]

"Matthias Clasen" <matthiasc poet de> writes:

> Speaking about redundant code, what is the purpose of functions like
 
"Hmmm, I think we might need to do something if the adjustment's bounds/step
change. Not sure what. Let me put in a prototype." (Or so I'd guess; 
that code has been there forever.)

[ The only legitimate reason for the bounds of a viewport to change
is if the viewport changes them itself, so nothing should be needed. ]
 
Regards,
                                        Owen

> static void
> gtk_viewport_adjustment_changed (GtkAdjustment *adjustment,
>      gpointer       data)
> {
>   GtkViewport *viewport;
> 
>   g_return_if_fail (adjustment != NULL);
>   g_return_if_fail (GTK_IS_VIEWPORT (data));
> 
>   viewport = GTK_VIEWPORT (data);
> }