Build fix for OSX 10.9

[grrussel]

diff --git a/src/Interface/TextList.cpp b/src/Interface/TextList.cpp
index 58adef3..f76de21 100644
--- a/src/Interface/TextList.cpp
+++ b/src/Interface/TextList.cpp
@@ -570,7 +570,7 @@ void TextList::setCondensed(bool condensed)
  */
 int TextList::getSelectedRow() const
 {
-       int selRow = std::min(_selRow, _rows.size());
+       int selRow = std::min(_selRow, (unsigned)_rows.size());
        return _rows[selRow];
 }
 
fixes the compilation.

[SupSuper]

But size_t's are always unsigned.

[myk002]

i think it may be because it's comparing an unsigned int to an unsigned long, and can't match a <template>(T, T).  casting the long to unsigned (that is, unsigned int) will make them the same bit size and match the template.

[grrussel]

Not always 32bit in size, however ;-)

The OSX builds are 64bit, and therefore would have a 64bit size_t type

[SupSuper]

Good catch. I've replaced "unsigned int" with "size_t" where relevant.

[grrussel]

One more;

Basescape/MiniBaseView.cpp:84:20: error: out-of-line definition of 'setSelectedBase' does not match any declaration in 'OpenXcom::MiniBaseView'
void MiniBaseView::setSelectedBase(unsigned int base)
                   ^~~~~~~~~~~~~~~

diff --git a/src/Basescape/MiniBaseView.cpp b/src/Basescape/MiniBaseView.cpp
index 0dbbce3..2c797f4 100644
--- a/src/Basescape/MiniBaseView.cpp
+++ b/src/Basescape/MiniBaseView.cpp
@@ -81,7 +81,7 @@ size_t MiniBaseView::getHoveredBase() const
  * the mini base view.
  * @param base ID of base.
  */
-void MiniBaseView::setSelectedBase(unsigned int base)
+void MiniBaseView::setSelectedBase(size_t base)
 {
        _base = base;
        _redraw = true;

[SupSuper]

Should be fixed.

[myk002]

@grrussel: check out https://github.com/SupSuper/OpenXcom/pulls?direction=desc&page=1&sort=created&state=open for submitting pull requests.  it's easier on the devs than posting diffs in the forums, and the diffs will stay up to date even when surrounding code changes.

[grrussel]

The makefile.simple no longer specifies a default target of OSX, so the OSX autobuilder got broken.

Now, the makefile can detect OSX automatically, se

https://gist.github.com/anonymous/626242249065b31f39d9

for the 3 line patch.

[SupSuper]

Added. I removed the OSX by default because some Linux users also use that Makefile and it threw them off