Skip to content

Commit ced71df

Browse files
aayush0325aayush0325
committed
chore: stuff from code review
1 parent f78fa4a commit ced71df

File tree

7 files changed

+64
-62
lines changed

7 files changed

+64
-62
lines changed

‎lib/node_modules/@stdlib/math/base/special/lcmf/README.md‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ v = lcmf( 48, NaN );
102102

103103
```javascript
104104
var randu = require( '@stdlib/random/base/randu' );
105-
var round = require( '@stdlib/math/base/special/round' );
105+
var roundf = require( '@stdlib/math/base/special/roundf' );
106106
var lcmf = require( '@stdlib/math/base/special/lcmf' );
107107

108108
var a;
@@ -111,8 +111,8 @@ var v;
111111
var i;
112112

113113
for ( i = 0; i < 100; i++ ) {
114-
a = round( randu()*50 );
115-
b = round( randu()*50 );
114+
a = roundf( randu() * 50 );
115+
b = roundf( randu() * 50 );
116116
v = lcmf( a, b );
117117
console.log( 'lcmf(%d,%d) = %d', a, b, v );
118118
}
@@ -153,7 +153,7 @@ for ( i = 0; i < 100; i++ ) {
153153
Computes the [least common multiple][lcm] (lcm) of two single-precision floating-point numbers.
154154

155155
```c
156-
double v = stdlib_base_lcmf( 48.0f, 18.0f );
156+
float v = stdlib_base_lcmf( 48.0f, 18.0f );
157157
// returns 144.0f
158158
```
159159

‎lib/node_modules/@stdlib/math/base/special/lcmf/benchmark/c/native/benchmark.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
#include <time.h>
2424
#include <sys/time.h>
2525

26-
#define NAME "lcm"
26+
#define NAME "lcmf"
2727
#define ITERATIONS 1000000
2828
#define REPEATS 3
2929

‎lib/node_modules/@stdlib/math/base/special/lcmf/examples/index.js‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
'use strict';
2020

2121
var randu = require( '@stdlib/random/base/randu' );
22-
var round = require( '@stdlib/math/base/special/round' );
22+
var roundf = require( '@stdlib/math/base/special/roundf' );
2323
var lcmf = require( './../lib' );
2424

2525
var a;
@@ -28,8 +28,8 @@ var v;
2828
var i;
2929

3030
for ( i = 0; i < 100; i++ ) {
31-
a = round( randu() * 50 );
32-
b = round( randu() * 50 );
31+
a = roundf( randu() * 50 );
32+
b = roundf( randu() * 50 );
3333
v = lcmf( a, b );
3434
console.log( 'lcmf(%d,%d) = %d', a, b, v );
3535
}

‎lib/node_modules/@stdlib/math/base/special/lcmf/lib/main.js‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,12 +56,13 @@ function lcmf( a, b ) {
5656
if ( b < 0 ) {
5757
b = -b;
5858
}
59+
5960
// Note: we rely on `gcdf` to perform further argument validation...
6061
d = gcdf( a, b );
6162
if ( isnanf( d ) ) {
6263
return d;
6364
}
64-
return (a/d) * b;
65+
return ( a / d ) * b;
6566
}
6667

6768

‎lib/node_modules/@stdlib/math/base/special/lcmf/src/main.c‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
* @return least common multiple
2929
*
3030
* @example
31-
* double out = stdlib_base_lcmf( 21.0f, 6.0f );
31+
* float out = stdlib_base_lcmf( 21.0f, 6.0f );
3232
* // returns 42.0f
3333
*/
3434
float stdlib_base_lcmf( const float a, const float b ) {
@@ -49,6 +49,7 @@ float stdlib_base_lcmf( const float a, const float b ) {
4949
} else {
5050
bn = b;
5151
}
52+
5253
// Note: we rely on `gcdf` to perform further argument validation...
5354
d = stdlib_base_gcdf( an, bn );
5455
if ( stdlib_base_is_nanf( d ) ) {

‎lib/node_modules/@stdlib/math/base/special/lcmf/test/test.js‎

Lines changed: 26 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -39,13 +39,13 @@ tape( 'the function returns `NaN` if either argument is `NaN`', function test( t
3939
var v;
4040

4141
v = lcmf( NaN, 10 );
42-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
42+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
4343

4444
v = lcmf( 10, NaN );
45-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
45+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
4646

4747
v = lcmf( NaN, NaN );
48-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
48+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
4949

5050
t.end();
5151
});
@@ -54,13 +54,13 @@ tape( 'the function returns `NaN` if either argument is `+infinity`', function t
5454
var v;
5555

5656
v = lcmf( PINF, 10 );
57-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
57+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
5858

5959
v = lcmf( 10, PINF );
60-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
60+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
6161

6262
v = lcmf( PINF, PINF );
63-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
63+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
6464

6565
t.end();
6666
});
@@ -69,13 +69,13 @@ tape( 'the function returns `NaN` if either argument is `-infinity`', function t
6969
var v;
7070

7171
v = lcmf( NINF, 10 );
72-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
72+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
7373

7474
v = lcmf( 10, NINF );
75-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
75+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
7676

7777
v = lcmf( NINF, NINF );
78-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
78+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
7979

8080
t.end();
8181
});
@@ -84,26 +84,26 @@ tape( 'the function returns `NaN` if either argument is not an integer value', f
8484
var v;
8585

8686
v = lcmf( 3.14, 10 );
87-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
87+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
8888

8989
v = lcmf( 10, 3.14 );
90-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
90+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
9191

9292
v = lcmf( 3.14, 6.18 );
93-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
93+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
9494

9595
t.end();
9696
});
9797

9898
tape( 'the function returns `0` if either argument is `0`', function test( t ) {
9999
var v = lcmf( 0, 0 );
100-
t.strictEqual( v, 0, 'returns 0' );
100+
t.strictEqual( v, 0, 'returns expected value' );
101101

102102
v = lcmf( 2, 0 );
103-
t.strictEqual( v, 0, 'returns 0' );
103+
t.strictEqual( v, 0, 'returns expected value' );
104104

105105
v = lcmf( 0, -3 );
106-
t.strictEqual( v, 0, 'returns 0' );
106+
t.strictEqual( v, 0, 'returns expected value' );
107107

108108
t.end();
109109
});
@@ -112,37 +112,37 @@ tape( 'the function computes the least common multiple', function test( t ) {
112112
var v;
113113

114114
v = lcmf( 0, 0 );
115-
t.strictEqual( v, 0, 'returns 0' );
115+
t.strictEqual( v, 0, 'returns expected value' );
116116

117117
v = lcmf( 1, 0 );
118-
t.strictEqual( v, 0, 'returns 0' );
118+
t.strictEqual( v, 0, 'returns expected value' );
119119

120120
v = lcmf( 0, 1 );
121-
t.strictEqual( v, 0, 'returns 0' );
121+
t.strictEqual( v, 0, 'returns expected value' );
122122

123123
v = lcmf( 6, 4 );
124-
t.strictEqual( v, 12, 'returns 12' );
124+
t.strictEqual( v, 12, 'returns expected value' );
125125

126126
v = lcmf( 6, -4 );
127-
t.strictEqual( v, 12, 'returns 12' );
127+
t.strictEqual( v, 12, 'returns expected value' );
128128

129129
v = lcmf( -6, -4 );
130-
t.strictEqual( v, 12, 'returns 12' );
130+
t.strictEqual( v, 12, 'returns expected value' );
131131

132132
v = lcmf( 2, 8 );
133-
t.strictEqual( v, 8, 'returns 8' );
133+
t.strictEqual( v, 8, 'returns expected value' );
134134

135135
v = lcmf( 15, 20 );
136-
t.strictEqual( v, 60, 'returns 60' );
136+
t.strictEqual( v, 60, 'returns expected value' );
137137

138138
v = lcmf( 20, 15 );
139-
t.strictEqual( v, 60, 'returns 60' );
139+
t.strictEqual( v, 60, 'returns expected value' );
140140

141141
v = lcmf( 35, -21 );
142-
t.strictEqual( v, 105, 'returns 105' );
142+
t.strictEqual( v, 105, 'returns expected value' );
143143

144144
v = lcmf( 48, 18 );
145-
t.strictEqual( v, 144, 'returns 144' );
145+
t.strictEqual( v, 144, 'returns expected value' );
146146

147147
t.end();
148148
});

‎lib/node_modules/@stdlib/math/base/special/lcmf/test/test.native.js‎

Lines changed: 26 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -48,13 +48,13 @@ tape( 'the function returns `NaN` if either argument is `NaN`', opts, function t
4848
var v;
4949

5050
v = lcmf( NaN, 10 );
51-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
51+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
5252

5353
v = lcmf( 10, NaN );
54-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
54+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
5555

5656
v = lcmf( NaN, NaN );
57-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
57+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
5858

5959
t.end();
6060
});
@@ -63,13 +63,13 @@ tape( 'the function returns `NaN` if either argument is `+infinity`', opts, func
6363
var v;
6464

6565
v = lcmf( PINF, 10 );
66-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
66+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
6767

6868
v = lcmf( 10, PINF );
69-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
69+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
7070

7171
v = lcmf( PINF, PINF );
72-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
72+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
7373

7474
t.end();
7575
});
@@ -78,13 +78,13 @@ tape( 'the function returns `NaN` if either argument is `-infinity`', opts, func
7878
var v;
7979

8080
v = lcmf( NINF, 10 );
81-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
81+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
8282

8383
v = lcmf( 10, NINF );
84-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
84+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
8585

8686
v = lcmf( NINF, NINF );
87-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
87+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
8888

8989
t.end();
9090
});
@@ -93,26 +93,26 @@ tape( 'the function returns `NaN` if either argument is not an integer value', o
9393
var v;
9494

9595
v = lcmf( 3.14, 10 );
96-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
96+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
9797

9898
v = lcmf( 10, 3.14 );
99-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
99+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
100100

101101
v = lcmf( 3.14, 6.18 );
102-
t.strictEqual( isnanf( v ), true, 'returns NaN' );
102+
t.strictEqual( isnanf( v ), true, 'returns expected value' );
103103

104104
t.end();
105105
});
106106

107107
tape( 'the function returns `0` if either argument is `0`', opts, function test( t ) {
108108
var v = lcmf( 0, 0 );
109-
t.strictEqual( v, 0, 'returns 0' );
109+
t.strictEqual( v, 0, 'returns expected value' );
110110

111111
v = lcmf( 2, 0 );
112-
t.strictEqual( v, 0, 'returns 0' );
112+
t.strictEqual( v, 0, 'returns expected value' );
113113

114114
v = lcmf( 0, -3 );
115-
t.strictEqual( v, 0, 'returns 0' );
115+
t.strictEqual( v, 0, 'returns expected value' );
116116

117117
t.end();
118118
});
@@ -121,37 +121,37 @@ tape( 'the function computes the least common multiple', opts, function test( t
121121
var v;
122122

123123
v = lcmf( 0, 0 );
124-
t.strictEqual( v, 0, 'returns 0' );
124+
t.strictEqual( v, 0, 'returns expected value' );
125125

126126
v = lcmf( 1, 0 );
127-
t.strictEqual( v, 0, 'returns 0' );
127+
t.strictEqual( v, 0, 'returns expected value' );
128128

129129
v = lcmf( 0, 1 );
130-
t.strictEqual( v, 0, 'returns 0' );
130+
t.strictEqual( v, 0, 'returns expected value' );
131131

132132
v = lcmf( 6, 4 );
133-
t.strictEqual( v, 12, 'returns 12' );
133+
t.strictEqual( v, 12, 'returns expected value' );
134134

135135
v = lcmf( 6, -4 );
136-
t.strictEqual( v, 12, 'returns 12' );
136+
t.strictEqual( v, 12, 'returns expected value' );
137137

138138
v = lcmf( -6, -4 );
139-
t.strictEqual( v, 12, 'returns 12' );
139+
t.strictEqual( v, 12, 'returns expected value' );
140140

141141
v = lcmf( 2, 8 );
142-
t.strictEqual( v, 8, 'returns 8' );
142+
t.strictEqual( v, 8, 'returns expected value' );
143143

144144
v = lcmf( 15, 20 );
145-
t.strictEqual( v, 60, 'returns 60' );
145+
t.strictEqual( v, 60, 'returns expected value' );
146146

147147
v = lcmf( 20, 15 );
148-
t.strictEqual( v, 60, 'returns 60' );
148+
t.strictEqual( v, 60, 'returns expected value' );
149149

150150
v = lcmf( 35, -21 );
151-
t.strictEqual( v, 105, 'returns 105' );
151+
t.strictEqual( v, 105, 'returns expected value' );
152152

153153
v = lcmf( 48, 18 );
154-
t.strictEqual( v, 144, 'returns 144' );
154+
t.strictEqual( v, 144, 'returns expected value' );
155155

156156
t.end();
157157
});

0 commit comments

Comments
 (0)