Review: Approve
Yes, that looks good. Thanks! :)
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/findpath_modification/+merge/337103
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/findpath_modification.
next try please :)
--
https://code.launchpad.net/~widelands-dev/widelands/findpath_modification/+merge/337103
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/findpath_modification.
___
Mailing list: https://launchpad.
I replied in the diff.
Maybe you need a different tutorial, or to sit down with somebody to teach you.
That tutorial is maybe too abstract, and you need some exercises to go along
with it? Or maybe what's confusing you here is the combination of pointers and
object orientation?
Diff comments:
see comment in diff
I know the link with tutorial - very well. If it is about integers I somehow
understand it, but objects or std containers are too confusing to me. I know, I
should try harder ;)
Diff comments:
>
> === modified file 'src/ai/defaultai.cc'
> --- src/ai/defaultai.cc 2018
> Also see comment in the code...
Comment looks good, can you please add it to the code?
Regarding pointers and references, they are an important feature of C++, so if
you can find the time to read up on them, it will be time well spent. Here's a
tutorial: http://www.cplusplus.com/doc/tutorial/
Appveyor is not able to parallel build x64 Debug due to memory constraints on
Appveyor (see build times for every job: x86 release/debug + x64 release builds
are taking ~30 minutes, x64 debug takes ~90 minutes.)
But now in this branch we also hit the time limit of 120 minutes with the x64
debug
I am not sure what appveyor problem exactly is...
--
https://code.launchpad.net/~widelands-dev/widelands/findpath_modification/+merge/337103
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/findpath_modification.
___
M
I am hopelessly lost in regard to pointers and references and I am afraid it
will never get better.
Also see comment in the code...
Diff comments:
>
> === modified file 'src/ai/defaultai.cc'
> --- src/ai/defaultai.cc 2018-02-12 11:43:28 +
> +++ src/ai/defaultai.cc 2018-02-13 07
The constructor failed because you were initializing with a reference in the
other constructor. I have changed that to a pointer, since the object has a
pointer internally anyway.
For the loop, it's not clear to me how the loop above the end condition
actually ensures that eventually there's no
Gun,
your suggestion about NearFlag constructor is failing with:
no matching function for call to ‘Widelands::NearFlag::NearFlag(std::nullptr_t,
int)’
Also in regard to (;;) I changed a comment few lines bellow it to be obvious
where the looping ends:
"OK, so no NearFlag left to be checked -
Various small code style nits. There is 1 loop that I'm worried about - can you
please revisit the code there and add an explicit break condition, and/or a
comment?
Diff comments:
> === modified file 'src/ai/ai_help_structs.cc'
> --- src/ai/ai_help_structs.cc 2017-11-24 09:19:52 +
> +++ src
Review: Resubmit
OK, ready for testing
--
https://code.launchpad.net/~widelands-dev/widelands/findpath_modification/+merge/337103
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/findpath_modification.
___
Mailing lis
wait a bit, I am doing further modification to the code..
--
https://code.launchpad.net/~widelands-dev/widelands/findpath_modification/+merge/337103
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/findpath_modification into lp:widelands.
__
13 matches
Mail list logo