OpenDoPEHandler is not thread-safe

[tinne]

One cannot run OpenDoPEHandler twice in the same jvm at the same time, as the static field shallowTraversor might be accessed in the wrong way by two parallel runs of preprocess.

The easiest way to cure this might be to make everything non-static. The most portable way might be to add some static helper methods to aid porting.

As an instant cure, I suggest adding a warning and/or synchronizing preprocess.

[olap]

Hi!

It looks like current implementation is not thread-safe
Is there any chance to fix this in the nearest future ?

Thanks,
Oleg

[jason]

I had no immediate plans to work on it.

That said, I don't think its a big task. I'll have a look at in tomorrow or more likely Friday. Alternatively, if you'd like to make everything non-static yourself, I'd be happy to accept the patch.

[jason]

See http://www.docx4java.org/trac/docx4j/changeset/1639