Commit b89790c
refactor(crypto): remove unused web3j-era helpers from WalletUtils
The following methods had no production callers after the keystore
refactor — the only reference was a coverage-padding test at
SupplementTest.testGet (no assertions, just "call these to bump the
coverage number") plus internal self-references:
- generateFullNewWalletFile(String, File, boolean)
- generateLightNewWalletFile(String, File, boolean)
- generateNewWalletFile(String, File, boolean, boolean)
- getDefaultKeyDirectory() / getDefaultKeyDirectory(String)
- getTestnetKeyDirectory()
- getMainnetKeyDirectory()
The three generate* wrappers were identical calls into the real
generateWalletFile(String, SignInterface, File, boolean) (which is kept
and actively used). The three getDirectory helpers returned Ethereum-
style paths (~/Library/Ethereum, %APPDATA%/Ethereum, ~/.ethereum) —
hard-coded from the web3j origin of this code, never applicable in a
TRON context, and capable of misleading anyone who tried to use them.
Also drops the corresponding five noop invocations from
SupplementTest.testGet and its now-unused WalletUtils import (the
static import of passwordValid elsewhere in the file is preserved —
it's still exercised by testPasswordValid).
No production behavior change. Reduces the crypto module's public API
surface and keeps the keystore library focused on what TRON actually
uses.1 parent fc6ab92 commit b89790c
2 files changed
Lines changed: 0 additions & 64 deletions
File tree
- crypto/src/main/java/org/tron/keystore
- framework/src/test/java/org/tron/program
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
15 | | - | |
16 | | - | |
17 | | - | |
18 | 15 | | |
19 | 16 | | |
20 | 17 | | |
| |||
24 | 21 | | |
25 | 22 | | |
26 | 23 | | |
27 | | - | |
28 | | - | |
29 | 24 | | |
30 | 25 | | |
31 | 26 | | |
| |||
44 | 39 | | |
45 | 40 | | |
46 | 41 | | |
47 | | - | |
48 | | - | |
49 | | - | |
50 | | - | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | | - | |
55 | | - | |
56 | | - | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | | - | |
61 | | - | |
62 | | - | |
63 | | - | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | | - | |
72 | 42 | | |
73 | 43 | | |
74 | 44 | | |
| |||
159 | 129 | | |
160 | 130 | | |
161 | 131 | | |
162 | | - | |
163 | | - | |
164 | | - | |
165 | | - | |
166 | | - | |
167 | | - | |
168 | | - | |
169 | | - | |
170 | | - | |
171 | | - | |
172 | | - | |
173 | | - | |
174 | | - | |
175 | | - | |
176 | | - | |
177 | | - | |
178 | | - | |
179 | | - | |
180 | | - | |
181 | | - | |
182 | | - | |
183 | | - | |
184 | | - | |
185 | | - | |
186 | | - | |
187 | | - | |
188 | | - | |
189 | 132 | | |
190 | 133 | | |
191 | 134 | | |
| |||
Lines changed: 0 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
30 | | - | |
31 | 30 | | |
32 | 31 | | |
33 | 32 | | |
| |||
54 | 53 | | |
55 | 54 | | |
56 | 55 | | |
57 | | - | |
58 | | - | |
59 | | - | |
60 | | - | |
61 | | - | |
62 | | - | |
63 | 56 | | |
64 | 57 | | |
65 | 58 | | |
| |||
0 commit comments