search for: array_size

...i: Do not free hPackage twice or uninitialized (Coverity). ole32: Read vt into a DWORD, as propvariant->vt is 16 bit (Coverity). Matteo Bruni (2): d3dcompiler: Make types array static const. dxgi: Stub DXGIGetDebugInterface1(). Michael Stefaniuc (113): mmdevapi: Use the ARRAY_SIZE() macro. localspl: Use the ARRAY_SIZE() macro. mapi32: Use the ARRAY_SIZE() macro. krnl386.exe16: Use the ARRAY_SIZE() macro. inetcpl.cpl: Use the ARRAY_SIZE() macro. hhctrl.ocx: Use the ARRAY_SIZE() macro. dpnet: Use the ARRAY_SIZE() macro. dx8vb: Use the...

...Sort WINED3DSIH enum values. winex11: Get rid of unnecessary assignments. dxgi: Add FIXME() for ignored output parameter in dxgi_factory_CreateSwapChainForHwnd(). dxgi: Use DXGI_MAX_SWAP_CHAIN_BUFFERS. dxgi: Forbid multisampling with flip swap effects. dxgi: Introduce ARRAY_SIZE() macro. dxgi/tests: Release IDXGIFactory in test_swapchain_present(). dxgi: Create Vulkan swapchains for D3D12. dxgi: Implement d3d12_swapchain_GetBuffer(). dxgi: Implement d3d12_swapchain_GetCurrentBackBufferIndex(). dxgi: Implement d3d12_swapchain_Present1()....

Hi everyone, Using ARRAY_SIZE improves the code readability. I used coccinelle (I made a change to the array_size.cocci file [1]) to find several places where ARRAY_SIZE could be used instead of other macros or sizeof division. I tried to divide the changes into a patch per subsystem (excepted for staging). If one of the patch...

Using the ARRAY_SIZE macro improves the readability of the code. Also, it is not always useful to use a variable to store this constant calculated at compile time nor to re-invent the ARRAY_SIZE macro. Found with Coccinelle with the following semantic patch: @r depends on (org || report)@ type T; T[] E; position p; @@...

...s: Add KeInitializeTimerEx tests. Andrey Gusev (1): msi: Remove checks for negative value. André Hentschel (4): dbghelp: Map SIMD dwarf registers on ARM64. dbghelp: Recompute location after fixing it. ole32/tests: Don't crash if EnumFormatEtc failed. advpack: Use ARRAY_SIZE() macro. Aurimas Fišeras (3): po: Update Lithuanian translation. po: Update Lithuanian translation. po: Update Lithuanian translation. Austin English (1): winecfg: Recommend against overriding secur32. Dmitry Timoshkov (18): fonts: Add clock images to Wingdings font...

...caling. Alistair Leslie-Hughes (1): winex11: Correct compile without vulkan support. Andrew Eikum (1): winex11.drv: Refcount the vulkan surface window. Andrey Gusev (1): winex11.drv: Move condition to the proper place. André Hentschel (1): ws2_32/tests: Use the available ARRAY_SIZE() macro. Dmitry Timoshkov (6): mstask: Retry opening a .job file only in case of sharing violation. schedsvc: Retry opening a .job file only in case of sharing violation. mstask: Implement ITask::SetWorkItemData(). mstask: Implement ITask::GetWorkItemData(). mstask/te...

...to hint user at broken .NET installation. Mark Jansen (3): ntdll/tests: Tests for RtlIpv6StringToAddressEx. ntdll/tests: Tests for RtlIpv4StringToAddressEx. ntdll/tests: Add tests for RtlIpv6AddressToString and RtlIpv6AddressToStringEx. Mathew Hodson (1): msvcp90: Use the ARRAY_SIZE() macro. Michael Müller (3): browseui: Implement IProgressDialog::SetAnimation. gdi32: Treat lpResults as optional in GetCharacterPlacement. ole32: Support reading VT_BOOL, VT_R8 and VT_I8 into propery storage. Michael Stefaniuc (44): riched20: Avoid an ARRAY_SIZE-like mac...

On Fri, 23 Jun 2023 23:14:39 +0200, Julia Lawall <Julia.Lawall at inria.fr> wrote: > Use array_size to protect against multiplication overflows. > > The changes were done using the following Coccinelle semantic patch: > > // <smpl> > @@ > expression E1, E2; > constant C1, C2; > identifier alloc = {vmalloc,vzalloc}; > @@ > > ( > alloc(C1...

This patch series adds support for preservation of the VM generation ID buffer address in xenstore across save/restore and migrate, and also code to increment the value in all cases except for migration. Patch 1 modifies the guest ro and rw node creation to an open coding style and cleans up some extraneous node creation. Patch 2 modifies creation of the hvmloader key in xenstore and adds

Use ARRAY_SIZE macro, rather than explicitly coding some variant of it yourself. Found with: find -type f -name "*.c" -o -name "*.h" | xargs perl -p -i -e 's/\bsizeof\s*\(\s*(\w+)\s*\)\s*\ /\s*sizeof\s*\(\s*\1\s*\[\s*0\s*\]\s*\) /ARRAY_SIZE(\1)/g' and manual check/verification. Signed...

...CH to registry functions. ws2_32: Ignore a SO_SNDBUF value of 0 on macOS. include: Add IMFSequencerSource interface. api-ms-win-core-libraryloader-l1-2-1: Sync spec file. Andre Heider (4): server: Fix keyboard hardware message regression. Revert "include: Use the ARRAY_SIZE() macro in debug.h.". winegcc: Add -I for the prefix's include dir. winegcc: Fix cross linking on multiarch platforms. Andreas Maier (1): dbghelp: Search for debug files in module path too. Andrew Eikum (1): winebus: Don't override real VID/PID for controllers...

On Sun, Oct 01, 2017 at 03:30:38PM -0400, Jérémy Lefaure wrote: > Hi everyone, > Using ARRAY_SIZE improves the code readability. I used coccinelle (I > made a change to the array_size.cocci file [1]) to find several places > where ARRAY_SIZE could be used instead of other macros or sizeof > division. > > I tried to divide the changes into a patch per subsystem (excepted for >...

...ck); > if (list_empty(&vsock->send_pkt_list)) { > @@ -117,16 +118,17 @@ vhost_transport_do_send_pkt(struct vhost_vsock *vsock, > list_del_init(&pkt->list); > spin_unlock_bh(&vsock->send_pkt_list_lock); > > - head = vhost_get_vq_desc(vq, vq->iov, ARRAY_SIZE(vq->iov), > - &out, &in, NULL, NULL); > - if (head < 0) { > + ret = vhost_get_avail_buf(vq, &buf, > + vq->iov, ARRAY_SIZE(vq->iov), > + &out, &in, NULL, NULL); > + if (ret < 0) { > spin_lock_bh(&vsock->send_pkt_l...

...ck); > if (list_empty(&vsock->send_pkt_list)) { > @@ -117,16 +118,17 @@ vhost_transport_do_send_pkt(struct vhost_vsock *vsock, > list_del_init(&pkt->list); > spin_unlock_bh(&vsock->send_pkt_list_lock); > > - head = vhost_get_vq_desc(vq, vq->iov, ARRAY_SIZE(vq->iov), > - &out, &in, NULL, NULL); > - if (head < 0) { > + ret = vhost_get_avail_buf(vq, &buf, > + vq->iov, ARRAY_SIZE(vq->iov), > + &out, &in, NULL, NULL); > + if (ret < 0) { > spin_lock_bh(&vsock->send_pkt_l...

...Mark Jansen (1): shell32: Expand environment strings in ShellExecute. Matteo Bruni (1): wined3d: Invalidate shader resource bindings when blitting. Michael Müller (1): browseui: Implement PROGDLG_AUTOTIME flag for IProgressDialog. Michael Stefaniuc (19): dinput: Use the ARRAY_SIZE() macro. rpcrt4: Avoid a variable holding an ARRAY_SIZE result. rpcrt4: Use the ARRAY_SIZE() macro. rpcrt4: Use the standard ARRAY_SIZE() macro. winedump: Use the ARRAY_SIZE() macro. server: Use the ARRAY_SIZE() macro. wmc: Use the ARRAY_SIZE() macro. wrc:...

Using the ARRAY_SIZE macro improves the readability of the code. Also, it is useless to re-invent it. Found with Coccinelle with the following semantic patch: @r depends on (org || report)@ type T; T[] E; position p; @@ ( (sizeof(E)@p /sizeof(*E)) | (sizeof(E)@p /sizeof(E[...])) | (sizeof(E)@p /sizeof(T)) ) Review...

...g for. Do you want me to > apply this, or to ACK it so someone else can? If it's sent as a series > I tend to assume the latter. > > But in this case I'm assuming it's the former, so I'll pick up the nfsd > one.... Could you to apply the NFSD patch ("nfsd: use ARRAY_SIZE") with the Reviewed-by: Jeff Layton <jlayton at redhat.com>) tag please ? This patch is an individual patch and it should not have been sent in a series (sorry about that). Thank you, Jérémy

...-864,8 +864,7 @@ __initdata_or_module = unsigned long __init_or_module ia64_native_patch_bundle(void *sbundle, void *ebundle, unsigned long type) { - const unsigned long nelems = sizeof(ia64_native_patch_bundle_elems) / - sizeof(ia64_native_patch_bundle_elems[0]); + const unsigned long nelems = ARRAY_SIZE(ia64_native_patch_bundle_elems); return __paravirt_patch_apply_bundle(sbundle, ebundle, type, ia64_native_patch_bundle_elems, @@ -894,9 +893,8 @@ __initconst = { static void __init ia64_native_patch_branch(unsigned long tag, unsigned long type) { - const unsigned long nelem = -...

...-864,8 +864,7 @@ __initdata_or_module = unsigned long __init_or_module ia64_native_patch_bundle(void *sbundle, void *ebundle, unsigned long type) { - const unsigned long nelems = sizeof(ia64_native_patch_bundle_elems) / - sizeof(ia64_native_patch_bundle_elems[0]); + const unsigned long nelems = ARRAY_SIZE(ia64_native_patch_bundle_elems); return __paravirt_patch_apply_bundle(sbundle, ebundle, type, ia64_native_patch_bundle_elems, @@ -894,9 +893,8 @@ __initconst = { static void __init ia64_native_patch_branch(unsigned long tag, unsigned long type) { - const unsigned long nelem = -...

...veau/nv50_display.c +++ b/drivers/gpu/drm/nouveau/nv50_display.c @@ -3141,7 +3141,7 @@ nv50_mstc_new(struct nv50_mstm *mstm, struct drm_dp_mst_port *port, mstc->connector.funcs->reset(&mstc->connector); nouveau_conn_attach_properties(&mstc->connector); - for (i = 0; i < ARRAY_SIZE(mstm->msto) && mstm->msto; i++) + for (i = 0; i < ARRAY_SIZE(mstm->msto) && mstm->msto[i]; i++) drm_mode_connector_attach_encoder(&mstc->connector, &mstm->msto[i]->encoder); drm_object_attach_property(&mstc->connector.base, dev->mode_...