These are chat archives for CSE450-MSU/Raven

18th
Nov 2014
Josh Curl
@joshwget
Nov 18 2014 00:57
I can't get the code to compile. Can whoever pushed last try to fix it?
grotheju
@grotheju
Nov 18 2014 01:03
I can't either but it's because the symbol table needs to be updated to take in functions
grotheju
@grotheju
Nov 18 2014 01:37
And another thing. Inside of the func_declaration I don't think It should be variable because it needs to have a type before it, not just the id
chelsea
@seadb
Nov 18 2014 03:38
I fixed it
grotheju
@grotheju
Nov 18 2014 03:38
Is it pushed?
chelsea
@seadb
Nov 18 2014 03:39
Yes
grotheju
@grotheju
Nov 18 2014 03:39
Okay I'll check it out
BTW are you guys getting the warning that two rules can't be matched in the lexer
chelsea
@seadb
Nov 18 2014 03:40
I think it has to do with array indexing or the sizes
if you want to fix it, look at tube7-parser.output
that's the bison debugging file
grotheju
@grotheju
Nov 18 2014 03:40
oh wait. look in the lexer line 74
chelsea
@seadb
Nov 18 2014 03:40
it prints out what the conflicts are form
grotheju
@grotheju
Nov 18 2014 03:41
and line 75
it's two statements and its interpreting them or is trying to interpret them as tokens
i couldnt understand why the declarations were below the %% but it worked
grotheju
@grotheju
Nov 18 2014 03:45
Okay I guess yeah. Thanks for the link
chelsea
@seadb
Nov 18 2014 03:47
it works without those declarations there
i think they accidentally put them in the wrong spot or something
grotheju
@grotheju
Nov 18 2014 03:48
Yeah possibly because it is literally trying to match those as tokens so I don't think the statement would work anyway
chelsea
@seadb
Nov 18 2014 03:49
yeah, i moved them to the normal spot and it still catches unterminated strings
we still have 22 shift/reduce conflicts
grotheju
@grotheju
Nov 18 2014 03:52
hmmm
chelsea
@seadb
Nov 18 2014 03:56
fixed it
grotheju
@grotheju
Nov 18 2014 03:56
What was wrong?
chelsea
@seadb
Nov 18 2014 03:57
i had expression "[" expression "]"
and ID "[" expression "]"
grotheju
@grotheju
Nov 18 2014 03:57
ahhh makes sense
chelsea
@seadb
Nov 18 2014 03:58
yeah, but it works
when you run it on a function example though it says "syntax error"
grotheju
@grotheju
Nov 18 2014 03:58
gimme a sec
What example are you using? The one from the project description?
chelsea
@seadb
Nov 18 2014 04:00
yeah
i just pushed my example.tube
grotheju
@grotheju
Nov 18 2014 04:01
It's giving a line number that doesn't exist
chelsea
@seadb
Nov 18 2014 04:01
yeah
grotheju
@grotheju
Nov 18 2014 04:01
hmmmm
chelsea
@seadb
Nov 18 2014 04:01
some of the line numbers are off in general
grotheju
@grotheju
Nov 18 2014 04:05
That's not helpful haha
chelsea
@seadb
Nov 18 2014 04:07
i have a print statement in the CompileTubeIC for ASTNodeFunction and ASTNodeReturn
its not getting to either of them though
grotheju
@grotheju
Nov 18 2014 04:10
Yeah i saw the return return function
chelsea
@seadb
Nov 18 2014 04:10
CompileTubeIC doesn't get called does it
grotheju
@grotheju
Nov 18 2014 04:11
I don't think so
chelsea
@seadb
Nov 18 2014 04:11
Actually it does
But for some reason its not even getting up the code inside statement_list
up to*
Ooooohh
grotheju
@grotheju
Nov 18 2014 04:12
that sounds promising
chelsea
@seadb
Nov 18 2014 04:12
Its because of line 13 i think
grotheju
@grotheju
Nov 18 2014 04:12
in .y or ast.cc
chelsea
@seadb
Nov 18 2014 04:12
but it doesn't use that line because the function is declared all at once
line 13 in the example.tube
actually its line 14 i think
grotheju
@grotheju
Nov 18 2014 04:15
Did you touch the symbol table for the pointer to the function body?
chelsea
@seadb
Nov 18 2014 04:16
I put it in symbol_table.h but don't use it yet
grotheju
@grotheju
Nov 18 2014 04:16
Okay cool
chelsea
@seadb
Nov 18 2014 19:31
i just pushed a lot of changes
so make sure to fetch and merge
]';
Josh Curl
@joshwget
Nov 18 2014 19:34
I'll be contributing more when we get to recursion
Sorry i haven't done much yet
chelsea
@seadb
Nov 18 2014 21:36
no problem