Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Don't initialize CoordinateSequence in constructor #765

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Don't initialize CoordinateSequence in constructor #765

wants to merge 1 commit into from

Commits on Dec 8, 2022

  1. Don't initialize CoordinateSequence in constructor 

    This ought to improve performance in the common case where
initialization is unnecessary.

However, the `getDimension` method, in the case where the dimension is
unknown (0) looks at the z dimension of the first element in the array
to determine if there is z.  So we need to initialize at least the
first element.
    @eyal0 @esoha-nvidia
    eyal0 authored and esoha-nvidia committed Dec 8, 2022
    Configuration menu
    Copy the full SHA
    23401e1 View commit details
    Browse the repository at this point in the history