ObjReader Community

Discussion => Post Your Questions & Comments Here => Topic started by: Michael Lobko-Lobanovsky on October 16, 2018, 11:38:02 pm

Title: Minor Quirks
Post by: Michael Lobko-Lobanovsky on October 16, 2018, 11:38:02 pm
Patrice,

There are three quirks in the current ObjReader that keep annoying me:TIA
Title: Re: Minor Quirks
Post by: Patrice Terrier on October 17, 2018, 10:28:15 am
Mike

While we are at it, VS2017 recommends to turn #define macro into constexpr, your thought?

Quote
2.When the Mipmap textures menu is clicked, there's some apparent activity going on while the spinner spins. But it doesn't result in the current textures being changed for unmipmapped ones, and vice versa. The new setting affects only the new model loaded into the viewer later. Can you ensure the menu first reloads the model's current textures?
Yes, we first delete the existing texture set, then we reload it, in the gl_LoadTexture.
I don't see any error there  :-[

I have attached the patch with my changes.

Title: Re: Minor Quirks
Post by: Michael Lobko-Lobanovsky on October 17, 2018, 02:02:41 pm
Patrice,

VS2017 recommends to turn #define macro into constexpr, your thought?

My dev platform is a commercial VS 2013 Ultimate, and I'm sorry to say but it doesn't support constexpr which appeared in VS 2015. I wouldn't want to change it for a later platform yet, mainly because of the necessity to renovate the free licenses of newer Community versions every 30 days or so.

Quote
... then we reload it, in the gl_LoadTexture.
I don't see any error there  :-[

We shouldn't just reload them as they were. If they were mipmapped, we should reload them un-mipmapped and vice versa. Alas, this isn't so. As far as my eyes can see, they are reloaded unaltered exactly as they were -- mipmapped or not. OTOH ObjReader preserves its Mipmap textures setting between model reloads, so that the next model loaded in the same session will have its textures (un-)mipmapped according to the new setting. Correct me if I'm wrong but I think I am not. ???

Thanks for the patch; my comment will follow after the merger.
Title: Re: Minor Quirks
Post by: Michael Lobko-Lobanovsky on October 17, 2018, 02:35:50 pm
OK,

Here are my observations:
Title: Re: Minor Quirks
Post by: Patrice Terrier on October 17, 2018, 02:51:58 pm
Quote
because of the necessity to renovate the free licenses of newer Community versions every 30 days or so
That doesn't occure by me, i never had to renovate my free license for the Community version.
Title: Re: Minor Quirks
Post by: Michael Lobko-Lobanovsky on October 18, 2018, 02:06:02 am
Patrice,

I fixed the texture un-/re-mipmapping issue, and I made the geo data/mesh/material displays get cleared before loading a new model.

I'll send you the sources for sync'ing a little later when I finish adding a few other minor fixes and additions.
Title: Re: Minor Quirks
Post by: Patrice Terrier on October 18, 2018, 08:56:48 am
Ok, i have been kept very busy on the 3wheeler, i hope to be able to post a first WIP today.