JalonSolov

Results 488 comments of JalonSolov

Actually, it appears this is working correctly, just not intuitively. The `rand.ascii()` function is generating the correct number of values, they are just not all **printable** values. Trying to print...

`set_cookie` should follow the official rules: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Set-Cookie#attributes If you try to set a cookie that doesn't follow the rules, it should return an error. Let's leave this open until that...

Can you post a link to the complete V source? Or create a minimal example and paste it here? Without a complete compilable example, it is extremely hard to reproduce...

With `V 0.3.3 6e1e406` I get the following: ``` $ v -g foo.v foo.v:32:2: warning: unused variable: `data` 30 | 31 | pub fn (mut node Node) save() ! {...

The problem is that it failed the sanitize-address tests. That generally points to memory leaks, array access problems, etc. Those need to be addressed, first.

Hmm... is it better to have `startup_msg` as boolean? What if you want to have a custom startup message?

To be _truly_ explicit (and even more verbose), it should be `show_default_startup_message` - since it only affects the default.

I couldn't find module `jwt` so I deleted that import. The code compiles cleanly and runs, but does not run as you expected. Using a minimal example: ```v import x.json2...

All those warnings are Windows specific. On Linux, I only get ``` /tmp/v_1000/../../../../../../home/jalon/test_file.v:2: error: 'stat' undeclared builder error: ================== ```

The problem comes if you declare the array len to be larger than the array type supports ```v arr := []u8{len: 10000, init: it} ``` Should V silently "wrap" the...