<div dir="ltr">OK Robert, will do. Thanks for the insight -gw</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 Tue, Mar 28, 2017 at 7:01 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 don't think the BufferData will be serialized correctly prior to<br>
3.5.6 so the second block in your revised code won't help,  I would<br>
suggest just leaving your original TextureBuffer serializer<br>
implementation in place for versions prior to 3.5.6.<br>
<br>
Cheers,<br>
Robert.<br>
<div class="HOEnZb"><div class="h5"><br>
On 27 March 2017 at 17:28, Glenn Waldron <<a href="mailto:gwaldron@gmail.com">gwaldron@gmail.com</a>> wrote:<br>
> Robert,<br>
><br>
> Only one :) And I will let them know the details. It's only used for<br>
> caching, so preserving existing files is not critical.<br>
><br>
> In the meantime I preserved our "old" way of doing it for pre-3.5.4 version<br>
> (before getBufferData was available), and copied your updated method for<br>
> 3.5.5-3.5.6 versions. Hopefully that plugs the holes.<br>
><br>
> <a href="https://github.com/gwaldron/osgearth/blob/master/src/osgEarth/TextureBufferSerializer.cpp" rel="noreferrer" target="_blank">https://github.com/gwaldron/<wbr>osgearth/blob/master/src/<wbr>osgEarth/<wbr>TextureBufferSerializer.cpp</a><br>
><br>
><br>
> Glenn Waldron<br>
><br>
> On Mon, Mar 27, 2017 at 11:13 AM, Robert Osfield <<a href="mailto:robert.osfield@gmail.com">robert.osfield@gmail.com</a>><br>
> wrote:<br>
>><br>
>> 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<br>
>> issue?<br>
>><br>
>> Robert<br>
>><br>
>> Robert.<br>
>><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<br>
>> > branch.<br>
>> ><br>
>> > Thanks!<br>
>> ><br>
>> ><br>
>> > Glenn Waldron<br>
>> ><br>
>> > On Mon, Mar 27, 2017 at 9:47 AM, Robert Osfield<br>
>> > <<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<br>
>> >> > 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>
>> >><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>
>> ><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>
><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>