Review Request 116024: Port IconItem to native QSGTexture

[David Edmundson]

> On Feb. 27, 2014, 8:17 p.m., David Edmundson wrote:
> > src/declarativeimports/core/fadingnode.cpp, line 85
> > <https://git.reviewboard.kde.org/r/116024/diff/3/?file=246683#file246683line85>
> >
> >     Despite my code being clearly right this doesn't work. (according to apitrace and then qDebug)
> >     
> >     oldState is always 0
> >     
> >     This means we are binding textures every frame, which is what we wanted to avoid.
> >     
> >     I'm quite confused as to why.

I understand it now (I think)
If I have 2 icons, I will have 2 materials, but still only 1 shader.

oldState is used so that when SG batches operations I don't have to update all my uniforms if properties are the same between different materials being rendered in different nodes. It is not the previous state of the current material being updated.

Sorry for the noise.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116024/#review51129
-----------------------------------------------------------


On Feb. 27, 2014, 1:44 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116024/
> -----------------------------------------------------------
> 
> (Updated Feb. 27, 2014, 1:44 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> Port IconItem to native QSGTexture including the animation.
> This will save constantly uploading a new texture to OpenGL throughout the animation.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 4bc47bf 
>   src/declarativeimports/core/CMakeLists.txt 97e9283 
>   src/declarativeimports/core/fadingnode.cpp PRE-CREATION 
>   src/declarativeimports/core/fadingnode_p.h PRE-CREATION 
>   src/declarativeimports/core/iconitem.h 26ee410 
>   src/declarativeimports/core/iconitem.cpp fed2f9b 
> 
> Diff: https://git.reviewboard.kde.org/r/116024/diff/
> 
> 
> Testing
> -------
> 
> Test app: http://paste.kde.org/pl5pwdnel
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140227/c232abfb/attachment.html>