[Ohrrpgce] SVN: teeemcee/8705 Add Slice.{Pos, ScreenPos, Size}, Frame.size, Surface.size, MouseInfo.pos,

[James Paige]

Because If it is in a VM, I won't accidentally break it when I mess with my
Android SDK for unrelated projects :)

(I could probably set up a temporary non-VM nightly build, but I have to
double check whether or not my Android SDK is *already* broken on my
nightly build machine from the last time I upgraded something Java related)

On Thu, May 18, 2017 at 9:34 AM, Ralph Versteegen <teeemcee at gmail.com>
wrote:

> Why would you need a VM for that?
>
> On 19 May 2017 at 04:09, James Paige <Bob at hamsterrepublic.com> wrote:
>
>> I haven't had time yet, but sooner or later I will get an Android nightly
>> build VM up and running so we can spot such things quicker.
>>
>> On Thu, May 18, 2017 at 9:05 AM, Ralph Versteegen <teeemcee at gmail.com>
>> wrote:
>>
>>> Fixed. The NDK r8 toolchain uses gcc 4.6. I was going to say it doesn't
>>> support that c++11 feature, but actually looks more like a compiler bug.
>>>
>>> On 19 May 2017 at 02:31, James Paige <Bob at hamsterrepublic.com> wrote:
>>>
>>>> Double Opps!
>>>>
>>>> 8704 is the last revision that worked on android.
>>>> 8705 was broken on other platforms
>>>> 8706 fixed other platforms, but is still broken on Android.
>>>>
>>>> Here is the place where it fails:
>>>>
>>>> Compile++ thumb  : application <= surface.cpp
>>>> /home/james/misc/android-ndk-r8d/toolchains/arm-linux-androi
>>>> deabi-4.6/prebuilt/linux-x86/bin/arm-linux-androideabi-g++ -MMD -MP
>>>> -MF ./obj/local/armeabi/objs-debug/application/ohrrpgce/tmp/surface.o.d
>>>> -fpic -ffunction-sections -funwind-tables -fstack-protector
>>>> -D__ARM_ARCH_5__ -D__ARM_ARCH_5T__ -D__ARM_ARCH_5E__ -D__ARM_ARCH_5TE__
>>>> -no-canonical-prefixes -march=armv5te -mtune=xscale -msoft-float
>>>> -fno-exceptions -fno-rtti -mthumb -Os -g -DNDEBUG -fomit-frame-pointer
>>>> -fno-strict-aliasing -finline-limit=64 -O0 -UNDEBUG -marm
>>>> -fno-omit-frame-pointer -Ijni/../jni/application/ohrrpgce/tmp
>>>> -Ijni/../jni/application/../sdl-1.2/include
>>>> -Ijni/../jni/application/../sdl_mixer/include
>>>> -Ijni/../jni/application/../fb/include -I/home/james/misc/android-ndk
>>>> -r8d/sources/cxx-stl/gnu-libstdc++/4.6/include
>>>> -I/home/james/misc/android-ndk-r8d/sources/cxx-stl/gnu-libstdc++/4.6/libs/armeabi/include
>>>> -Ijni/../jni/application -DANDROID -O3 -DFBCVERSION=1060 --std=c99
>>>> -Wa,--noexecstack --std=c++0x -g -Wall -Wno-non-virtual-dtor  -g
>>>> -I/home/james/misc/android-ndk-r8d/platforms/android-4/arch-arm/usr/include
>>>> -c  jni/../jni/application/ohrrpgce/tmp/surface.cpp -o
>>>> ./obj/local/armeabi/objs-debug/application/ohrrpgce/tmp/surface.o
>>>> cc1plus: warning: command line option '-std=c99' is valid for C/ObjC
>>>> but not for C++ [enabled by default]
>>>> jni/../jni/application/ohrrpgce/tmp/surface.cpp: In function 'void
>>>> shrinkrow(RGBcolor*, int, RGBcolor*, int, int, fixedpoint)':
>>>> jni/../jni/application/ohrrpgce/tmp/surface.cpp:157:85: error: cannot
>>>> convert '<brace-enclosed initializer list>' to 'uint32_t {aka unsigned
>>>> int}' in initialization
>>>> make: *** [obj/local/armeabi/objs-debug/application/ohrrpgce/tmp/surface.o]
>>>> Error 1
>>>> make: *** Waiting for unfinished jobs....
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> On Thu, May 18, 2017 at 7:12 AM, James Paige <Bob at hamsterrepublic.com>
>>>> wrote:
>>>>
>>>>> Opps!
>>>>>
>>>>> It looks like revision 8705 broke the Android build. It fails with the
>>>>> following:
>>>>>
>>>>> fbc -c gfx_sdl.bas -o build/game-gfx_sdl.o -mt -exx -g -O 2 -target
>>>>> linux -arch 686 -d DATAFILES='"/usr/share/games/ohrrpgce"' -d IS_GAME
>>>>> -m game
>>>>> /home/james/src/ohr/wip/surface.bi(37) error 14: Expected identifier,
>>>>> found 'XYPair' in 'size as XYPair'
>>>>> scons: *** [build/game-gfx_sdl.o] Error 1
>>>>> scons: building terminated because of errors.
>>>>>
>>>>>
>>>>> Not sure why this works on other platforms but fails on Android
>>>>>
>>>>>
>>>>>
>>>>> On Mon, May 1, 2017 at 9:29 AM, <subversion at hamsterrepublic.com>
>>>>> wrote:
>>>>>
>>>>>> teeemcee
>>>>>> 2017-05-01 09:29:43 -0700 (Mon, 01 May 2017)
>>>>>> 165
>>>>>> Add Slice.{Pos,ScreenPos,Size}, Frame.size, Surface.size,
>>>>>> MouseInfo.pos, which are in union with the existing members
>>>>>>
>>>>>> Start to use these aliases around the codebase
>>>>>> ---
>>>>>> U   wip/allmodex.bas
>>>>>> U   wip/allmodex.bi
>>>>>> U   wip/common.rbas
>>>>>> U   wip/scriptcommands.bas
>>>>>> U   wip/slices.bas
>>>>>> U   wip/slices.bi
>>>>>> U   wip/surface.bi
>>>>>> _______________________________________________
>>>>>> Ohrrpgce mailing list
>>>>>> ohrrpgce at lists.motherhamster.org
>>>>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherh
>>>>>> amster.org
>>>>>>
>>>>>
>>>>>
>>>>
>>>> _______________________________________________
>>>> Ohrrpgce mailing list
>>>> ohrrpgce at lists.motherhamster.org
>>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>>>
>>>>
>>>
>>> _______________________________________________
>>> Ohrrpgce mailing list
>>> ohrrpgce at lists.motherhamster.org
>>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>>
>>>
>>
>> _______________________________________________
>> Ohrrpgce mailing list
>> ohrrpgce at lists.motherhamster.org
>> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>>
>>
>
> _______________________________________________
> Ohrrpgce mailing list
> ohrrpgce at lists.motherhamster.org
> http://lists.motherhamster.org/listinfo.cgi/ohrrpgce-motherhamster.org
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.motherhamster.org/pipermail/ohrrpgce-motherhamster.org/attachments/20170518/7c4e071f/attachment-0001.htm>