[Libreoffice] [PATCH] [PUSHED] Dead code removal
kohei.yoshida at suse.com
Wed Jan 18 17:07:27 PST 2012
On Sun, 2012-01-15 at 21:52 -0500, Marc-André Laverdière wrote:
> Hello list,
> Since I went a bit beyond unusedcode.easy, I prefer to ask review on this one :)
I assume you are talking about the change in SVGExport? It all looks
reasonable to me. If the push method is not used, then that implies
that the clip container is always empty. With that, you can remove a
little more code like you did in your patch.
BTW, we ask all patch submitters to put [PATCH] in the subject line, or
prospective patch reviewers may overlook your patch.
Also, I didn't apply your changes in the datetime files in tools. Looks
like someone else has got to it first.
Kohei Yoshida, LibreOffice hacker, Calc
More information about the LibreOffice