Swastik Baranwal comments

Results 268 comments of


                                            Swastik Baranwal

cgen: checker bug; node.left_types.len < node.left.len

> @Delta456, can you fix it? Yes, I will fix this issue. Thanks for enlightening me on this.

hostname loginname functions

First run `v fmt -w` as said: ``` Failed command 1: '/home/runner/work/v/v/v' fmt -verify '/home/runner/work/v/v/vlib/os/os_test.v' Failed command 2: '/home/runner/work/v/v/v' fmt -verify '/home/runner/work/v/v/vlib/os/os_windows.c.v' ```

Option type comparisons are permitted by V compiler.

I am not sure if comparison operators should work on options.

C error with sqlite

I think this should be a V error because you shouldn't index queries like that.

Confusing errors when printing a C struct

On `x84-64` ``` ================== /tmp/v_1000/file.13233380909043727445.tmp.c:12655: error: too many initializers ... ================== (Use `v -cg` to print the entire error message) builder error: ================== C error. This should never happen. This...

feat: $if (int(typeof(val).idx) >> 16) & 0xff == 0 {

Does this not work?

feat: $if (int(typeof(val).idx) >> 16) & 0xff == 0 {

It compiles correctly for me. ![image](https://github.com/vlang/v/assets/28479139/f2ad313d-ce10-4651-afd0-e77486fa86b5)

feat: $if (int(typeof(val).idx) >> 16) & 0xff == 0 {

You ain't on the latest commit ``` V full version: V 0.4.3 e2334d8.c4541cb OS: linux, Pop!_OS 22.04 LTS Processor: 12 cpus, 64bit, little endian, AMD Ryzen 5 5600H with Radeon...

This code is in time out

Yes, you need to update again.

checker.orm: check if update expr is invalid ident

This PR may change according to what the syntax is decided for `set`.