<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

  <meta content="text/html;charset=ISO-8859-1" http-equiv="Content-Type">

</head>

<body bgcolor="#ffffff" text="#000000">

<br>

<blockquote

 cite="mid:e8e7199c0810180412v13cd1563j7e52666c2190845f@mail.gmail.com"

 type="cite">

  <blockquote type="cite">

    <pre wrap="">Regarding c) they should all be returning enumerated types but I guess

the side effect is that the QgsMapLayer::type() enumerators and those

of teh first two entries in QgsVectorLayer::type() return  matchin

numbers. I could force them to be higher no's to remove such

inadvertant collisions. I'll take a look at this later tomorrow -

thanks for the feedback on this.

    </pre>

  </blockquote>

  <pre wrap=""><!---->

AFAIK python doesn't support enumerations, you get an integer instead.

SIP generates objects for enumerations, so you can use e.g.

QgsMapLayer.VectorLayer instead of numbers.

  </pre>

</blockquote>

Yes this is what I tend to do...<br>

<blockquote

 cite="mid:e8e7199c0810180412v13cd1563j7e52666c2190845f@mail.gmail.com"

 type="cite">

  <pre wrap="">

Tim, please rename vector layer's type() either back to vectorType()

or some other name to avoid override.

  </pre>

</blockquote>

geometryType() would probably be good, and is consistent with

QgsGeometry.<br>

<blockquote

 cite="mid:e8e7199c0810180412v13cd1563j7e52666c2190845f@mail.gmail.com"

 type="cite">

  <pre wrap="">

  </pre>

  <blockquote type="cite">

    <pre wrap="">Any other comments on API changes as I do them are welcome...

    </pre>

  </blockquote>

  <pre wrap=""><!---->

I was not happy with all the renaming taking place, but generally

they're fine. One thing I don't really like is that

QgsVectorDataProvider::reset() was renamed to begin(). The real

purpose of this function is to restart reading of the data, one

doesn't have to call it when starting retrieval of features - but

that's what the current name suggest. I would suggest using e.g.

resetReading() as it better corresponds with the action it takes (and

stays consistent with OGR).

  </pre>

</blockquote>

+1<br>

I agree that begin is somewhat misleading...<br>

<br>

Carson

</body>

</html>