Changes between Initial Version and Version 1 of Ticket #4127, comment 4


Ignore:
Timestamp:
Aug 9, 2018, 1:58:59 PM (6 years ago)
Author:
esparano

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #4127, comment 4

    initial v1  
    1 I have to strongly disagree with changing the docs versus changing the behavior of the code to match the docs. The dimension of a Point is 0 because it is, mathematically, a 0-dimensional object. However, the dimension of an empty geometry is not actually 0. It's undefined. It makes more sense to return -1, like the function returns for NULL.
     1I think there may have been some misunderstanding and I'd like to get an idea of what the actual intended behavior of the code is. The dimension of a Point is 0 because it is, mathematically, a 0-dimensional object. The docs also say that the dimensionality of an array of geometries is the highest dimensionality of any of the subgeometries. So are you saying that, if there aren't any subgeometries in the first place, its dimensionality is actually 0? I was thinking it makes more sense to return -1, like the function returns for NULL.