Fix style.

Author: fangerer

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/BuiltinConstructors.java

@@ -1815,10 +1815,10 @@ Object strOneArg(VirtualFrame frame, LazyPythonClass strClass, Object obj, @Supp
             return result;
         }
 
-        @Specialization(guards = {"!isNativeClass(strClass)", "!isNoValue(encoding)"}, limit = "3" )
+        @Specialization(guards = {"!isNativeClass(strClass)", "!isNoValue(encoding)"}, limit = "3")
         Object doBuffer(VirtualFrame frame, LazyPythonClass strClass, Object obj, Object encoding, Object errors,
                         @CachedLibrary("obj") PythonObjectLibrary bufferLib) {
-            if(bufferLib.isBuffer(obj)) {
+            if (bufferLib.isBuffer(obj)) {
                 try {
                     // TODO(fa): we should directly call '_codecs.decode'
                     PBytes bytesObj = factory().createBytes(bufferLib.getBufferBytes(obj));
@@ -1847,10 +1847,9 @@ private Object decodeBytes(VirtualFrame frame, LazyPythonClass strClass, PBytes
          * into a natively allocated subtype structure
          */
         @Specialization(guards = {"isSubtypeOfString(frame, isSubtype, cls)", "isNoValue(encoding)", "isNoValue(errors)"}, limit = "1")
-        Object doNativeSubclass(VirtualFrame frame, PythonNativeClass cls, Object obj, @SuppressWarnings("unused") Object encoding,@SuppressWarnings("unused")  Object errors,
+        Object doNativeSubclass(VirtualFrame frame, PythonNativeClass cls, Object obj, @SuppressWarnings("unused") Object encoding, @SuppressWarnings("unused") Object errors,
                         @Cached @SuppressWarnings("unused") IsSubtypeNode isSubtype,
                         @Cached CoerceToStringNode castToStringNode,
-                        @Cached BranchProfile gotException,
                         @Cached CExtNodes.StringSubtypeNew subtypeNew) {
             return subtypeNew.call(cls, castToStringNode.execute(frame, obj));
         }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/slice/PSlice.java

@@ -118,6 +118,7 @@ public SliceInfo(int start, int stop, int step, int length) {
             this.length = length;
         }
     }
+
     public SliceInfo computeIndices(int length) {
         return PSlice.computeIndices(start, stop, step, length);
 

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/str/StringNodes.java

@@ -1,3 +1,43 @@
+/*
+ * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * The Universal Permissive License (UPL), Version 1.0
+ *
+ * Subject to the condition set forth below, permission is hereby granted to any
+ * person obtaining a copy of this software, associated documentation and/or
+ * data (collectively the "Software"), free of charge and under any and all
+ * copyright rights in the Software, and any and all patent rights owned or
+ * freely licensable by each licensor hereunder covering either (i) the
+ * unmodified Software as contributed to or provided by such licensor, or (ii)
+ * the Larger Works (as defined below), to deal in both
+ *
+ * (a) the Software, and
+ *
+ * (b) any piece of software and/or hardware listed in the lrgrwrks.txt file if
+ * one is included with the Software each a "Larger Work" to which the Software
+ * is contributed by such licensors),
+ *
+ * without restriction, including without limitation the rights to copy, create
+ * derivative works of, display, perform, and distribute the Software and make,
+ * use, sell, offer for sale, import, export, have made, and have sold the
+ * Software and the Larger Work(s), and to sublicense the foregoing rights on
+ * either these or other terms.
+ *
+ * This license is subject to the following condition:
+ *
+ * The above copyright notice and either this complete permission notice or at a
+ * minimum a reference to the UPL must be included in all copies or substantial
+ * portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ */
 package com.oracle.graal.python.builtins.objects.str;
 
 import static com.oracle.graal.python.runtime.exception.PythonErrorType.TypeError;
@@ -52,7 +92,6 @@ public static boolean isMaterialized(PString x) {
         return x.getCharSequence() instanceof String;
     }
 
-
     @GenerateUncached
     @ImportStatic(StringNodes.class)
     public abstract static class StringMaterializeNode extends Node {

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/statement/ImportStarNode.java

@@ -44,7 +44,6 @@
 import com.oracle.graal.python.nodes.util.CastToIndexNode;
 import com.oracle.graal.python.nodes.util.CastToJavaStringNode;
 import com.oracle.graal.python.nodes.util.CastToJavaStringNodeGen;
-import com.oracle.graal.python.parser.sst.SimpleSSTNode.Type;
 import com.oracle.graal.python.runtime.exception.PException;
 import com.oracle.truffle.api.CompilerDirectives;
 import com.oracle.truffle.api.CompilerDirectives.CompilationFinal;
@@ -131,8 +130,9 @@ public void executeVoid(VirtualFrame frame) {
                 for (int i = 0; i < n; i++) {
                     Object attrNameObj = ensureGetItemNode().executeWith(frame, attrAll, i);
                     String attrName = ensureCastToStringNode().execute(attrNameObj);
-                    if(attrName == null) {
-                        // TODO(fa): this error should be raised by the ReadAttributeFromObjectNode; but that needs some refactoring first.
+                    if (attrName == null) {
+                        // TODO(fa): this error should be raised by the ReadAttributeFromObjectNode;
+                        // but that needs some refactoring first.
                         throw raise(PythonBuiltinClassType.TypeError, "attribute name must be string, not '%p'", attrNameObj);
                     }
                     Object attr = readAttribute(frame, importedModule, attrName);