Mercury-IM/Smack
Mercury-IM/Smack
1
0
Fork 0
mirror of https://codeberg.org/Mercury-IM/Smack synced 2024-11-26 00:02:06 +01:00
Code Issues Releases Wiki Activity

LazyStringBuilder: Wrap NPE in RuntimeException

Browse source 
instead of using a Logger.
This commit is contained in:
Florian Schmaus 2019-09-18 08:58:51 +02:00
parent 8086a11c6c
commit 5483e95920
1 changed files with 2 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
smack-core/src/main/java/org/jivesoftware/smack/util/LazyStringBuilder.java
View file

@ -1,6 +1,6 @@
/** /**
* *
* Copyright 2014-2017 Florian Schmaus * Copyright 2014-2019 Florian Schmaus
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -19,13 +19,9 @@ package org.jivesoftware.smack.util;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.Collections; import java.util.Collections;
import java.util.List; import java.util.List;
import java.util.logging.Level;
import java.util.logging.Logger;
public class LazyStringBuilder implements Appendable, CharSequence { public class LazyStringBuilder implements Appendable, CharSequence {
private static final Logger LOGGER = Logger.getLogger(LazyStringBuilder.class.getName());
private final List<CharSequence> list; private final List<CharSequence> list;
private String cache; private String cache;
@ -80,8 +76,7 @@ public class LazyStringBuilder implements Appendable, CharSequence {
} }
catch (NullPointerException npe) { catch (NullPointerException npe) {
StringBuilder sb = safeToStringBuilder(); StringBuilder sb = safeToStringBuilder();
LOGGER.log(Level.SEVERE, "The following LazyStringBuilder threw a NullPointerException: " + sb, npe); throw new RuntimeException("The following LazyStringBuilder threw a NullPointerException: " + sb, npe);
throw npe;
} }
return length; return length;
} }