On 07/26/2016 08:54 PM, Scott Kostyshak wrote: > The attached patch constifies a function parameter. My question is > whether this patch causes more pain to other developers than it does > good to the code.
So the change is: -bool BufferView::scrollToCursor(DocIterator const & dit, bool recenter) +bool BufferView::scrollToCursor(DocIterator const & dit, bool const recenter) What is the significance of this change? Since recenter is passed by value, why would the caller care if it is const? Is this just a signal to the compiler? Richard
