Hi,
Looks like Max is busy with more urgent things :-)
As this patch will affect my future work on the layout engine, I’d like
to take over the patch review.
Your suggestion to use Skype sounds good. That will ease the job a bit.
I’ll contact you off-line to exchange details and arrange a time.
Hi *,
Vincent Hennebert schrieb:
Hi,
Looks like Max is busy with more urgent things :-)
Yes, work keeps me occupied most of the time. I was actually just
looking at the patch again, and decided that I am unable to apply it,
because I do cannot verify if the renames are correct, as it affects
Hi Max,
One more thing I noticed: Alex, you are introducing deprecated methods
and then using them. Please either
- don't mark the methods as deprecated, if they are valid helper methods or
- don't use the deprecated methods, but rather use whatever you inted to
replace it with.
I marked
Hi Alex,
Hi *,
if you do not yet have FOP developer access, and you are working on a
larger set of problems, please do not submit one large patch - current
committers will not have the time to go through every single change.
Instead, it is much nicer to have a series of small patches.
One option
Hi Max,
you are right. It's always better to have small patches focused on one
thing. I don't get my MinOptMax patch focused only on the refactoring of
making MinOptMax immutable.
In the last half-an-hour I walked myself through all the diffs,
file-by-file. I must say - except from
