Skip to content

Commit

Permalink
Updated OutputSettings inside of Document to be a static inner class.
Browse files Browse the repository at this point in the history
This addresses the issue with using jsoup and scala 2.8 discussed here:
http://groups.google.com/group/jsoup/browse_thread/thread/3f7ec2fa41dfb87f

This change should be safe to commit and won't make a backwards incompatible
to the public interface of jsoup (you can always reference a static member
via a non-static path) for any existing users.  A recompilation of their code
won't even be necessary.

All tests continue to pass for me after this change.
  • Loading branch information
bbeck committed Dec 22, 2010
1 parent 5824e29 commit 9a2e19e
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/main/java/org/jsoup/nodes/Document.java
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ public Document clone() {
/**
* A Document's output settings control the form of the text() and html() methods.
*/
public class OutputSettings implements Cloneable {
public static class OutputSettings implements Cloneable {
private Entities.EscapeMode escapeMode = Entities.EscapeMode.base;
private Charset charset = Charset.forName("UTF-8");
private CharsetEncoder charsetEncoder = charset.newEncoder();
Expand Down

0 comments on commit 9a2e19e

Please sign in to comment.