I haz a bank holiday today :D
Yay for no school :shipit:
Right then; onto optional and required arguments!
@liam4 Destroy Hound, it's too annoying :P
Release 0.3.2
- String escapes
@nanalan Yay for destroying hound :tada:
:P
Trying to get argument lists to work in tmLanguage
did it :tada:
@liam4 why did you spell "parameter" wrong in the source code
that's really annoying
Error: Function no-args expects 0 arguments, was called with 3
:D
it works
no-args => {
print('no-args');
};
no-args('too', 'many', 'arguments');
// Error: Function no-args expects 0 arguments, was called with 3
:tada:
good job! :P
xD
I also fixed the async functions eating errors @liam4
also all builtin functions have
fn.builtin set to true
:P
What's that useful for?
Builtins take any amount of parameters :P
Ew, why would it work like that?
I can't be arsed to add that
@liam4 can do it
:tada:
I may have broken
use(jsLibrary)
xD
gj
Error: Function VARIABLE_IDENTIFIER,fs expects 0 arguments, was called with 1
hm
:P
tulun arrays be like:
a = > arr();
a.0 -> 'foo';
:P
Maybe tulun "arrays" could just be objects
That's ugly.
xD
Like in lua?
mm
and objects be iritatorible(???)
xD
e.g.
loop(imAnObjectWith4Values, {});
lol, I have no idea
anyhoo, why does
potato => (foo, [bar]) {};
work
like, I havent even edited the grammar yet
xD
[] is not in special_chars?
Ah
it isn't
:P
WELL THATS SOMETHING TO ADD
xD
also I'll add
unevaluated to keywords
:P
yey
unevaluated => 'no hax for you';
wow
if I remove
(a space) from the special chars list
I can do this
let me just try this => 'foobar';
print(let me just try this);
and it works
:OOOO
welp, it generates 2 asts but who cares
:tada:
lol
heh
it sees the space before =>
and thinks "is that the variable name or is that just whitespace"
turn
_ "=>" _ into __ "=>" __
"let me just try this" vs "let me just try this " :P
what?
lol
ah
what would that do?
:P
_ is maybe whitespace,__ is required whitespace
ah
omg it works
THIS IS AMAZING
@liam4 pls can I add this
let me just try this => 'foobar';
print(let me just try this);
add credit to me pls
lol
gtg
:P
bye
Bai
pushed :tada:
Back.
@kvackkvack thoughts on spaces in varnames?
:P
Is it something worth pursuing or not...?
Ehh... Maybe.
Is there something that breaks currently when using them?
Yep.
Like what?
Lots of ambigous syntax
e.g. if()
Wait, why does it make if ambigious!?
basically any whitespace
ohh
Hang on, I think I might have a fix - I'll edit some of the grammar and see if it works.
I, uh, reverted the commits
:P
I know. :P
:cake:
It's not very hard to get back though, right?
sure
I dunno.
:P
It should be easy, yeah.
Just revert
arguments to add varname spaces to theme
@liam4 better tests activate
I just need to
nearleyc grammar.ne -o ../dist/grammar.js when editing the grammar, right?
No.
tulun rebuild
??
oh
k
also when you clone
make sure you
npm link
huh?
btw calling
tulun just opens tulun.js in Notepad++ for me
...
same for
tulun rebuild
git clone ...
npm link
tulun rebuild
try dat ^
what does
npm link do?
npm help link
:P
Basically it adds tulun.js to yer PATH
equivalent of
npm install -g
except for local packages
Ah.
It's taking a while... :P
:P
it will
yey, done
You'll only have to to do it once
:tada:
It still only opens tulun.js in Notepad++... Windows problems?
...
are you in the
tlnccuwagnf folder
yes
hum
try
./tulun rebuild
'.' is not recognized as an internal or external command,
operable program or batch file.
node tulun.js rebuild?
events.js:154
throw er; // Unhandled 'error' event
^
Error: spawn gulp ENOENT
at exports._errnoException (util.js:890:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:182:32)
at onErrorNT (internal/child_process.js:348:16)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
at Function.Module.runMain (module.js:449:11)
at startup (node.js:146:18)
at node.js:404:3
;-;
npm install -g gulp
.\tulun rebuild also just opened tulun.js in notepad++
:P
getting the same error still
npm install -g gulp then node tulun.js rebuild still breaks?
```C:\Users\admin\tlnccuwagnf>node -v
v5.10.1
C:\Users\admin\tlnccuwagnf>npm -v
3.8.3```
yes
try
gulp build
:P
bloody windows...
works
:D
So, is that my equivalent of
tulun rebuild?
yep
k
I'll add that to
help
soon, windows 10 will get bash
:P
:shipit:
Just merged arguments & master
for the sake of the new tests
how to run the REPL?
tulun help m8
use
tulun
that doesn't work tho
...oh.
node tulun.js
what happen
i thought we'd already talked about that :P
Welp, that should work
dis happen:
1 # Tulun Read Eval Print Loop #
2 repl => use('./repl.js');
...
then it won't let me input anything
er
npm link
^
what does this error?
i already did that
What does it do?
i'll do it again though
C:\Users\admin\AppData\Roaming\npm\tulun -> C:\Users\admin\AppData\Roaming\npm\node_modules\tlnccuwagnf\tulun.js
C:\Users\admin\AppData\Roaming\npm\node_modules\tlnccuwagnf -> C:\Users\admin\tlnccuwagnf
righty
hm
where tulun
C:\Users\admin\tlnccuwagnf\tulun.js
C:\Users\admin\AppData\Roaming\npm\tulun
C:\Users\admin\AppData\Roaming\npm\tulun.cmd
well there you go
that should work
huh?
C:\Users\admin\tlnccuwagnf\tulun.js
also why are you logged in as admin
well, i'm not actually, but it's my last name and like privacy ya know
:P
lol
wait
admin is your last name?
:tada:
lol
git pull?
yey new tests
ugh, repl is broken
whatever
um
C:/Users/admin/tlnccuwagnf/tulun.js does the same thing as tulun, node C:/Users/admin/tlnccuwagnf/tulun.js does the same thing as node tulun
so it's not just me? lol
Yep.
lol
Modify test.tul
I'll try fixing it then...
then
tulun test.tul
No.
:(
k
I need to fix things
:P
nodepad++ (or whatever) test.tul
tulun test.tul
theres yer dev env
edit test.tul and then
tulun it
:shipit:
gtg
See ya!
poof
but I need help ;_;
welp, it generates 2 asts but who cares
I do :P
Will this still work?
my variable => 42; # good #
my variable=>42; # does this work? #
spaces before variable name - are those included => 'idk';
what about after => 'baz';bloody windows...
^^^^^ This.
but I need help ;_;
What do you need help with?
You could make it so spaces are neither allowed at the beginning or end of identifiers
Kind of like numbers are right now
Ah.
Sounds good!
Sounds good!
That'd fix all problems, I think, and
my variable=>42 would still work
You'd need to modify the post-processor
Yeah
Trying to write a .sb2 writer for my language at the moment. :P
Identifier -> GenericCharacter:+ {% function(data, location, reject) {
// this needs work
if (/* insert regex here */) return reject;
else return data.join('');
} %}{% function(data, location, reject) {
var identifier = data[0].join('')
if(C.KEYWORDS.indexOf(identifier) == -1 && !(/[0-9\s]/.test(identifier.charAt(0))) && identifier.charAt(identifier.length - 1) !== ' ') {
return identifier
}
return reject
} %}
This message was deleted
Maybe?
Test it? :P
I can't.
Why not?
gulp build && node tulun.js test.tul
I can't tulun files
Yeah, that doesn't work
Get the bash for windows or whatever thing.
lol
Why don't you test it? :P
cygwin
I'm working on my own thing anyways
Because it almost definitely won't work because it's straight from your head without being tested at all?
?
xD
(hit enter accidentally)
First tries never work :P
No, every part except the added \s in the RegExp and the testing of last character being ' ' is tested already
(in my own nearley file)
Why is this not valid node!?
function(name, args, ...scripts) {
It's giving me an error on the
...
je suis bak
hi
help with node pls
idek
I don't have a Windows machine anymore :(
@liam4 does travis have windows vms too? :shipit:
OOO
aw no wundoge
:P
does node not support function(...args) !??!!????+
...yeah.
y
'use strict'; m8
:P
You need the --harmony flag
Node's annoying :P
m8
node 5 is ~old now
use
nvm :P
is it?
relatively
get
nvm for windoge
what's the current one?
# alex at jupiter in ~/tlnccuwagnf on git:arguments ● [19:47:54]
→ nvm install 6
Downloading https://nodejs.org/dist/v6.0.0/node-v6.0.0-linux-x64.tar.xz...
# # # ## ### ### #### #### ##### ###### ####### ######## ######### ########## ############ ############# ############### ################ ################## ################### ##################### ####################### ########################## ############################# ################################ ################################### ###################################### ######################################### ############################################## ################################################## ##################################################### ########################################################## ############################################################## ######################################################################################################################################### 100.0%
Now using node v6.0.0 (npm v3.8.6)
:P
do rest parameters work in it?
Yes.
What
node -v are you running?
5.10.1
use
node <file> --harmony if you want rest params
Or upgrade yer node
installing nvm right now :P
lol
Current stable is ~5
Current for the swagsters is ~6
:P
Nightly for people who like to live on the edge is Node v7 :O
anyhoo
how do i update
:P
nvm install <nodeversion>
nvm use <installed-nodeversion>
e.g.
nvm install 7
I tried
nvm install 6. turns out you have to do nvm install 6.0.0
works for nvm on linux :shipit:
:(
go get cygwin btw
wai
bash, for windoge
ik
ooo
Is that better?
Oh, it's the same thing
It's cygwin + sweg
But better
:P
anyways
Try the thing I wanted liam to try pls
@liam4 @kvackkvack you know that
String.prototype.trim() works right?
:P
you dummies
Huh?
trim whitespace from edges of strings in js
ik
.trim()
:P
but why would you think we don't know about it?
You did fancy stuff
rite?
:P
wat?
ignore me
nvm
install
7.0.0
:P
we did it!
Huh?
:package:
completed the chain
lol
yep
:P
Testing...
?
Try the thing I wanted liam to try pls
Oh
:shipit:
:P
hmmmmmmmmm
I think node v6 is broken
le revert
wat?
npm i installs the wrong dependencies
xD
WELLL
Error: spawn gulp ENOENT
at exports._errnoException (util.js:896:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:182:32)
at onErrorNT (internal/child_process.js:348:16)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
at Function.Module.runMain (module.js:577:11)
at startup (node.js:159:18)
at node.js:444:3
at exports._errnoException (util.js:896:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:182:32)
at onErrorNT (internal/child_process.js:348:16)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
at Function.Module.runMain (module.js:577:11)
at startup (node.js:159:18)
at node.js:444:3
weren't you getting this?
hey, that's the error I was getting
;-;
Well then.
Crap.
errrr
:/
:(
:O
it works
yer identifier thingy works
:D
take that, @liam4 !
will this work => 'yes!';
print( will this work );yes!
yey
wait, why does that work
that should be ambigious
nope
hm
lol
lol
it reads it and trims the leading/trailing whitespace
so
( will this work )
You added trimming?
I added your thingy silly
ik
But, yeah, apparently so
but it shouldn't reject it for all leading/trailing whitespace
No, it just trims it?
only the first or last characters
oh
lemme test
Identifier -> GenericValidIdentifierCharacter:+ {% function(data, location, reject) {
var identifier = data[0].join('')
if(C.KEYWORDS.indexOf(identifier) == -1 && !(/[0-9\s]/.test(identifier.charAt(0))) && identifier.charAt(identifier.length - 1) !== ' ') {
return identifier
}
return reject
} %}
this is the Identifier code
change it to this:
{% function(data, location, reject) {
var identifier = data[0].join('')
if(C.KEYWORDS.indexOf(identifier) == -1 && !(/[0-9]/.test(identifier.charAt(0))) && identifier.trim() === identifier) {
return identifier
}
return reject
} %}
still works
go with the latter then
idk why the first even works
:P
whitespace test => true;
whitespace test();
ambiguous
did it work before?
um
maybe?
lol
[[["VARIABLE_ASSIGN","whitespace test",["BOOLEAN_PRIM",true]]],[["FUNCTION_CALL",["VARIABLE_IDENTIFIER","whitespace test"],[]]]]
----------------------------
[[["VARIABLE_IDENTIFIER","whitespace test => true"]],[["FUNCTION_CALL",["VARIABLE_IDENTIFIER","whitespace test"],[]]]]
O_o
hm
apparently it sees the entire first line as a varname
:P
yeah
what did you break
:P
why does it allow for an identifier to have => !?
owait, those were removed from special_chars
¯_(ツ)_/¯
here, you dropped this: \
allExports.KEYWORDS = ['true', 'false', '=>', '->', 'async', 'unevaluated']
Nah, gitter just displays it wrong
yeah, that checks for the whole identifier though
hm
ah
so we can't allow "=>" anywhere?
that'd solve it, yeah
how about required whitespace before
=>
:P
no
liam doesn't want that
kek
edit the ValididentifierChar's or whatever it's calledpost processor
mm
I am
yey
allExports.NOT_ALLOWED_IN_IDENTIFIERS = ['=>', '->']
hows dis
:shipit:
lol
yeah
or just SPECIAL_STRINGS
:P
how about NO
allExports.NO = ['=>', '->']
OMG
:P
this works
hm?
wait
what?
const Scratch = require('./sb2')
Scratch.init()
Scratch.addList("I am a List!", ["Item", 2, 6])
Scratch.addScript(
Scratch.HatScript("whenGreenFlag", null, Scratch.Script("stopScripts", "all"))
)
console.log(Scratch.asJSON())
in my thing
OOOOOO
WHAT
das awesome
yey
it was super easy too :P
put it on the githubz
make a repo for it?
yas
it's barely complicated at all
here'
s the code
call it "toshy"
:P
'use strict'
var Scratch = {}
Scratch.JSON = {}
Scratch.init = function() {
Scratch.JSON.objName = "Stage",
Scratch.JSON.variables = [],
Scratch.JSON.lists = [],
Scratch.JSON.scripts = [],
Scratch.JSON.sounds = [],
Scratch.JSON.costumes = [],
Scratch.JSON.currentCostume = 0,
Scratch.JSON.penLayerMD5 = "5c81a336fab8be57adc039a8a2b33ca9.png",
Scratch.JSON.penLayerID = 0,
Scratch.JSON.tempoBPM = 60,
Scratch.JSON.videoAlpha = 0.5,
Scratch.JSON.children = []
Scratch.JSON.info = {
"projectID": "-1",
"videoOn": false,
"spriteCount": 0,
"swfVersion": "v446",
"hasCloudData": false,
"scriptCount": 0
}
Scratch.addCostume("backdrop1")
}
Scratch.asJSON = function() {
return JSON.stringify(Scratch.JSON)
}
Scratch.addCostume = function(name, md5, x, y) {
if(!name) throw "ERRROR: name must be specified, was " + name + "!"
Scratch.JSON.costumes.push({
"costumeName": name,
"baseLayerID": 1,
"baseLayerMD5": md5 || "739b5e2a2435f6e1ec2993791b423146.png",
"bitmapResolution": 1,
"rotationCenterX": x || 240,
"rotationCenterY": y || 180
})
}
Scratch.addScript = function(script) {
Scratch.JSON.scripts.push(script)
}
Scratch.addList = function(name, contents, x, y, w, h, visible) {
if(!name) throw "ERRROR: name must be specified, was " + name + "!"
Scratch.JSON.lists.push({
"listName": name,
"contents": contents || [],
"isPersistent": false,
"x": x || 0,
"y": y || 0,
"width": w || 100,
"height": h || 200,
"visible": visible || false
})
}
Scratch.addVar = function(name, value, x, y, visible) {
if(!name) throw "ERRROR: name must be specified, was " + name + "!"
Scratch.JSON.variables.push({
"name": name,
"value": value || "",
"isPersistent": false,
"x": x || 0,
"y": y || 0,
"visible": visible || false
})
}
Scratch.HatScript = function(name, args, ...scripts) {
if(!scripts) scripts = [null]
return [0, 0, [args ? Scratch.Script(name, ...args) : Scratch.Script(name), ...scripts]]
}
Scratch.Script = function(name, ...args) {
return args ? [name, ...args] : [name]
}
module.exports = Scratch7
xD
it's incredibly modified to fit simply with the project I'm using it in
throw "ERRROR:
lol
nice spelling
missed that
it has terrible costumes, only the stage, and no sounds
and probably other things I can't think of
lol
new Scratch.comment('lolol')
:P
But it works for what I'm using it for! :P
.....which is?
:shipit:
The language that compiles to .sb2
.sb12
o.
:P
um
A total of 105 ASTs were generated.
Please report this on the official issue tracker:
https://github.com/liam4/tlnccuwagnf/issues
Using first AST.
well
...
lol
erm
all the asts are the same
wut
probably whitespace getting ignored
what was the input code?
whitespace test => true;
whitespace test();
lol
try to fix it yourself
:P
"no"
[[["VARIABLE_ASSIGN","whitespace test",["VARIABLE_IDENTIFIER","true"]]],[["FUNCTION_CALL",["VARIABLE_IDENTIFIER","whitespace test"],[]]]]
ugh
So, that's actually correct at least
It's just the ambiguity that's a problem
mm
ah
I did
|| instead of &&
lol
gg
derp x10
welp, still 105 asts
:P
xD
Y'all better have been working in a separate branch!
you know that String.prototype.trim() works right?
Oh yeah. I guess you'd do this?
var id = data[0].join('')
if (id !== id.trim()) {
return reject
}
return idI think node v6 is broken
Check Travis CI to see if node 6 is broken. If it isn't there and it is on your computer try re-installing node 6 :P
how about required whitespace before
=>no
liam doesn't want that
^^^
Also I need to make it fail on 2+ ASTs so that travis fails
see, that was what I changed it to liam :P
Whoa. Why do the tests look so good? :P https://travis-ci.org/liam4/tlnccuwagnf/jobs/127271661#L568