Delyan Angelov
Delyan Angelov
Good work, however, it would be breaking to existing code: ```v o_test.v:13:18: error: only `$tmpl()`, `$env()`, `$embed_file()`, `$pkgconfig()`, `$vweb.html()`, `$compile_error()` and `$compile_warn()` comptime functions are supported right now 11 |...
@medvednikov do we need to support migration of the older code written with `$Array` to `$array` ?
> It is simple enough to return error('No results') and check for that specifically. No, it is not simple, and creates more problems on several levels. SQL returns result *sets*,...
> But I also, think that we need to discuss a pattern to solve this kind of problem. I feel that this PR will solve a important problem creating another...
`error('No results')` != `[]` , that is the difference. 0 results is *NOT* an error. It is a valid result set.
> Just curious. Why `match` expression with Option type only checks against `none`? Is matching its value and checking against `none` at the same time a bad idea? Why? Because...
> Hm I thought `?Node` would imply `?&Node`. I think, that it is still too early (opened bugs concerning option values), to change the representation of `Option`, which will be...
This needs rebasing over master.
> `startup_message` sounds like a string, it should be `show_startup_message: false` indeed, my bad
What was the performance impact of the `[inline]` tag? Did it help say `./vprod -o x.c cmd/v` vs the version on master?