Differential D2597 Diff 11239 source/simulation2/components/ICmpGarrisonHolder.cpp

Changeset View

Standalone View

source/simulation2/components/ICmpGarrisonHolder.cpp

This file was added.

				/* Copyright (C) 2020 Wildfire Games.
				* This file is part of 0 A.D.
				*
				* 0 A.D. is free software: you can redistribute it and/or modify
				* it under the terms of the GNU General Public License as published by
				* the Free Software Foundation, either version 2 of the License, or
				* (at your option) any later version.
				*
				* 0 A.D. is distributed in the hope that it will be useful,
				* but WITHOUT ANY WARRANTY; without even the implied warranty of
				* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
				* GNU General Public License for more details.
				*
				* You should have received a copy of the GNU General Public License
				* along with 0 A.D. If not, see <http://www.gnu.org/licenses/>.
				*/

				#include "precompiled.h"

				#include "ICmpGarrisonHolder.h"

				#include "simulation2/scripting/ScriptComponent.h"
				#include "simulation2/system/InterfaceScripted.h"

				BEGIN_INTERFACE_WRAPPER(GarrisonHolder)
				END_INTERFACE_WRAPPER(GarrisonHolder)

				class CCmpGarrisonHolderScripted : public ICmpGarrisonHolder
				{
				public:
				DEFAULT_SCRIPT_WRAPPER(GarrisonHolderScripted)

				virtual std::set<entity_id_t> GetEntities() const
				{
				SilierUnsubmitted Done Inline Actions does this need to require set ? if code could use vector, that copying could be avoided Silier: does this need to require set ? if code could use vector, that copying could be avoided
				std::vector<entity_id_t> entities = m_Script.Call<std::vector<entity_id_t>>("GetEntities");
				std::set<entity_id_t> result;
				for (std::vector<entity_id_t>::iterator it = entities.begin(); it != entities.end(); ++it)
				elexisUnsubmitted Done Inline Actions `>>` -> `> >` elexis: `>>` -> `> >`
				result.insert(*it);
				StanUnsubmitted Done Inline Actions range based for loop? Stan: range based for loop?
				return result;
				}

				virtual void SetInitEntities(std::set<entity_id_t> entities)
				{
				for (std::set<entity_id_t>::iterator it = entities.begin(); it != entities.end(); ++it)
				{
				StanUnsubmitted Done Inline Actions Range based for loop? Stan: Range based for loop?
				m_Script.CallVoid("AddInitGarrisonEntity", *it);
				SilierUnsubmitted Done Inline Actions you can remove these brackets :) Silier: you can remove these brackets :)
				}
				SilierUnsubmitted Done Inline Actions btw, this could pass array of them, no ? Instead one by one Silier: btw, this could pass array of them, no ? Instead one by one
				FreagarachAuthorUnsubmitted Done Inline Actions I tried that at first but it didn't work. Freagarach: I tried that at first but it didn't work.
				}
				};

				REGISTER_COMPONENT_SCRIPT_WRAPPER(GarrisonHolderScripted)