On Tue, Oct 13, 2015 at 8:45 AM, Jean-Marc Lasgouttes <lasgout...@lyx.org> wrote: > Le 13/10/15 02:08, Scott Kostyshak a écrit : >> >> On Mon, Oct 12, 2015 at 11:49:42AM +0200, Jean-Marc Lasgouttes wrote: >>> >>> Le 12/10/2015 11:16, Stephan Witt a écrit : >>>> >>>> Am 12.10.2015 um 11:04 schrieb Jean-Marc Lasgouttes >>>> <lasgout...@lyx.org>: >>>> >>>>> Le 10/10/2015 21:31, Stephan Witt a écrit : >>>>>> >>>>>> Another question with "dead store" code is in RowPainter::paintFirst: >>>>>> >>>>>> The y_top assignments are not used. >>>>>> JMarc, do you know if the code block simply can be removed? >>>>> >>>>> >>>>> The code was already like that in 2.1. >>>> >>>> >>>> Yes, it dates back to at least 2007. Earlier I couldn't "git blame" >>>> anymore. >>> >>> >>> I found the same. Who has a good tutorial about git archeology? >> >> >> How about the following? >> >> git log -Sy_top > > > I did that, but the problem was with renames. The best I could come up with > is >
I don't see the problem. For me it goes back to the "Initial revision" by LGB in 1999. Anyway, the _usage_ of y_top was removed at: 7de76711 (http://git.lyx.org/?p=lyx.git;a=commitdiff;h=7de76711#patch35) 8c8e438e4 (http://git.lyx.org/?p=lyx.git;a=commitdiff;h=8c8e438e4cb0d2209cd9fc1990041c17028f3ff8;hp=318bdb52f3ecfd2c2f1ef5bd9f7dcfa4a5ad8edd). Vincent