<div dir="ltr">Robert,<div><br></div><div>Only one :) And I will let them know the details. It's only used for caching, so preserving existing files is not critical.</div><div><br></div><div>In the meantime I preserved our "old" way of doing it for pre-3.5.4 version (before getBufferData was available), and copied your updated method for 3.5.5-3.5.6 versions. Hopefully that plugs the holes.</div><div><br></div><div><a href="https://github.com/gwaldron/osgearth/blob/master/src/osgEarth/TextureBufferSerializer.cpp">https://github.com/gwaldron/osgearth/blob/master/src/osgEarth/TextureBufferSerializer.cpp</a><br></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Glenn Waldron</div></div></div></div>
<br><div class="gmail_quote">On Mon, Mar 27, 2017 at 11:13 AM, Robert Osfield <span dir="ltr"><<a href="mailto:robert.osfield@gmail.com" target="_blank">robert.osfield@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Glenn,<br>
<br>
I have just had a look at osgEarth's implementation of the<br>
TextureBuffer serializer and it's different from the core OSG one, the<br>
later is a more complete implementation.<br>
<br>
For backwards compatibility of older .osgb/.osgx/.osgt files generated<br>
by osgEarth applications that leverage the osgEarth's TexutreBuffer<br>
serializer we'd have a potential conflict.  Perhaps one way to resolve<br>
this would be to version OSG's new TextureBuffer so that for older<br>
versions the osgEarth style implementation is used, while on recent<br>
versions the latest is used.<br>
<br>
How many files/osgEarth users do you think might be affected by this issue?<br>
<br>
Robert<br>
<span class="HOEnZb"><font color="#888888"><br>
Robert.<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
On 27 March 2017 at 15:16, Glenn Waldron <<a href="mailto:gwaldron@gmail.com">gwaldron@gmail.com</a>> wrote:<br>
> Robert,<br>
><br>
> Yes, it's in the latest stable release - version 2.8.<br>
><br>
> I will add the versioning to our master branch as well as the 2.8 branch.<br>
><br>
> Thanks!<br>
><br>
><br>
> Glenn Waldron<br>
><br>
> On Mon, Mar 27, 2017 at 9:47 AM, Robert Osfield <<a href="mailto:robert.osfield@gmail.com">robert.osfield@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Hi Glenn,<br>
>><br>
>> On 27 March 2017 at 14:22, Glenn Waldron <<a href="mailto:gwaldron@gmail.com">gwaldron@gmail.com</a>> wrote:<br>
>> > Agreed, we did this out of temporary necessity; the right thing moving<br>
>> > forward is for osgEarth to check the OSG version when<br>
>> > creating/registering<br>
>> > the osg::TextureBuffer serializer.<br>
>><br>
>> Testing for an OSG version of 3.5.6 or later would probably be<br>
>> appropriate.<br>
>><br>
>> Are the osgEarth TextureBuffer serilializer part of a stable release<br>
>> of osgEarth?<br>
>><br>
>> Robert.<br>
>> ______________________________<wbr>_________________<br>
>> osg-users mailing list<br>
>> <a href="mailto:osg-users@lists.openscenegraph.org">osg-users@lists.<wbr>openscenegraph.org</a><br>
>> <a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.<wbr>org/listinfo.cgi/osg-users-<wbr>openscenegraph.org</a><br>
><br>
><br>
><br>
> ______________________________<wbr>_________________<br>
> osg-users mailing list<br>
> <a href="mailto:osg-users@lists.openscenegraph.org">osg-users@lists.<wbr>openscenegraph.org</a><br>
> <a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.<wbr>org/listinfo.cgi/osg-users-<wbr>openscenegraph.org</a><br>
><br>
______________________________<wbr>_________________<br>
osg-users mailing list<br>
<a href="mailto:osg-users@lists.openscenegraph.org">osg-users@lists.<wbr>openscenegraph.org</a><br>
<a href="http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org" rel="noreferrer" target="_blank">http://lists.openscenegraph.<wbr>org/listinfo.cgi/osg-users-<wbr>openscenegraph.org</a><br>
</div></div></blockquote></div><br></div>