From dc0e9097d3144a3dda81039f666ee43ba29cbc36 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Blot?= Date: Fri, 21 Apr 2017 23:40:48 +0200 Subject: Fix various performance issues reported by cppcheck + code style (CI) (#5635) * Make CI happy with code style on master * guiFileSelectMenu: remove useless includes * some performance fixes pointed by cppcheck * remove some useless casts * TextDest: remove unused setFormSpec function --- src/wieldmesh.h | 25 +++++++++++-------------- 1 file changed, 11 insertions(+), 14 deletions(-) (limited to 'src/wieldmesh.h') diff --git a/src/wieldmesh.h b/src/wieldmesh.h index c98b469d9..ef164c11f 100644 --- a/src/wieldmesh.h +++ b/src/wieldmesh.h @@ -31,7 +31,8 @@ struct ContentFeatures; /*! * Holds color information of an item mesh's buffer. */ -struct ItemPartColor { +struct ItemPartColor +{ /*! * If this is false, the global base color of the item * will be used instead of the specific color of the @@ -43,15 +44,12 @@ struct ItemPartColor { */ video::SColor color; - ItemPartColor(): - override_base(false), - color(0) - {} + ItemPartColor() : override_base(false), color(0) {} - ItemPartColor(bool override, video::SColor color): - override_base(override), - color(color) - {} + ItemPartColor(bool override, video::SColor color) + : override_base(override), color(color) + { + } }; struct ItemMesh @@ -75,8 +73,7 @@ public: s32 id = -1, bool lighting = false); virtual ~WieldMeshSceneNode(); - void setCube(const ContentFeatures &f, v3f wield_scale, - ITextureSource *tsrc); + void setCube(const ContentFeatures &f, v3f wield_scale, ITextureSource *tsrc); void setExtruded(const std::string &imagename, v3f wield_scale, ITextureSource *tsrc, u8 num_frames); void setItem(const ItemStack &item, Client *client); @@ -133,7 +130,7 @@ scene::SMesh *getExtrudedMesh(ITextureSource *tsrc, const std::string &imagename * be NULL to leave the original material. * \param colors returns the colors of the mesh buffers in the mesh. */ -void postProcessNodeMesh(scene::SMesh *mesh, const ContentFeatures &f, - bool use_shaders, bool set_material, video::E_MATERIAL_TYPE *mattype, - std::vector *colors); +void postProcessNodeMesh(scene::SMesh *mesh, const ContentFeatures &f, bool use_shaders, + bool set_material, video::E_MATERIAL_TYPE *mattype, + std::vector *colors); #endif -- cgit v1.2.3