index.js ➔ swap - Code Metrics - Inspection of "renamed byteSwap to swap" - rochars/endianness - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 1b2938...d40a74 )

by Rafael S.

created 2017-12-01 16:46 UTC

index.js ➔ swap A

↳ Parent: index.js

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
nc	2
nop	3
dl	0
loc	13
rs	9.4285
c	0
b	0
f	0

/*!
 * endianness
 * Swap endianness in byte arrays.
 * Copyright (c) 2017 Rafael da Silva Rocha.
 * https://github.com/rochars/endianness
 *
 */

/**
 * Swap the endianness of units of information in a byte array.
 * The original array is modified in-place.
 * @param {!Array<number>|!Array<string>|Uint8Array} bytes The bytes.
 * @param {number} offset The number of bytes of each unit of information.
 */
function endianness(bytes, offset) {
    let len = bytes.length;
    let i = 0;
    while (i < len) {
        swap(bytes, offset, i);
        i += offset;
    }
}

/**
 * Swap the endianness of a unit of information in a byte array.
 * The original array is modified in-place.
 * @param {!Array<number>|!Array<string>|Uint8Array} bytes The bytes.
 * @param {number} offset The number of bytes of the unit of information.
 * @param {number} index The start index of the unit of information.
 */
function swap(bytes, offset, index) {
    let x = 0;
    let y = offset - 1;
    let limit = parseInt(offset / 2, 10);
    let swap;

    while(x < limit) {
        swap = bytes[index + x];

        bytes[index + x] = bytes[index + y];
        bytes[index + y] = swap;
        x++;
        y--;
    }
}

module.exports = endianness;


1			/*!
2			* endianness
3			* Swap endianness in byte arrays.
4			* Copyright (c) 2017 Rafael da Silva Rocha.
5			* https://github.com/rochars/endianness
6			*
7			*/
8
9			/**
10			* Swap the endianness of units of information in a byte array.
11			* The original array is modified in-place.
12			* @param {!Array<number>\|!Array<string>\|Uint8Array} bytes The bytes.
13			* @param {number} offset The number of bytes of each unit of information.
14			*/
15			function endianness(bytes, offset) {
16			let len = bytes.length;
17			let i = 0;
18			while (i < len) {
19			swap(bytes, offset, i);
20			i += offset;
21			}
22			}
23
24			/**
25			* Swap the endianness of a unit of information in a byte array.
26			* The original array is modified in-place.
27			* @param {!Array<number>\|!Array<string>\|Uint8Array} bytes The bytes.
28			* @param {number} offset The number of bytes of the unit of information.
29			* @param {number} index The start index of the unit of information.
30			*/
31			function swap(bytes, offset, index) {
32			let x = 0;
33			let y = offset - 1;
34			let limit = parseInt(offset / 2, 10);
35			let swap;
			0 ignored issues – show Unused Code introduced 2017-12-01 16:47 UTC by Report Bug Copy Issue Report The variable `swap` seems to be never used. Consider removing it. Loading history...
36			while(x < limit) {
37			swap = bytes[index + x];
			0 ignored issues – show Comprehensibility introduced 2017-12-01 16:47 UTC by Report Bug Copy Issue Report It seems like you are trying to overwrite a function name here. `swap` is already defined in line 31 as a function. While this will work, it can be very confusing. Loading history...
38			bytes[index + x] = bytes[index + y];
39			bytes[index + y] = swap;
40			x++;
41			y--;
42			}
43			}
44
45			module.exports = endianness;
46

rochars / endianness

Push — master ( 1b2938...d40a74 )

index.js ➔ swap A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like