Convert to QVector3D in LineMesh
ClosedPublic

Authored by ervin on Aug 23 2018, 9:47 PM.

Details

Summary

No need to carry the weight of that extra float all the way down or to
depend on it in GridMesh. I suspect even the FileLoader doesn't need it
since in every point of use I found so far w is just dropped.

Diff Detail

Repository
R231 Atelier
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ervin requested review of this revision.Aug 23 2018, 9:47 PM
ervin created this revision.
ervin set the repository for this revision to R231 Atelier.Aug 23 2018, 9:55 PM
rizzitello requested changes to this revision.Aug 24 2018, 1:39 AM

I was able to apply this but it is not based on master and should be rebased.

src/widgets/3dview/linemesh.cpp
63

This is not our style

 std::transform(pos.cbegin(), pos.cend(), std::back_inserter(vertices), [] (const QVector4D &x) { 
    return x.toVector3D(); 
});
This revision now requires changes to proceed.Aug 24 2018, 1:39 AM
tcanabrava accepted this revision.Aug 24 2018, 9:22 AM
tcanabrava added inline comments.
src/widgets/3dview/linemesh.cpp
63

this is the style for stl - with c++ moving to c++20 I have no problem with it (I actually prefer some of that)

laysrodrigues accepted this revision.Aug 25 2018, 12:30 PM

You should run the style script in scripts/style/sh for all your diffs.

ervin updated this revision to Diff 40454.Aug 26 2018, 9:04 AM

Applying the style script

rizzitello accepted this revision.Aug 26 2018, 10:57 AM
This revision is now accepted and ready to land.Aug 26 2018, 10:57 AM
This revision was automatically updated to reflect the committed changes.