mirror/v
1
0
Fork 0
mirror of https://github.com/vlang/v.git synced 2023-08-10 21:13:21 +03:00
Code Issues Projects Releases Wiki Activity

checker: apply the new array check only to len:, not cap: inits

Browse Source
This commit is contained in:
Alexander Medvednikov 2022-07-06 07:03:36 +03:00
parent fab5802deb
commit d3090de02e
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
vlib/v/checker/containers.v
View File

@ -58,9 +58,8 @@ pub fn (mut c Checker) array_init(mut node ast.ArrayInit) ast.Type {
} }
// &int{} check // &int{} check
if node.elem_type.is_any_kind_of_pointer() && !c.inside_unsafe if node.elem_type.is_any_kind_of_pointer() && !c.inside_unsafe && node.has_len {
&& (node.has_len || node.has_cap) { c.warn('arrays of references need to be initialized right away, therefore `len:` cannot be used (unless inside `unsafe`)',
c.warn('arrays of references need to be initialized right away (unless used inside `unsafe`)',
node.pos) node.pos)
} }
return node.typ return node.typ

2
vlib/v/checker/tests/ptr_array_init.out
View File

@ -1,4 +1,4 @@
vlib/v/checker/tests/ptr_array_init.vv:2:14: warning: arrays of references need to be initialized right away (unless used inside `unsafe`) vlib/v/checker/tests/ptr_array_init.vv:2:14: warning: arrays of references need to be initialized right away, therefore `len:` cannot be used (unless inside `unsafe`)
1 | fn main() { 1 | fn main() {
2 | println(*[]&int{len: 1}[0]) 2 | println(*[]&int{len: 1}[0])
| ~~~~~~~ | ~~~~~~~