Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add install target for Linux and Mac #206

Open
wants to merge 26 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
5475d38
Add install targets for the libraries and headers.
kuzi117 Aug 18, 2017
1b16bcc
Generate protocol headers instead of glob.
kuzi117 Aug 18, 2017
e2c5c3e
Disable installing CivetWeb's executable.
kuzi117 Aug 18, 2017
1d886d0
Remove debug print.
kuzi117 Aug 19, 2017
ac9855f
Add export and components.
kuzi117 Aug 20, 2017
ad776a3
Add install scripts.
kuzi117 Aug 20, 2017
dbb6817
Use install script
kuzi117 Aug 20, 2017
d488e71
Fix cmake files destination.
kuzi117 Aug 21, 2017
beafffb
Disable SC2APITargets.cmake.
kuzi117 Aug 21, 2017
3e246b4
Disable installing export set.
kuzi117 Aug 21, 2017
b90715e
Nit pick cleanup spacing and indentation.
drivehappy Aug 16, 2017
f4930ea
Refactoring manual memory management to use unique_ptr's where approp…
drivehappy Aug 16, 2017
6f6eb44
Refactor to range-based for loop for consistency.
drivehappy Aug 16, 2017
331074f
Switched implementation of for-loop to use the any_of algorithm. Refa…
drivehappy Aug 16, 2017
c546e9c
Add linking to pthread.
kuzi117 Aug 21, 2017
2c67723
Updated civetweb to latest.
AnthonyBrunasso Aug 21, 2017
7f0e11b
Make include path absolute.
kuzi117 Aug 23, 2017
5501549
Merge branch 'master' of github.com:Blizzard/s2client-api
kuzi117 Sep 8, 2017
6901d7d
Move to correct civetweb.
kuzi117 Sep 8, 2017
90b194a
Make windows install to Documents directory.
kuzi117 Sep 12, 2017
da749e9
Revert "Make windows install to Documents directory."
kuzi117 Sep 12, 2017
c07abae
Merge remote-tracking branch 'blizz/master'
kuzi117 Sep 28, 2017
028086a
Merge remote-tracking branch 'blizz/master'
kuzi117 Nov 25, 2017
e4da6db
Move cmake include to keep readability.
kuzi117 Nov 25, 2017
732aba0
Move protocol headers to folder.
kuzi117 Nov 25, 2017
8b258a3
Merge branch 'master' into master
kuzi117 Apr 14, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Refactor to range-based for loop for consistency.
drivehappy authored and kuzi117 committed Aug 21, 2017
commit 6f6eb44dc732dea767349aed34757815bc1d6f3b
7 changes: 3 additions & 4 deletions src/sc2api/sc2_client.cc
Original file line number Diff line number Diff line change
@@ -1234,13 +1234,12 @@ void DebugImp::SendDebug() {
unit_value->set_unit_tag(set_unit_value.tag);
}

for (std::size_t i = 0; i < debug_state_.size(); ++i) {
for (const SC2APIProtocol::DebugGameState& state : debug_state_) {
SC2APIProtocol::DebugCommand* command = request_debug->add_debug();
command->set_game_state(debug_state_[i]);
command->set_game_state(state);
}

for (const DebugUnit& unit : debug_unit_)
{
for (const DebugUnit& unit : debug_unit_) {
if (unit.count < 1) {
continue;
}