Skip to content

Commit d4dc759

Browse files
committed
Make NumberBuilder no longer Serializable
NumberBuilder holds onto a CharSequence rather than a specific concrete implementation (like String). CharSequence is not Serializable, but it needs to be if I want NumberBuilder to Serialize properly. In theory I could make it Externalizable but I don't see the need so for now I'm just removing Serializable from the declaration.
1 parent 25fb4f5 commit d4dc759

File tree

1 file changed

+1
-7
lines changed

1 file changed

+1
-7
lines changed

module/jsonurl-core/src/main/java/org/jsonurl/NumberBuilder.java

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
import static org.jsonurl.CharUtil.digits;
2121
import static org.jsonurl.CharUtil.isDigit;
2222

23-
import java.io.Serializable;
2423
import java.math.BigDecimal;
2524
import java.math.BigInteger;
2625

@@ -34,12 +33,7 @@
3433
* @author David MacCormack
3534
* @since 2019-09-01
3635
*/
37-
public class NumberBuilder implements NumberText, Serializable {
38-
39-
/**
40-
* serialVersionUID.
41-
*/
42-
private static final long serialVersionUID = 1L;
36+
public class NumberBuilder implements NumberText {
4337

4438
/**
4539
* The maximum number of digits I consider when parsing a

0 commit comments

Comments
 (0)