Removing the old LIFE_RANGE_WITHOUT_LAST_READ support

strip the newlines out of ir error messages, they're added by the msgprint routine

more context copies

have the ir take over some more context data from the ast

add the backticks for a warning

add the type for the invalid-type-for-global message

ir_value_set_string needs to use a strdup which doesn't return NULL for an emptystring

take 'warnreturn' into account, don't ignore the parsewarnin's Werror status

Don't have the ast's codegen error on missing returns

set -Wmissing-return-values to on by default, new warning: control reaches end of non-void...

parse_block gets a warn-return bool flag

functions which have no living values but store_return values may cause the allocator to be run with no allocations being made - in this case we shouldn't try accessing the empty array and simply exit the allocator

same as before for loops: don't try creating a jump if the body ends in a return

don't try to create jumps in ifs if the block is already done

have function declarations of functions with prototypes take over the actual declaration's body's context data

provide the context to ir_functions

relax parse_close_paren to allow values in parenthesis with no operators... duh

fix: use thinkfunc not nextthink, nextthink is set later

Automatic prototyping of frame-functions

Removing old commented-out code

missing initializer

IR generation: for functions: first generate all defs, then the code, so access to prototyped functions doesn't result in NULL code

setting data/frames.qc right again

STOREP instructions' first operand has to be considered a read-operand, not a write operand, otherwise the life ranges will be wrong, and the value itself IS only read, not written to

for void returning functions we can use store_return to avoid a useless store after it

calls for now use store_value since we don't properly deal with its liferange yet; fixing some local-alloation things; taking call parameters into account in ir_block_life_propagate

Fixing a misleading error message in ir.c

ir: fix bug that functions without declared locals would put temps at position zero ...

Nicer prints in exec's prog_print_statement

fix a wrong print in parsing.qc test

Oops setting wrong members

data/frames.qc to test [frame,think]

First draft of handling the [frame,think] notation

qcvm: -trace flag

Factoring out duplicate code into a function: create_vector_members; creating vector members for vector function parameters

implement correct behaviour for shadows locals

-Wlocal-shadows - note that -std=gmqcc is very different from the rest

splitting parser_find_local up into find_local and find_param - find_local also takes a bool-ptr to tell the caller if it was a local or a global

quotes around the variable name in some messages

remove trailing \n from 2 error messages

set olddecl so we don't access a NULL value when a variable already exists

oops - parsewarning returns true on -Werror, so we gotta invert here

-Wtoo-few-parameters on by deafult - for -std=gmqcc it's always an error

ast_type_adopt - ast_entfield now adopts the full type of the field

-Wmissing-return-values, off by default, ignored with -std=gmqcc because in that standard it'll error

data/vars.qc - when declaring a function, and it had a prototype - use the new parameter names instead of the ones from the prototype, otherwise things get messy

ast_unary needs to set its type

Disable the member-of namespace check when -std != gmqcc

move the member-of check for '.' to after applying the previous dot operators so we don't need parens around 'a.b' of 'a.b.c = x'

data/functions.qc for some more tests

ast_call: adopt the funciton's return type

ast_return_delete: check if self->operand is actually set before ast_unref-ing it

Use the current context, not the one from a NULL value... for ast_return

 parse 'return' without values better

ast_return should accept NULL as value to create a simple 'return' without a value

Fixup some possibly uninitialized value when parsing 'return'

set expression.next to NULL in ast_block_set_type if it won't be set to a new value

Don't re-add defs which had a prototype

adding data/proto.qc

ast_value_copy should copy the expression substructure as well - exposing ast_value_copy to the outside since the parser needs to copy complete types when multiple variables are declared with commas

initialize the ast_expression node in ast_shallow_type properly as well

initialize the ast_expression node in ast_type_copy properly

keep the filenames of all lexed files in a global lex_filenames vector because we don't strdup the filenames into lex_ctx, but copy the pointer

set parser->lex to NULL after closing

don't deallocate the parser on a compile error -_-

ast_member's are now kept, since they are used like variables: multiple times

Add a comment about why we don't delete the owner

Doh... wrong comment type

ast_member will not unref/delete the owner: first of all the owner is always an ast_value, which won't get deleted anyway, secondly: the ast_members are collected and deleted after the owners, so ast_unref(owner) will cause invalid memory access

ast_block_collect: add to ast_block->collect and set the node's .keep=true, those will now always be deleted by the ast_block dtor

initialize ast_block->collect properly

collect vector-member locals in ast_block->collect, like a garbage collection...

Revert "let ast_node have a use-counter, helpful for the parser to delete unused fields which otherwise get lost in the void"

This reverts commit 25ffd11aa63c018632f3906092622353f79a8129.

Revert "ast_usecount macro for convenience"

This reverts commit e86c8ff0eb55b40c5a30accd5915be3a9b08e784.

Revert "delete ast_members from parser->locals when they were unused"

This reverts commit cdeffb6de9b92b7c498380d802d3250ef9ba73aa.

delete ast_members from parser->locals when they were unused

ast_usecount macro for convenience

let ast_node have a use-counter, helpful for the parser to delete unused fields which otherwise get lost in the void

clear lex->modelname

clear all the remaining vectors in the parser

clear items_data in main

clean lex->frames in lex_close

free parser->fields and parser->locals at parser_cleanup

fix a MEM_VECTOR_CLEAR to be done AFTER clearing the contents...

Delete fields in parser_cleanup

the opening paren is now an operator - to fix up the precedence rules, now 'anentity.afunction()' compiles

add 'local' keyword in fields.qc, it's for qcc standard anyway

get rid of additional trailing \n on parseerror

WARN_ERROR -> opts_werror in parser.c

fields: testing function-fields

replacing WARN_ERROR with opts_werror

Fix indentation of lexer.c - was still using some tabs there

lexer now turns '(' into an operator if noops=false

debug warn def on top of the flag list

add the type of whatever is 'not a function' to that error message

Add an error output in the ast where I missed one (there are probably more places which need one)

add LOAD for TYPE_FUNCTION as LOAD_FNC so we can actually access function fields

parser_strdup to wrap util_strdup and actually dup empty strings

Support for $modelname and $framerestore

Don't expand frame macros when they don't have a $ prefix