Cool.  If anyone familiar with the planet dumper tool is listening...

In http://svn.openstreetmap.org/applications/utils/planet.osm/C/output_osm.c

} else if ((*in >= 0) && (*in < 32)) {
            escape_tmp[len] = '?';
            len++;

should be something like

} else if ((*in > 0) && (*in < 32)) {
            len+=sprintf(&escape_tmp[len], "&#%d;", *in);

"Something like" as in I haven't even checked if that compiles :).

Of course, another thing to consider is that 1024 bytes isn't enough for the
truly pathological cases.  I think you need like 1531 or something to handle
that.  Fixing this might be enough to properly process the current db,
though.

Any chance of adding num_changes?

On Mon, Dec 21, 2009 at 12:54 AM, Brett Henderson <br...@bretth.com> wrote:

> Hi Anthony,
>
> No, that's not me.  I only look after the osmosis changesets.  Hmm, the
> word changeset has become overloaded ... by osmosis changesets I mean the
> diff/delta files.
>
> The planet and changesets-xx.osm.bz2 files are created using the planet
> dumper tool.  I believe Jon and Grant look after those.
>
> Brett
>
>
> On Mon, Dec 21, 2009 at 4:09 PM, Anthony <o...@inbox.org> wrote:
>
>> Hi Brett,
>>
>> Do you also maintain the changesets files (e.g.
>> changesets-091216.osm.bz2)?  Now that the full history dump is out I can
>> confirm that there are a relatively large number of corrupted key/values in
>> that file.  For example, see
>> http://www.openstreetmap.org/browse/changeset/2498325 and
>> http://www.openstreetmap.org/browse/changeset/966232 .  The dumper is
>> converting control characters into question marks.
>>
>> Anthony
>>
>
>
_______________________________________________
dev mailing list
dev@openstreetmap.org
http://lists.openstreetmap.org/listinfo/dev

Reply via email to