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

[Ralph Versteegen]

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-
> androideabi-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-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/20170519/ec84753e/attachment.htm>