k done

[Josh @ Dreamland]

After some grueling moments of agony, I have the new system working—as far as I can tell—on Linux. TGMG is testing it for Mac, and then I guess I'll make it work on Windows and merge it in to the main branch.

I'm more concerned about Mac because, number one, its headers are older and very different, and number two, TGMG has a massive testing platform running on it which we can use for automated regression testing.

In other news, I fixed depth, too.

[luiscubal]

TGMG has a massive testing platform running on it

Can't(in fact, shouldn't) you port it to other platforms too? I mean, wouldn't this testing platform also help finding platform-specific bugs that would otherwise not be found (since an OS X-only testing suite can't find Windows/Linux bugs)

[Josh @ Dreamland]

You make a great point, but let me offer this as a rebuttal: I'm lazy. I have higher priorities.

(Note from Ism: ftfy)

[cheeseboy]

@josh
Since even your great generosity has bounds, and you have thus disallowed me from spamming shit messages on your IRC channel while still graciously allowing me to shitpost here, I suppose I will do so. Well it parses 99% of my game. Apparently you broke median (it only accepts 1 argument now?). Also almost none of my sprites animate anymore and sounds completly borked too. I hear speakers click on then nothing. Way to break everything (not that I didn't expect it).

@ismavatar
Definitions aren't geting reset. I have to delete them everytime I open a new instance of lgm. Also I'm onubuntu temporarily and I have to say having the console as a seperate popup is 9001x more annoying unity. Once it gets pushed to the back I have no idea when it errors then I have to deal with unity's shitty UI to find it. Shouldn't the console be intergrated into main window? or jump to foreground when theres an error? Also what happened to making errors red and warnings yellow in console?

@darkacez
You're 12.

@polygon
You're gayer than me.

[IsmAvatar]

Shouldn't the console be intergrated into main window?

Already planned. I think RetroX was working on it, but I haven't seen him for a while. He did some of the core work, and I think I can probably plug it in myself now, but I was wondering if he was also going to plug it in himself and save me the trouble. Guess not, so I'll go ahead and do that when I get a chance.

Also what happened to making errors red and warnings yellow in console?

Not sure. I'll look into it. I vaguely thought I already had it implemented. Maybe it broke. At any rate, I'll poke at that too when I get a chance, and hopefully get that working.

[cheeseboy]

@ismavatar
It appears the sprite issue is egm related. Any egm I open only shows the first image under subimages. I'm not sure if there there and not loading or getting deleted upon save. If i open gm8 or similar then save as egm and reeopen I can no loger see them though. Also josh edit has this odd behavior when being dragged It's pretty harmless but made a video so you could see http://ubuntuone.com/4ruUhJemKrGvWfhZcQGq8Y .

[Josh @ Dreamland]

Bear in mind, cheeseboy, I can very easily apply all the IP bans from the IRC here.

I have not touched sounds. If they don't work anymore, it has nothing to do with the parser. The parser only determines whether or not the sounds are functions. The other problems you are describing sound like an issue identifying locals, which was already fixed.

[cheeseboy]

Fixed and commited?

Code: [Select]

greg@greg-laptop:~/enigma-dev$ git status
# On branch enigma-jdi
# Untracked files:
#   (use "git add <file>..." to include in what will be committed)
#
# fail.log
# update.sh
nothing added to commit but untracked files present (use "git add" to track)

draw_sprite_part(sprite_index, -1, 0, 0, median(0, obj_house.x + wall - bbox_left, sprite_width), sprite_height, x, y);

Line 1, position 82: Too many arguments to function `median': provided 3, allowed 1.


[Josh @ Dreamland]

That, I am looking into now. Median is the only function correctly identified to take variadic parameters. Due to issues in resolving ::, min, max, and choose are not read correctly. The bug in resolving median as variadic is ENIGMA's fault; I'll look into it after the other three read properly.

[cheeseboy]

Whatever you just commited broke instance_nearest.

Code: [Select]

Line 4, position 41 (absolute 121): Too few arguments to function `instance_nearest': provided 3, required 4.also unban kthnx.

[Josh @ Dreamland]

What I committed didn't break anything. It fixed your problem, revealing the next error it encountered while trying to parse your code.

I will commit the fix to the instance_nearest problem after I finish writing these diagnostic toString methods.

[cheeseboy]

Uhm most definetly broke it. Whole game ran after comenting out the median line before .

also unban kthnx.

[Josh @ Dreamland]

If it broke, it was because the parser was not checking errors correctly at all, because the `variadic' field given to each parameter in memory was not initialized. While it's possible for what seemed like a regression to surface, it's a sharp statistical improbability.

Either way, I've pushed the version that initializes those fields, and sets them correctly.

[cheeseboy]

\o/ It compiles. Now If only lgm could load my my sprites...

(I think thats also your domain. Mr. APNG)

also unban kthnx.

[Josh @ Dreamland]

I have JDI compiling and working (zero errors, zero warnings) on Windows with the following headers:

Code: (cpp) [Select]

#include <GL/gl.h>
#include <GL/glu.h>

#include <stdio.h> //printf, NULL
#include <stdlib.h> //malloc
#include <time.h> //clock

#include <stdio.h>
#include <stdlib.h>

#include <errno.h>
#include <float.h>
#include <iso646.h>
#include <math.h>
#include <locale.h>
#include <stdarg.h>
#include <wchar.h>
#include <stdint.h>
#include <string.h>
#include <stdbool.h>
#include <stddef.h>
#include <malloc.h>

#include <stdint.h>
#include <time.h>

#include <fcntl.h>
#include <sys/types.h>
#include <signal.h>
#include <setjmp.h>
#include <inttypes.h>

// Once affected by lack of '?' operator token
#include <ctype.h>
#include <wctype.h>
#include <limits.h>
#include <assert.h>


#include <windows.h>


// Once affected by lack of support for public/private/protected, and by lack of support for constructors.
#include <pthread.h>

#include <sys/stat.h>
#include <sys/time.h>

#include <fenv.h>
#include <tgmath.h>
#include <complex.h>

I'll probably commit what I have shortly, assuming nothing segfaults and dies in ENIGMA (Which I'm already sure that it will).