Attention is currently required from: laforge, pespin.
1 comment:
Patchset:
Ok, if others are fine with it let's merge it, though I really think this paradigm of having to call […]
codec_filter_run() does not log.
I believe the strong opinion is a gut feel that the inputs do not change often, that "filter_run()" sounds expensive, and that the results are not used often. All of these are not accurate impressions. Inputs update more often than results are being used, and codec_filter_run() is not expensive.
The amount of us discussing when to call run() far exceeds the significance of this aspect. You've asked about this over an extended period of time and in many different patches. I'm trying to convince you that it is not an important issue, but it seems my responses don't reach home.
Important at this point is that it has correct results. There are a number of quite non-trivial aspects of this patch series, IMHO this part is just a bike shed slash premature optimization, rather don't let review get distracted by this marginal aspect.
To view, visit change 31698. To unsubscribe, or for help writing mail filters, visit settings.