blitz-research / monkey

Blitz Research Monkey Source
225 stars 59 forks source link

added *.build* to gitignore #61

Open skn3 opened 10 years ago

skn3 commented 10 years ago

Hey Mark,

The "ANDROID_MAINFEST_ACTIVITY_PROPERTIES" needs to be replicated into the android_ndk target.

added gitignore .build added androidmanifest and transcc var ANDROID_MAINFEST_ACTIVITY_PROPERTIES

skn3 commented 10 years ago

Also added small tweak to FixDataPath so it doesn't mangle paths starting with ~

skn3 commented 10 years ago

Made FixPath public so we can manually use monkey compatible paths (monkey://blah) from 3rd party modules.

blitz-research commented 10 years ago

FixPath is a bit vague - perhaps FilePath? FileSystemPath?

mojo FixDataPath needs to go! mojo file paths shouldn't be different from 'other' file paths - ie: the paths you pass to DataBuffer.Load should be handled the same as the paths you pass to LoadImage.

All would work more or less equivalently if app currentdir started in 'data' dir. ChangeDir would then 'break' LoadImage( "blah" ) if blah were in data dir, but user just needs to use monkey://data/blah instead. Need to think about this a bit more when I get a chance.

On Thu, Jul 17, 2014 at 2:34 AM, Jonathan notifications@github.com wrote:

Made FixPath public so we can manually use monkey compatible paths (monkey://blah) from 3rd party modules.

— Reply to this email directly or view it on GitHub https://github.com/blitz-research/monkey/pull/61#issuecomment-49174249.

skn3 commented 10 years ago

FileSystemPath sounds good. Is this something I should change on my repo or are you going to merge the other changes and then rename?

I am using it in a module called imagecache. Essentially it lets you do threaded download of images. It also allows you to load/save monkey images too. I use a similar technique with a webview module I made to take snapshots of the webview and then load/save them as monkey images.

Unfortunately this require two changes to the image surface class to add a GetSurface() and GetSurfaceDimensions(). Is this something you will see supporting in monkey ever? (probably a bit off topic)