Hi, When you assign ids to the new fields do take care to assign small ids. Every id < 32 can be encoded in one byte by protobuf.
Regards, Christian Vetter On Fri, May 6, 2011 at 1:45 PM, Oliver Tonnhofer <[email protected]> wrote: > > On 06.05.2011, at 13:32, Jochen Topf wrote: > >> On Fri, May 06, 2011 at 12:16:48PM +0200, Christian Vetter wrote: >>> On Fri, May 6, 2011 at 10:25 AM, Oliver Tonnhofer <[email protected]> wrote: >>>> Isn't `repeated` like `optional`, since it can have zero occurrences? >>>> I haven't tested that, though. >>> >>> Exactly. You should be able to add repeated fields. Just be sure not >>> to reuse any IDs. >> >> Ok, that makes sense. That leaves the problem with the wrong default. You >> can't set a default on a repeated field (I tried and the protobuf compiler >> complained). But the default for the visible flag should be "true" to be >> backwards compatible. > > > The repeated list is empty by default, which a client can interpret as "all > nodes visible". > > Regards, > Oliver > > -- > Oliver Tonnhofer | Omniscale GmbH & Co KG | http://omniscale.de > http://mapproxy.org | https://bitbucket.org/olt | @oltonn > > > > > _______________________________________________ dev mailing list [email protected] http://lists.openstreetmap.org/listinfo/dev
