On 22/11/12 17:08, Glenn Adams wrote:
> What do you mean by "moved to build/gensrc"? There is no build/gensrc
> directory in the FOP source tree.
> Are you suggesting it be regenerated at build time? If so, then that would
> slow the build process unnecessarily since it requires downloading data
> from the unicode web site. Further, there is no logical need to regenerate
> it at build time any more than there is a need to rebuild
> LineBreakUtils.java.

Well, then maybe we can move it to a new src/generated directory or
something like that? I think it would be safer to separate generated
classes from the ones that are manually edited.

> None of the rationale you provide below are compelling. So I object to this
> proposed change.
> On Thu, Nov 22, 2012 at 2:52 AM, Vincent Hennebert 
> <vhenneb...@gmail.com>wrote:
>> Hi,
>> I’ve recently stumbled upon this o.a.f.complexscripts.bidi.BidiClass
>> that appears to be generated. I think it should be moved to the
>> build/gensrc directory. Reasons are:
>> • It doesn’t need to be understood and maintained by developers in the
>>   same way as other classes. Changes will be made to the generator and
>>   not this class. Therefore it doesn’t need to follow the usual style
>>   and readability rules.
>> • Despite the warning at the beginning of the class, there is still
>>   a risk that it is accidentally modified during some refactoring
>>   session.
>> • There is also a risk that the generator is modified and this class not
>>   re-generated, introducing a discrepancy that may take some time to be
>>   tracked down.
>> Like any other generated file, I think the generation should happen at
>> any clean build.
>> Any objection?
>> Thanks,
>> Vincent

Reply via email to