Bug #4816
closedMisspelt property name: XsltCompiler.FastCompliation (.NET)
100%
Description
According to the documentation, the property name FastCompilation
has been misspelt as FastCompliation
.
Updated by Michael Kay over 3 years ago
This arises from https://stackoverflow.com/questions/64609315/how-to-get-warnings-from-schema-aware-xslt-using-saxon-api/64611632?noredirect=1#comment114245448_64611632
Actually, looking at the Java code, I'm not sure that setting FastCompilation to false achieves much. It sets the optimizer options to be what they are at Configuration level, which I would think has JIT=true.
Updated by O'Neil Delpratt over 3 years ago
- Status changed from New to Resolved
- Applies to branch 10, trunk added
- Fix Committed on Branch 10, trunk added
Fixed typo on both Saxon 10 and trunk branches.
Updated by Michael Kay over 3 years ago
- Status changed from Resolved to In Progress
Let's add some tests to see if the property actually has any effect...
I'll do this on the Java side.
Also, is it possible to retain the misspelt name as a deprecated synonym in 10.x (dropping it on trunk)?
Updated by O'Neil Delpratt over 3 years ago
Michael Kay wrote:
Let's add some tests to see if the property actually has any effect...
I'll do this on the Java side.
I will port junit tests to nunit when ready.
Also, is it possible to retain the misspelt name as a deprecated synonym in 10.x (dropping it on trunk)?
Added back the property, with obsolete
status added.
Updated by Michael Kay over 3 years ago
Two other observations:
-
XsltCompiler
on .NET lacks the methodXsltCompiler.setJustInTimeCompilation()
-
XsltCompiler
on .NET has a methodGetUnderlyingCompilerInfo()
, perhaps it should also haveGetUnderlyingXsltCompiler()
- and perhaps this applies to all the C# classes that are layered on top of Java s9api classes.
Updated by O'Neil Delpratt over 3 years ago
Michael Kay wrote:
Two other observations:
XsltCompiler
on .NET lacks the methodXsltCompiler.setJustInTimeCompilation()
Method added.
XsltCompiler
on .NET has a methodGetUnderlyingCompilerInfo()
, perhaps it should also haveGetUnderlyingXsltCompiler()
- and perhaps this applies to all the C# classes that are layered on top of Java s9api classes.
I have added the underlying implementation object where required. Some class have the underlying object already as the property Implementation
.
Updated by Michael Kay about 3 years ago
- Status changed from In Progress to Resolved
This appears to have been resolved.
Updated by O'Neil Delpratt about 3 years ago
- Status changed from Resolved to Closed
- % Done changed from 0 to 100
- Fixed in Maintenance Release 10.5 added
Bug fix applied to Saxon 10.5 maintenance release.
Please register to edit this issue