<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class FaunusTitanGraph extends StandardTitanGraph {
 }

 public FaunusVertex readFaunusVertex ( final Pair < ByteBuffer , SortedMap < ByteBuffer , IColumn > > row ) {
 - FaunusVertexRelationLoader loader = new FaunusVertexRelationLoader ( row . left . duplicate ( ) ) ;
 + FaunusVertexRelationLoader loader = new FaunusVertexRelationLoader ( IDHandler . getKeyID ( row . left . duplicate ( ) ) ) ;
 loadRelations ( new CassandraMapIterable ( row . right ) , loader , tx ) ;
 return loader . getVertex ( ) ;
 }


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;

 import NAMESPACE ;

 <nb> public interface TestRunConfigurationReRunResponsible {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 - @ notnull
 + @ nullable
 RunProfileState rerunTests ( @ notnull final Executor executor , @ notnull final ExecutionEnvironment environment ,
 @ notnull Collection < PsiElement > failedTests ) throws ExecutionException ;
 }


<nb> public class AddColumnChangeTest extends AbstractChangeTest {
 ColumnConfig column = new ColumnConfig ( ) ;
 column . setName ( STRING0 ) ;
 column . setType ( STRING1 ) ;
 +
 + ConstraintsConfig constraints = new ConstraintsConfig ( ) ;
 + constraints . setPrimaryKey ( Boolean . FALSE ) ;
 + constraints . setNullable ( Boolean . FALSE ) ;
 +
 + column . setConstraints ( constraints ) ;
 +
 refactoring . setColumn ( column ) ;

 assertEquals ( STRING2 , refactoring . generateStatements ( new OracleDatabase ( ) ) [ NUMBER0 ] ) ;

Fixed test failing in AddColumnChangeTest due to changes in method AddColumnChangeTest generateStatements


<nb> public class EmulatorView extends View implements GestureDetector . OnGestureListe
 COMMENT
 private ColorScheme mColorScheme = BaseTextRenderer . defaultColorScheme ;

 - COMMENT
 - COMMENT
 - COMMENT
 - private Paint mCursorPaint ;
 -
 private Paint mForegroundPaint ;

 private Paint mBackgroundPaint ;
 <nb> public class EmulatorView extends View implements GestureDetector . OnGestureListe
 COMMENT
 public void attachSession ( TermSession session ) {
 mTextRenderer = null ;
 - mCursorPaint = new Paint ( ) ;
 - mCursorPaint . setARGB ( NUMBER0 , NUMBER1 , NUMBER1 , NUMBER1 ) ;
 mForegroundPaint = new Paint ( ) ;
 mBackgroundPaint = new Paint ( ) ;
 mTopRow = NUMBER2 ;


<nb> import NAMESPACE ;
 import NAMESPACE ;

 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public final class ModelSpecProcessor extends AbstractProcessor {
 }

 @ override
 + public Set < String > getSupportedOptions ( ) {
 + return Collections . singleton ( PLUGINS_KEY ) ;
 + }
 +
 + @ override
 public synchronized void init ( ProcessingEnvironment env ) {
 super . init ( env ) ;

Explicitly declare squidbPlugins as a supported option in ModelSpecProcessor


<nb> import NAMESPACE ;
 import static NAMESPACE ;

 COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 public class App {


<nb> public class TypeMigrationLabeler {
 if ( usageInfo instanceof OverridenUsageInfo ) {
 final String migrationName = ( ( OverridenUsageInfo ) usageInfo ) . getMigrateMethodName ( ) ;
 if ( migrationName != null ) {
 - new RenameProcessor ( project , element , migrationName , false , false ) . run ( ) ;
 + ApplicationManager . getApplication ( ) . invokeLater ( ( ) - > new RenameProcessor ( project , element , migrationName , false , false ) . run ( ) ) ;
 }
 }
 }


<nb> public class IdeBackgroundUtil {
 PaintersHelper . initWallpaperPainter ( STRING0 , painters ) ;

 ApplicationInfoEx appInfo = ApplicationInfoEx . getInstanceEx ( ) ;
 - String path = UIUtil . isUnderDarcula ( ) ? appInfo . getEditorBackgroundImageUrl ( ) : null ;
 + String path = COMMENT null ;
 URL url = path == null ? null : appInfo . getClass ( ) . getResource ( path ) ;
 Image centerImage = url == null ? null : ImageLoader . loadFromUrl ( url ) ;


<nb> public class MaxCulMsgHandler implements CULListener {
 this . endOfQueueTransmit = this . lastTransmit ;
 }

 - COMMENT
 - SINGLE
 - SenderQueueItem qi = new SenderQueueItem ( ) ;
 - qi . msg = data ;
 - qi . expiry = new Date ( this . lastTransmit . getTime ( ) + MESSAGE_EXPIRY_PERIOD ) ;
 - this . pendingAckQueue . put ( qi . msg . msgCount , qi ) ;
 + if ( data . msgType != MaxCulMsgType . ACK )
 + {
 + COMMENT
 + SINGLE
 + SenderQueueItem qi = new SenderQueueItem ( ) ;
 + qi . msg = data ;
 + qi . expiry = new Date ( this . lastTransmit . getTime ( ) + MESSAGE_EXPIRY_PERIOD ) ;
 + this . pendingAckQueue . put ( qi . msg . msgCount , qi ) ;
 + }
 }

 private void sendMessage ( BaseMsg msg )


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 + import NAMESPACE ;

 public class AllTypesRealmProxy extends AllTypes {

 <nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 + import NAMESPACE ;

 public class BooleansRealmProxy extends Booleans {

 <nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 + import NAMESPACE ;

 public class SimpleRealmProxy extends Simple {

Updating unit tests of the annotation processor


<nb> public class ExtensionAutoUpdate extends ExtensionAdaptor implements CheckForUpd

 if ( latestVersionInfo != null ) {
 AddOn addOn = latestVersionInfo . getAddOn ( ao . getId ( ) ) ;
 - if ( AddOn . InstallationStatus . DOWNLOADING == addOn . getInstallationStatus ( ) ) {
 + if ( addOn != null && AddOn . InstallationStatus . DOWNLOADING == addOn . getInstallationStatus ( ) ) {
 addOn . setInstallationStatus ( AddOn . InstallationStatus . INSTALLED ) ;
 }
 }

Fix NullPointerException after installing an add on


<nb> import NAMESPACE ;
 public class GestureController implements View . OnTouchListener {

 private static final float ZOOM_GESTURE_MIN_SPAN_DP = NUMBER0 f ;
 - private static final float FLING_COEFFICIENT = FLOAT0 ;
 + private static final float FLING_COEFFICIENT = FLOAT1 ;

 SINGLE
 private final float mZoomGestureMinSpan ;


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;

 import NAMESPACE ;
 <nb> public final class BloomFilterPostingsFormat extends PostingsFormat {

 @ override
 public long ramBytesUsed ( ) {
 - return RamUsageEstimator . sizeOf ( this ) ;
 + long size = delegateFieldsProducer . ramBytesUsed ( ) ;
 + for ( BloomFilter bloomFilter : bloomsByFieldName . values ( ) ) {
 + size += bloomFilter . getSizeInBytes ( ) ;
 + }
 + return size ;
 }
 -
 -
 }

 public static final class BloomFilteredTerms extends FilterAtomicReader . FilterTerms {

fix computation of ram bytes used in bloom filter posting format


<nb> public class SingleWindowContainer

 addConversationTab (
 callPanel . getCallTitle ( ) , null , callPanel , true ) ;
 +
 + callPanel . requestFocus ( ) ;
 }

 COMMENT

Explicitly request the focus in the call panel in single window mode


<nb> public class MediaGalleryPickerActivity extends SherlockActivity implements Mult

 @ override
 public void onFailure ( ApiHelper . ErrorType errorType , String errorMessage , Throwable throwable ) {
 - if ( errorType == ApiHelper . ErrorType . NO_ERROR ) {
 + if ( errorType != ApiHelper . ErrorType . NO_ERROR ) {
 String message = errorType == ApiHelper . ErrorType . NO_UPLOAD_FILES_CAP
 ? getString ( R . string . media_error_no_permission )
 : getString ( R . string . error_refresh_media ) ;


<nb> public class RabbitMQFirehoseFactory implements FirehoseFactory < StringInputRowPa
 return config ;
 }

 - @ jsonproperty
 + @ jsonproperty ( STRING0 )
 public JacksonifiedConnectionFactory getConnectionFactory ( )
 {
 return connectionFactory ;
 <nb> public class HadoopTuningConfig implements TuningConfig
 return indexSpec ;
 }

 - @ jsonproperty
 + @ jsonproperty ( STRING1 )
 public int getRowFlushBoundary ( )
 {
 return rowFlushBoundary ;
 <nb> public class MoveTask extends AbstractFixedIntervalTask
 return TaskStatus . success ( getId ( ) ) ;
 }

 - @ jsonproperty
 + @ jsonproperty ( STRING2 )
 public Map < String , Object > getTargetLoadSpec ( )
 {
 return targetLoadSpec ;
 <nb> public class DefaultObjectMapper extends ObjectMapper
 configure ( MapperFeature . AUTO_DETECT_FIELDS , false ) ;
 configure ( MapperFeature . AUTO_DETECT_IS_GETTERS , false ) ;
 configure ( MapperFeature . AUTO_DETECT_SETTERS , false ) ;
 + configure ( MapperFeature . ALLOW_FINAL_FIELDS_AS_MUTATORS , false ) ;
 configure ( SerializationFeature . INDENT_OUTPUT , false ) ;
 }

Change DefaultObjectMapper to NOT overwrite final fields unless explicitly asked to


<nb> public class MultiPartInputStreamParser
 File f = new File ( _tmpDir , fileName ) ;
 if ( _file . renameTo ( f ) )
 _file = f ;
 + else
 + throw new IOException ( STRING0 + _file . getName ( ) + STRING1 + fileName + STRING2 + _tmpDir . getAbsolutePath ( ) ) ;
 }
 }

Throw exception if rename during Part write filename fails


<nb> public class RestClientUtils {
 }

 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + public void getNotification ( Map < String , String > params , String noteId , Listener listener , ErrorListener errorListener ) {
 + params . put ( STRING0 , NOTIFICATION_FIELDS ) ;
 + String path = String . format ( STRING1 , noteId ) ;
 + get ( path , params , null , listener , errorListener ) ;
 + }
 +
 + COMMENT
 COMMENT
 COMMENT
 COMMENT

added getNotification method to RestClientUtils in networking lib


<nb> public class RPubsUploadDialog extends ModalDialogBase
 previewButton . addClickHandler ( new ClickHandler ( ) {
 @ override
 public void onClick ( ClickEvent event )
 - {
 - String title = validateAndGetGeneratorTitle ( ) ;
 - if ( title == null )
 - return ;
 -
 + {
 htmlGenerator_ . generateRPubsHtml (
 - title ,
 + titleTextBox_ . getText ( ) . trim ( ) ,
 commentTextArea_ . getText ( ) . trim ( ) ,
 new CommandWithArg < String > ( ) {
 @ override


<nb> public class DescriptorRendererImpl implements DescriptorRenderer {
 assert annotationClass != null ;

 if ( ! excludedAnnotationClasses . contains ( DescriptorUtils . getFQName ( annotationClass ) . toSafe ( ) ) ) {
 - builder . append ( renderType ( annotation . getType ( ) ) ) . append ( STRING0 ) ;
 + builder . append ( renderType ( annotation . getType ( ) ) ) ;
 + if ( verbose ) {
 + builder . append ( STRING1 ) . append ( StringUtil . join ( DescriptorUtils . getSortedValueArguments ( annotation ) , STRING2 ) ) . append ( STRING3 ) ;
 + }
 + builder . append ( STRING0 ) ;
 }
 }
 }


<nb> final class WindowState implements WindowManagerPolicy . WindowState {
 }

 void applyDimLayerIfNeeded ( ) {
 + SINGLE
 + SINGLE
 + SINGLE
 + final AppWindowToken token = mAppToken ;
 + if ( token != null && token . removed ) {
 + return ;
 + }
 +
 if ( ! mExiting && mAppDied ) {
 SINGLE
 mDisplayContent . mDimLayerController . applyDimAbove ( getDimLayerUser ( ) , mWinAnimator ) ;


<nb> public class RefLogEntry implements Serializable , Comparable < RefLogEntry > {
 }

 public PersonIdent getAuthorIdent ( ) {
 + if ( getAuthorCount ( ) == NUMBER0 ) {
 + return getCommits ( ) . get ( NUMBER1 ) . getAuthorIdent ( ) ;
 + }
 return getCommitterIdent ( ) ;
 }


<nb> public class CategoryREST extends AbstractREST {

 @ path ( STRING0 )
 @ post
 - @ apioperation ( value = STRING1 , notes = STRING2 )
 + @ apioperation ( value = STRING1 , notes = STRING2 , responseClass = STRING3 )
 public Response addCategory ( @ apiparam ( required = true ) AddCategoryRequest req ) {
 Preconditions . checkNotNull ( req ) ;
 Preconditions . checkNotNull ( req . getName ( ) ) ;
 <nb> public class CategoryREST extends AbstractREST {
 }
 feedCategoryDAO . saveOrUpdate ( cat ) ;
 cache . invalidateUserRootCategory ( getUser ( ) ) ;
 - return Response . ok ( ) . build ( ) ;
 + return Response . ok ( cat . getId ( ) ) . build ( ) ;
 }

 @ post


<nb> public class OnesideDiffChange {
 myViewer . applyChange ( OnesideDiffChange . this , sourceSide ) ;
 }
 } ) ;
 + SINGLE
 + SINGLE
 + myViewer . rediff ( ) ;
 }
 } ;
 }
 <nb> public class OnesideDiffViewer extends TextDiffViewerBase {

 SINGLE
 SINGLE
 - rediff ( ) ;
 + scheduleRediff ( ) ;
 }

 SINGLE

oneside viewer do not rebuild diff in EDT on clicking chevrone button


<nb> public final class DictionaryVectorizer extends AbstractJob implements Vectorize
 COMMENT
 COMMENT
 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 <nb> public final class DictionaryVectorizer extends AbstractJob implements Vectorize
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 COMMENT
 COMMENT
 COMMENT
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 COMMENT
 public static void createTermFrequencyVectors ( Path input ,
 Path output ,

Revert reorder javadoc paramter comments


<nb> public class RoleHierarchyImpl implements RoleHierarchy {
 COMMENT
 COMMENT
 private void buildRolesReachableInOneStepMap ( ) {
 - String parsingRegex = STRING0 ;
 + String parsingRegex = STRING1 ;
 Pattern pattern = Pattern . compile ( parsingRegex ) ;

 Matcher roleHierarchyMatcher = pattern . matcher ( roleHierarchyStringRepresentation ) ;

Expand regular expression used in hierarchical roles


<nb> public interface FallbackFactory < T > {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 T create ( Throwable cause ) ;
 <nb> final class HystrixInvocationHandler implements InvocationHandler {
 return super . getFallback ( ) ;
 }
 try {
 - Object fallback = fallbackFactory . create ( getFailedExecutionException ( ) ) ;
 + Object fallback = fallbackFactory . create ( getExecutionException ( ) ) ;
 Object result = fallbackMethodMap . get ( method ) . invoke ( fallback , args ) ;
 if ( isReturnsHystrixCommand ( method ) ) {
 return ( ( HystrixCommand ) result ) . execute ( ) ;

Replaces getFailedExecutionException on getExecutionException for fallback factory


<nb> public class BitmapDrawable extends Drawable {
 }

 @ override
 + public boolean isFilterBitmap ( ) {
 + return mBitmapState . mPaint . isFilterBitmap ( ) ;
 + }
 +
 + @ override
 public void setDither ( boolean dither ) {
 mBitmapState . mPaint . setDither ( dither ) ;
 invalidateSelf ( ) ;
 <nb> public class NinePatchDrawable extends Drawable {
 }

 @ override
 + public boolean isFilterBitmap ( ) {
 + if ( mPaint == null ) {
 + return false ;
 + }
 + return getPaint ( ) . isFilterBitmap ( ) ;
 + }
 +
 + @ override
 public void inflate ( Resources r , XmlPullParser parser , AttributeSet attrs , Theme theme )
 throws XmlPullParserException , IOException {
 super . inflate ( r , parser , attrs , theme ) ;

Add isFilterBitmap overrides where appropriate


<nb>
 package NAMESPACE ;

 import NAMESPACE ;
 - import NAMESPACE ;

 import NAMESPACE ;

 <nb> public class SingleValueBlockWithSortedInvertedIndex implements Block {
 private final BlockId id ;
 private final ImmutableDictionaryReader dictionary ;
 private final ColumnMetadata columnMetadata ;
 - private List < Integer > filteredIds ;
 private Predicate p = null ;

 public SingleValueBlockWithSortedInvertedIndex ( BlockId id , FixedBitCompressedSVForwardIndexReader singleValueReader ,
 <nb> public class SingleValueBlockWithSortedInvertedIndex implements Block {
 @ override
 public boolean applyPredicate ( Predicate predicate ) {
 p = predicate ;
 - filteredIds = DictionaryIdFilterUtils . filter ( predicate , dictionary ) ;
 return true ;
 }


<nb> public class CassandraTransaction extends AbstractStoreTransaction {
 Preconditions . checkArgument ( txh instanceof CassandraTransaction , STRING0 , txh . getClass ( ) . getName ( ) ) ;
 return ( CassandraTransaction ) txh ;
 }
 +
 + public String toString ( ) {
 + StringBuilder sb = new StringBuilder ( NUMBER0 ) ;
 + sb . append ( STRING1 ) ;
 + sb . append ( Integer . toHexString ( hashCode ( ) ) ) ;
 + sb . append ( STRING2 ) ;
 + sb . append ( read ) ;
 + sb . append ( STRING3 ) ;
 + sb . append ( write ) ;
 + sb . append ( STRING4 ) ;
 + return sb . toString ( ) ;
 + }
 }


<nb> public class GitBranchUiUtil {
 return currentBranch . getName ( ) ;
 } else {
 String currentRevision = repository . getCurrentRevision ( ) ;
 - return currentRevision != null ? currentRevision : STRING0 ;
 + return currentRevision != null ? currentRevision . substring ( NUMBER0 , NUMBER1 ) : STRING0 ;
 }
 }


<nb>

 package NAMESPACE ;

 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class FilePropertyContainer {
 }
 filePropertiesMap . put ( propertyName , propertySpec . getPropertyFiles ( ) ) ;
 }
 - return filePropertiesMap ;
 + return ImmutableSortedMap . copyOf ( filePropertiesMap ) ;
 }

 protected static < T extends TaskPropertySpec & TaskPropertyBuilder > void ensurePropertiesHaveNames ( Iterable < T > properties ) {


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class GeneralPreferencesPane extends PreferencesPane
 public GeneralPreferencesPane ( RemoteFileSystemContext fsContext ,
 FileDialogs fileDialogs ,
 UIPrefs prefs ,
 + final GlobalDisplay globalDisplay ,
 SourceServerOperations server ,
 final DefaultCRANMirror defaultCRANMirror )
 {
 <nb> public class GeneralPreferencesPane extends PreferencesPane
 {
 String ver = Desktop . getFrame ( ) . chooseRVersion ( ) ;
 if ( ! StringUtil . isNullOrEmpty ( ver ) )
 + {
 rVersion_ . setText ( ver ) ;
 +
 + globalDisplay . showMessage ( MessageDialog . INFO ,
 + STRING0 ,
 + STRING1 +
 + STRING2 ) ;
 + }
 }
 } ) ;
 rVersion_ . setWidth ( STRING3 ) ;

notify user that they must quit and re open RStudio for version change to take effect


<nb> public class IOBalancerStressTest extends HazelcastTestSupport {

 IMap < Integer , Integer > map = instance1 . getMap ( randomMapName ( ) ) ;

 - for ( int i = NUMBER0 ; map . size ( ) < NUMBER1 ; i ++ ) {
 - map . put ( i % NUMBER2 , i ) ;
 + for ( int i = NUMBER0 ; i < NUMBER1 ; i ++ ) {
 + map . put ( i , i ) ;
 }

 assertBalanced ( instance1 ) ;


<nb> public class LayerDrawable extends Drawable implements Drawable . Callback {
 inflateLayers ( r , parser , attrs , theme ) ;

 ensurePadding ( ) ;
 - onStateChange ( getState ( ) ) ;
 }

 COMMENT
 <nb> public class LayerDrawable extends Drawable implements Drawable . Callback {
 }

 ensurePadding ( ) ;
 - onStateChange ( getState ( ) ) ;
 }

 @ override

Don t change state when inflating LayerDrawable


<nb> public class MediaCommandPlayerImpl extends AbstractPrologCommandPlayer implemen
 bld . append ( s ) . append ( STRING0 ) ;
 }
 if ( ctx != null ) {
 - sendAlertToAndroidWear ( ctx , bld . toString ( ) ) ;
 + SINGLE
 }
 return ;
 }
 <nb> public class TTSCommandPlayerImpl extends AbstractPrologCommandPlayer {
 mTts . speak ( bld . toString ( ) , TextToSpeech . QUEUE_ADD , params ) ;
 SINGLE
 } else if ( ctx != null && vrt . isMute ( ) ) {
 - sendAlertToAndroidWear ( ctx , bld . toString ( ) ) ;
 + SINGLE
 }
 }


<nb> public class SubRectangularMatrixWithMaximumSum {

 public static void main ( String args [ ] ) {
 - int input [ ] [ ] = { { NUMBER0 , NUMBER1 , - NUMBER0 , - NUMBER2 , - NUMBER3 } ,
 - { - NUMBER4 , - NUMBER5 , NUMBER2 , NUMBER1 , NUMBER0 } ,
 - { NUMBER5 , NUMBER4 , NUMBER6 , NUMBER0 , NUMBER5 } ,
 - { - NUMBER2 , - NUMBER0 , NUMBER0 , NUMBER7 , - NUMBER8 } } ;
 + int input [ ] [ ] = { { NUMBER1 , NUMBER0 , - NUMBER5 , - NUMBER2 , NUMBER9 } ,
 + { NUMBER10 , NUMBER8 , NUMBER5 , NUMBER2 , NUMBER0 } ,
 + { NUMBER1 , - NUMBER1 , - NUMBER0 , NUMBER2 , - NUMBER9 } ,
 + { - NUMBER5 , NUMBER5 , NUMBER0 , NUMBER10 , NUMBER5 } } ;
 SubRectangularMatrixWithMaximumSum saw = new SubRectangularMatrixWithMaximumSum ( ) ;
 System . out . println ( saw . maxSum ( input ) ) ;
 }

Changing example of max sum sub rectangular matrix question


<nb> public class ConfigUpdateInstallReceiver extends BroadcastReceiver {
 SINGLE
 String currentHash = getCurrentHash ( getCurrentContent ( ) ) ;
 if ( ! verifyVersion ( currentVersion , altVersion ) ) {
 - EventLog . writeEvent ( EventLogTags . CONFIG_INSTALL_FAILED ,
 - STRING0 ) ;
 + Slog . i ( TAG , STRING1 ) ;
 } else if ( ! verifyPreviousHash ( currentHash , altRequiredHash ) ) {
 EventLog . writeEvent ( EventLogTags . CONFIG_INSTALL_FAILED ,
 STRING2 ) ;

Remove logging for old version mismatches


<nb> import NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;

 COMMENT
 <nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;

 COMMENT
 COMMENT
 <nb> public class StoreAccess
 }

 @ suppresswarnings ( STRING0 )
 - protected void apply ( Processor processor , RecordStore < ? > store )
 + protected void apply ( RecordStore . Processor processor , RecordStore < ? > store )
 {
 processor . apply ( store , RecordStore . IN_USE ) ;
 }


<nb> public class AtmosphereFramework {
 interceptors . clear ( ) ;

 broadcasterFactory = null ;
 + arFactory = null ;
 annotationFound = false ;
 return this ;
 }


<nb> import NAMESPACE ;
 COMMENT
 COMMENT
 COMMENT
 - class ControlFlowGraph < N > extends
 + public class ControlFlowGraph < N > extends
 LinkedDirectedGraph < N , ControlFlowGraph . Branch > {

 COMMENT


<nb> public final class BlockIdTests {
 public void createBlockIdTest ( ) {
 Assert . assertEquals ( NUMBER0 , BlockId . createBlockId ( NUMBER1 , NUMBER2 ) ) ;
 Assert . assertEquals ( NUMBER2 , BlockId . createBlockId ( NUMBER3 , NUMBER2 ) ) ;
 + Assert . assertEquals ( NUMBER4 , BlockId . createBlockId ( NUMBER5 , NUMBER6 ) ) ;
 }

 @ test
 <nb> public final class BlockIdTests {
 Assert . assertEquals ( MAX_SEQUENCE_NUMBER , BlockId . getSequenceNumber ( NUMBER7 ) ) ;
 Assert . assertEquals ( NUMBER8 , BlockId . getContainerId ( NUMBER9 ) ) ;
 Assert . assertEquals ( MAX_SEQUENCE_NUMBER , BlockId . getSequenceNumber ( NUMBER9 ) ) ;
 + Assert . assertEquals ( NUMBER6 , BlockId . getContainerId ( NUMBER4 ) ) ;
 + Assert . assertEquals ( NUMBER6 , BlockId . getSequenceNumber ( NUMBER4 ) ) ;
 }

 @ test


<nb> public class JetTestUtils {
 }

 public static String getFilePath ( File file ) {
 - return file . getPath ( ) . replaceAll ( STRING0 , STRING1 ) ;
 + return FileUtil . toSystemIndependentName ( file . getPath ( ) ) ;
 }

 public interface TestFileFactory < F > {
 <nb> public class JetTestUtils {
 Class < ? extends TestCase > [ ] innerClasses = innerClassesAnnotation == null ? NO_INNER_CLASSES : innerClassesAnnotation . value ( ) ;
 for ( Class < ? > innerClass : innerClasses ) {
 TestMetadata testMetadata = innerClass . getAnnotation ( TestMetadata . class ) ;
 - if ( testMetadata != null && testMetadata . value ( ) . equals ( testDataDir . getPath ( ) ) ) {
 + if ( testMetadata != null && testMetadata . value ( ) . equals ( getFilePath ( testDataDir ) ) ) {
 return ;
 }
 }


<nb> public class main {
 if ( commandLine . hasOption ( STRING0 ) ) {
 doDump = true ;
 disassemble = false ;
 - dumpFileName = commandLine . getOptionValue ( STRING1 , inputDexFileName + STRING2 ) ;
 + dumpFileName = commandLine . getOptionValue ( STRING0 , inputDexFileName + STRING2 ) ;
 }

 if ( commandLine . hasOption ( STRING3 ) ) {

fixed an issue where D didn t use the specified file


<nb> public class InterpreterFactory {
 InterpreterSetting setting = info . interpreterSettings . get ( k ) ;

 SINGLE
 - SINGLE
 + SINGLE
 + SINGLE
 SINGLE
 setting . getOption ( ) . setRemote ( true ) ;


<nb> public class PackageManagerService extends IPackageManager . Stub {
 if ( pkg . mVersionCode < ps . versionCode ) {
 SINGLE
 SINGLE
 - logCriticalInfo ( Log . INFO , STRING0 + ps . name + STRING1 + scanFile
 + Slog . i ( TAG , STRING0 + ps . name + STRING1 + scanFile
 + STRING2 + ps . versionCode
 + STRING3 + pkg . mVersionCode ) ;
 if ( ! updatedPkg . codePath . equals ( scanFile ) ) {

Reduce spammy PackageManager critical logs


<nb> public interface AgentBuilder {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 <nb> public interface AgentBuilder {
 COMMENT
 COMMENT
 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 REDEFINITION {
 @ override
 <nb> public interface AgentBuilder {
 } ,

 COMMENT
 + COMMENT
 COMMENT
 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 RETRANSFORMATION {
 @ override

Added documentation to clarify use of redefinition from an agent


<nb> public class ApplicationActionBarAdvisor extends ActionBarAdvisor
 helpMenu . add ( showHelpAction ) ;
 SINGLE
 SINGLE
 + helpMenu . add ( new Separator ( ) ) ;
 + helpMenu . add ( ActionUtils . makeCommandContribution ( getActionBarConfigurer ( ) . getWindowConfigurer ( ) . getWindow ( ) , STRING0 ) ) ;
 helpMenu . add ( ActionUtils . makeAction ( checkUpdatesAction , null , null , CoreMessages . actions_menu_check_update , null , null ) ) ;
 }


<nb> public final class StripedExecutor implements Executor {

 for ( Worker worker : workers ) {
 worker . workQueue . clear ( ) ;
 + worker . interrupt ( ) ;
 }
 }

StripedExecutor should interrupt worker threads during shutdown


<nb> public class MultiUserChat {
 COMMENT
 COMMENT
 COMMENT
 - public void changeNickname ( Resourcepart nickname ) throws NoResponseException , XMPPErrorException , NotConnectedException , InterruptedException , MucNotJoinedException {
 + public synchronized void changeNickname ( Resourcepart nickname ) throws NoResponseException , XMPPErrorException , NotConnectedException , InterruptedException , MucNotJoinedException {
 StringUtils . requireNotNullOrEmpty ( nickname , STRING0 ) ;
 SINGLE
 SINGLE

Synchronize MultiUserchat changeNickname Resourcepart


<nb> public class BottomSheetBehavior < V extends View > extends CoordinatorLayout . Behav
 COMMENT
 COMMENT
 public final void setState ( @ state int state ) {
 + if ( mViewRef == null ) {
 + return ;
 + }
 V child = mViewRef . get ( ) ;
 if ( child == null ) {
 return ;

Fix NPE when calling setState before onLayoutChild


<nb> public class HistoryEntry {
 }

 public void addFile ( String file ) {
 - files . add ( file ) ;
 + if ( ! files . contains ( file ) ) {
 + files . add ( file ) ;
 + }
 }

 public List < String > getFiles ( ) {
 <nb> public class HistoryEntry {
 }

 public void addChangeRequest ( String changeRequest ) {
 - changeRequests . add ( changeRequest ) ;
 + if ( ! changeRequests . contains ( changeRequest ) ) {
 + changeRequests . add ( changeRequest ) ;
 + }
 }

 public List < String > getChangeRequests ( ) {

Don t store multiple occurences of filenames or changerequests pr


<nb>

 package NAMESPACE ;

 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> import static NAMESPACE ;
 COMMENT
 public class UpdateNumberOfReplicasTests extends AbstractSharedClusterTest {

 +
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 @ test
 + @ awaitsfix ( bugUrl = STRING0 )
 public void simpleUpdateNumberOfReplicasTests ( ) throws Exception {
 logger . info ( STRING1 ) ;
 prepareCreate ( STRING2 , NUMBER0 ) . execute ( ) . actionGet ( ) ;


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 - import NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class MockMultipartFile implements MultipartFile {
 this ( name , originalFilename , contentType , FileCopyUtils . copyToByteArray ( contentStream ) ) ;
 }

 -
 public String getName ( ) {
 return this . name ;
 }


<nb> public class ScheduledReporterTest {

 @ test
 public void pollsPeriodically ( ) throws Exception {
 - verify ( reporter , timeout ( NUMBER0 ) . times ( NUMBER1 ) ) . report (
 + Thread . sleep ( NUMBER0 ) ;
 + verify ( reporter , times ( NUMBER1 ) ) . report (
 map ( STRING0 , gauge ) ,
 map ( STRING1 , counter ) ,
 map ( STRING2 , histogram ) ,


<nb>
 COMMENT
 package NAMESPACE ;

 + import NAMESPACE ;
 +
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class SourceCode {
 } catch ( Exception e ) {
 throw new RuntimeException ( STRING0 + getFileName ( ) + STRING1 + e . getMessage ( ) , e ) ;
 } finally {
 - try {
 - if ( lnr != null ) {
 - lnr . close ( ) ;
 - }
 - } catch ( Exception e ) {
 - throw new RuntimeException ( STRING0 + getFileName ( ) + STRING1 + e . getMessage ( ) , e ) ;
 - }
 + Closeables . closeQuietly ( lnr ) ;
 }
 }
 }


<nb> import NAMESPACE ;
 public class Response {

 COMMENT
 + @ getter
 @ xmlelement
 private HashMap < String , String > requestParameters ;
 private TripPlan plan ;
 <nb> public class Response {
 public Debug debug = null ;

 COMMENT
 + @ suppresswarnings ( STRING0 )
 private Response ( ) { } ;

 COMMENT

Add request parameters back to JSON response


<nb> public class KeyguardTransportControlView extends FrameLayout implements OnClick
 }

 @ override
 + protected void onSizeChanged ( int w , int h , int oldw , int oldh ) {
 + if ( mAttached ) {
 + int dim = Math . min ( NUMBER0 , Math . max ( w , h ) ) ;
 + if ( DEBUG ) Log . v ( TAG , STRING0 + dim ) ;
 + mAudioManager . remoteControlDisplayUsesBitmapSize ( mIRCD , dim , dim ) ;
 + }
 + }
 +
 + @ override
 public void onDetachedFromWindow ( ) {
 if ( DEBUG ) Log . v ( TAG , STRING1 ) ;
 super . onDetachedFromWindow ( ) ;

Keyguard remote control display artwork size


<nb> public class LeakHunter {
 cached = fields . isEmpty ( ) ? EMPTY_FIELD_ARRAY : fields . toArray ( new Field [ fields . size ( ) ] ) ;
 }
 catch ( IncompatibleClassChangeError e ) {
 - throw new RuntimeException ( STRING0 + aClass , e ) ;
 + SINGLE
 + SINGLE
 + cached = EMPTY_FIELD_ARRAY ;
 }
 catch ( SecurityException e ) {
 cached = EMPTY_FIELD_ARRAY ;

suppressed IncompatibleClassChangeError in testProjectLeak


<nb> public class GuardedDatabaseFactory implements GraphDatabaseFactory {
 @ override protected WrappedNode < WrappedGraphDatabase > node ( final Node node , boolean created ) {
 SINGLE
 guard . check ( ) ;
 + if ( node == null ) {
 + return null ;
 + }
 return new WrappedNode < WrappedGraphDatabase > ( this ) {
 @ override protected Node actual ( ) {
 return node ;
 <nb> public class GuardedDatabaseFactory implements GraphDatabaseFactory {
 protected WrappedRelationship < WrappedGraphDatabase > relationship ( final Relationship relationship , boolean created ) {
 SINGLE
 guard . check ( ) ;
 + if ( relationship == null ) {
 + return null ;
 + }
 return new WrappedRelationship < WrappedGraphDatabase > ( this ) {
 @ override protected Relationship actual ( ) {
 return relationship ;

fix wrapping null nodes relationships that may occur on putIfAbsent


<nb> public class ApacheAsyncHttpProvider implements AsyncHttpProvider < HttpClient > {
 }

 int requestTimeout = requestTimeout ( config , request . getPerRequestConfig ( ) ) ;
 - if ( config . getIdleConnectionTimeoutInMs ( ) > NUMBER0 ) {
 + if ( config . getIdleConnectionTimeoutInMs ( ) > NUMBER0 && requestTimeout != - NUMBER1 && requestTimeout < config . getIdleConnectionTimeoutInMs ( ) ) {
 idleConnectionTimeoutThread = new IdleConnectionTimeoutThread ( ) ;
 idleConnectionTimeoutThread . setConnectionTimeout ( config . getIdleConnectionTimeoutInMs ( ) ) ;
 idleConnectionTimeoutThread . addConnectionManager ( connectionManager ) ;

Fix Thread Race between the two mechanism


<nb> class LdapConfigUtils {
 if ( registry . containsBeanDefinition ( BeanIds . CONTEXT_SOURCE_SETTING_POST_PROCESSOR ) ) {
 if ( defaultNameRequired ) {
 BeanDefinition bd = registry . getBeanDefinition ( BeanIds . CONTEXT_SOURCE_SETTING_POST_PROCESSOR ) ;
 - bd . getPropertyValues ( ) . addPropertyValue ( STRING0 , defaultNameRequired ) ;
 + bd . getPropertyValues ( ) . addPropertyValue ( STRING0 , Boolean . valueOf ( defaultNameRequired ) ) ;
 }
 return ;
 }

 BeanDefinition bd = new RootBeanDefinition ( ContextSourceSettingPostProcessor . class ) ;
 registry . registerBeanDefinition ( BeanIds . CONTEXT_SOURCE_SETTING_POST_PROCESSOR , bd ) ;
 - bd . getPropertyValues ( ) . addPropertyValue ( STRING0 , defaultNameRequired ) ;
 + bd . getPropertyValues ( ) . addPropertyValue ( STRING0 , Boolean . valueOf ( defaultNameRequired ) ) ;
 }

 }


<nb> public class OrcFileStripeMergeRecordReader implements
 keyWrapper . setRowIndexStride ( reader . getRowIndexStride ( ) ) ;
 keyWrapper . setTypes ( reader . getTypes ( ) ) ;
 } else {
 + stripeIdx ++ ;
 continue ;
 }
 return true ;

Alter table concatenate oparetor will cause duplicate data guoliming reviewed by Prasanth Jayachandran


<nb> public class JavaScriptScriptSearchTests extends ElasticsearchIntegrationTest {

 logger . info ( STRING0 ) ;
 SearchResponse response = client ( ) . prepareSearch ( )
 - . setQuery ( scriptQuery ( new Script ( STRING1 , ScriptService . ScriptType . INLINE , STRING2 , null ) ) )
 + . setQuery ( scriptQuery ( new Script ( STRING1 , ScriptService . ScriptType . INLINE , STRING3 , null ) ) )
 . addSort ( STRING4 , SortOrder . ASC )
 - . addScriptField ( STRING5 , new Script ( STRING6 , ScriptService . ScriptType . INLINE , STRING2 , null ) )
 + . addScriptField ( STRING5 , new Script ( STRING6 , ScriptService . ScriptType . INLINE , STRING3 , null ) )
 . execute ( ) . actionGet ( ) ;

 assertThat ( response . getHits ( ) . totalHits ( ) , equalTo ( NUMBER0 ) ) ;

TEST Fix JavaScript tests case matters here


<nb> public class Blog {
 }

 public boolean checkCapability ( Capability capability ) {
 + SINGLE
 if ( capabilities == null || capabilities . isEmpty ( ) || capability == null ) {
 return false ;
 }

Adds a helpful comment to checkCapability


<nb> public class ObservableList implements List {
 private List values = new ArrayList ( ) ;

 public MultiElementRemovedEvent ( Object source , List values ) {
 - super ( source , ChangeType . oldValue , ChangeType . newValue , NUMBER0 , ChangeType . MULTI_ADD ) ;
 + super ( source , ChangeType . oldValue , ChangeType . newValue , NUMBER0 , ChangeType . MULTI_REMOVE ) ;
 if ( values != null ) {
 this . values . addAll ( values ) ;
 }

Fix wrong event type in ObservableList MultiElementRemovedEvent


<nb> public class DefaultPackageManagerIntentComparatorTest {
 public void canSustainConcurrentModification ( ) {
 final DefaultPackageManager . IntentComparator intentComparator = new DefaultPackageManager . IntentComparator ( ) ;

 - Intent mockedIntent1 = mock ( Intent . class ) ;
 - when ( mockedIntent1 . getAction ( ) ) . thenReturn ( STRING0 , ( String ) null ) ;
 -
 - Intent mockedIntent2 = mock ( Intent . class ) ;
 - when ( mockedIntent2 . getAction ( ) ) . thenReturn ( STRING1 , ( String ) null ) ;
 -
 - Assertions . assertThat ( intentComparator . compare ( mockedIntent1 , mockedIntent2 ) ) . isEqualTo ( - NUMBER0 ) ;
 + Intent intent1 = new Intent ( STRING0 ) ;
 + Intent intent2 = new Intent ( STRING1 ) ;
 + Assertions . assertThat ( intentComparator . compare ( intent1 , intent2 ) ) . isEqualTo ( - NUMBER0 ) ;
 }

 }


<nb> public abstract class AbstractShortestPathTree implements ShortestPathTree {
 }

 @ override
 + public List < GraphPath > getPaths ( ) {
 + return getPaths ( rctx . target , true ) ;
 + }
 +
 + @ override
 public List < GraphPath > getPaths ( Vertex dest , boolean optimize ) {
 List < ? extends State > stateList = getStates ( dest ) ;
 if ( stateList == null )
 <nb> public interface ShortestPathTree {
 COMMENT
 public List < GraphPath > getPaths ( Vertex dest , boolean optimize ) ;

 + COMMENT
 + public List < GraphPath > getPaths ( ) ;
 +
 public GraphPath getPath ( Vertex dest , boolean optimize ) ;

 COMMENT

add method to fetch default paths from SPT based on routing context


<nb> public class MUCPersistenceManager {
 STRING0 ;
 private static final String LOAD_HISTORY =
 STRING1 +
 - STRING2 ;
 + STRING3 ;
 private static final String LOAD_ALL_ROOMS =
 STRING4 +
 STRING5 +
 <nb> public class MUCPersistenceManager {
 STRING6 ;
 private static final String LOAD_ALL_HISTORY =
 STRING7 +
 - STRING8 ;
 + STRING9 ;
 private static final String UPDATE_ROOM =
 STRING10 +
 STRING11 +


<nb> public class FileMetadata implements BatchComponent {
 this . analysisMode = analysisMode ;
 }

 - private static abstract class CharHandler {
 + private abstract static class CharHandler {

 void handleAll ( char c ) {
 }
 <nb> public class FileMetadata implements BatchComponent {

 private static class LineHashComputer extends CharHandler {
 private final MessageDigest lineMd5Digest = DigestUtils . getMd5Digest ( ) ;
 - private final StringBuffer sb = new StringBuffer ( ) ;
 + private final StringBuilder sb = new StringBuilder ( ) ;
 private final LineHashConsumer consumer ;
 private int line = NUMBER0 ;


<nb> public class Alert extends BaseIdentifiable implements Cloneable {
 }

 COMMENT
 - COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 + @ deprecated
 public String getAlertLabel ( Metric . Level level ) {
 return new StringBuilder ( )
 . append ( getMetric ( ) . getName ( ) )


<nb> public final class ResourceUtils {

 COMMENT
 COMMENT
 + COMMENT
 COMMENT
 + @ deprecated
 public static boolean isUnitTestClass ( Resource resource ) {
 + return isUnitTestFile ( resource ) ;
 + }
 +
 + COMMENT
 + COMMENT
 + COMMENT
 + public static boolean isUnitTestFile ( Resource resource ) {
 return Qualifiers . UNIT_TEST_FILE . equals ( resource . getQualifier ( ) ) ;
 }

Replace isUnitTestClass by isUnitTestFile


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 <nb> public class GoModuleLibrariesInitializer implements ModuleComponent {
 }

 private boolean isAppropriateModule ( ) {
 - return ! myModule . isDisposed ( ) && ModuleUtil . getModuleType ( myModule ) == GoModuleType . getInstance ( ) ;
 + return ! myModule . isDisposed ( ) && ( PlatformUtils . isIntelliJ ( ) || ModuleUtil . getModuleType ( myModule ) == GoModuleType . getInstance ( ) ) ;
 }

 @ override

Consider any module as an appropriate in little IDE


<nb> public class AllocationService extends AbstractComponent {
 }

 private boolean reroute ( RoutingAllocation allocation ) {
 - Iterable < DiscoveryNode > dataNodes = allocation . nodes ( ) . dataNodes ( ) . values ( ) ;
 -
 boolean changed = false ;
 SINGLE
 changed |= deassociateDeadNodes ( allocation ) ;


<nb> public class AppWidgetManager {
 COMMENT
 public List < AppWidgetProviderInfo > getInstalledProviders ( ) {
 try {
 - return sService . getInstalledProviders ( ) ;
 + List < AppWidgetProviderInfo > providers = sService . getInstalledProviders ( ) ;
 + for ( AppWidgetProviderInfo info : providers ) {
 + SINGLE
 + info . minWidth =
 + TypedValue . complexToDimensionPixelSize ( info . minWidth , mDisplayMetrics ) ;
 + info . minHeight =
 + TypedValue . complexToDimensionPixelSize ( info . minHeight , mDisplayMetrics ) ;
 + }
 + return providers ;
 }
 catch ( RemoteException e ) {
 throw new RuntimeException ( STRING0 , e ) ;

Convert the minWidth and minHeight to device units when calling getInstalledProviders


<nb> public class TachyonFS {
 ClientFileInfo info = null ;
 if ( ! fetch ) {
 info = mClientFileInfos . get ( fId ) ;
 - if ( info . blockIds . size ( ) <= blockIndex ) {
 + if ( ! info . isFolder ( ) && info . blockIds . size ( ) <= blockIndex ) {
 fetch = true ;
 }
 }
 <nb> public class TachyonFS {
 if ( info == null ) {
 throw new IOException ( STRING0 + fId + STRING1 ) ;
 }
 + if ( info . isFolder ( ) ) {
 + throw new IOException ( new FileDoesNotExistException ( STRING0 + fId + STRING2 ) ) ;
 + }
 if ( info . blockIds . size ( ) <= blockIndex ) {
 throw new IOException ( STRING3 + blockIndex + STRING4 + info ) ;
 }

Fix the NPE in getClientBlockInfo while INode is a DIR


<nb> public class StandardPipeline extends LifecycleBase
 if ( valve instanceof Contained ) {
 ( ( Contained ) valve ) . setContainer ( this . container ) ;
 }
 - if ( valve instanceof Lifecycle ) {
 + if ( getState ( ) . isAvailable ( ) && valve instanceof Lifecycle ) {
 try {
 ( ( Lifecycle ) valve ) . start ( ) ;
 } catch ( LifecycleException e ) {
 log . error ( STRING0 , e ) ;
 return ;
 }
 + SINGLE
 + registerValve ( valve ) ;
 }

 SINGLE

Fix some inconsistencies identified during lifecycle refactoring for valves


<nb> public class MessageParser extends AbstractParser implements OnMessagePacketRece
 return ;
 }

 - if ( extractChatState ( mXmppConnectionService . find ( account , counterpart . toBareJid ( ) ) , packet ) ) {
 + if ( ! isTypeGroupChat
 + && query == null
 + && extractChatState ( mXmppConnectionService . find ( account , counterpart . toBareJid ( ) ) , packet ) ) {
 mXmppConnectionService . updateConversationUi ( ) ;
 }

don t handle chat states in muc or from archive


<nb> public class DTree extends Iced {
 default : assert false : STRING0 + lmask + STRING1 + ab . position ( ) + STRING2 + Arrays . toString ( _bits ) ;
 }

 - if ( ! Double . isNaN ( row [ colId ] ) ) SINGLE
 + if ( ! Double . isNaN ( row [ colId ] ) ) { SINGLE
 if ( ( equal && ( ( float ) row [ colId ] ) == splitVal ) ||
 ( ! equal && ( ( float ) row [ colId ] ) >= splitVal ) ) {
 ab . position ( ab . position ( ) + skip ) ; SINGLE
 lmask = rmask ; SINGLE
 }
 + } else { ab . position ( ab . position ( ) + skip ) ; lmask = rmask ; }
 if ( ( lmask & NUMBER0 ) == NUMBER0 ) return scoreLeaf ( ab ) ;
 }
 }


<nb> class ConsumerRepository < T > implements Iterable < ConsumerInfo >
 }
 }

 + @ override
 public Iterator < ConsumerInfo > iterator ( )
 {
 return consumerInfos . iterator ( ) ;


<nb> public class RCompletionManager implements CompletionManager

 AutocompletionContext context = getAutocompletionContext ( ) ;

 - if ( ! input_ . hasSelection ( ) )
 - {
 - Debug . log ( STRING0 ) ;
 - return false ;
 - }
 -
 context_ = new CompletionRequestContext ( invalidation_ . getInvalidationToken ( ) ,
 selection ,
 canAutoInsert ) ;

remove old code path that was never touched


<nb> public class SessionManagementFilter extends SpringSecurityFilter {
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 - public void setInvalidSessionUrl ( String sessionTimeoutUrl ) {
 - this . invalidSessionUrl = sessionTimeoutUrl ;
 + public void setInvalidSessionUrl ( String invalidSessionUrl ) {
 + this . invalidSessionUrl = invalidSessionUrl ;
 }
 }

Renamed setter argument to match property


<nb> public class NameUtil {
 }

 if ( StringUtil . toLowerCase ( name . charAt ( nameIndex ) ) != StringUtil . toLowerCase ( myPattern [ patternIndex ] ) ) {
 + if ( Character . isDigit ( name . charAt ( nameIndex ) ) ) {
 + return matchName ( name , patternIndex , nameIndex + NUMBER0 ) ;
 + }
 return null ;
 }


<nb> public class InferenceIncorporationPhase {
 PsiType [ ] typeArgs = right . getParameters ( ) ;
 if ( parameters . length != typeArgs . length ) continue ;
 for ( int i = NUMBER0 ; i < typeArgs . length ; i ++ ) {
 - PsiType aType = typeArgs [ i ] ;
 - if ( aType instanceof PsiCapturedWildcardType ) {
 - aType = ( ( PsiCapturedWildcardType ) aType ) . getWildcard ( ) ;
 - }
 + final PsiType aType = typeArgs [ i ] ;
 final InferenceVariable inferenceVariable = mySession . getInferenceVariable ( parameters [ i ] ) ;
 LOG . assertTrue ( inferenceVariable != null ) ;


<nb> public class IndexShard extends AbstractIndexShardComponent {
 } finally {
 final Engine engine = this . currentEngineReference . getAndSet ( null ) ;
 try {
 - if ( flushEngine && this . flushOnClose ) {
 + if ( engine != null && flushEngine && this . flushOnClose ) {
 engine . flushAndClose ( ) ;
 }
 } finally { SINGLE

Check engine reference for null before flushing


<nb> public class DocumentImpl extends UserDataHolderBase implements DocumentEx {
 }
 assertInsideCommand ( ) ;

 + getLineSet ( ) ; SINGLE
 +
 DocumentEvent event = new DocumentEventImpl ( this , offset , oldString , newString , myModificationStamp , wholeTextReplaced ) ;

 if ( ! ShutDownTracker . isShutdownHookRunning ( ) ) {
 <nb> public class DocumentImpl extends UserDataHolderBase implements DocumentEx {
 try {
 if ( LOG . isDebugEnabled ( ) ) LOG . debug ( event . toString ( ) ) ;

 - synchronized ( myLineSetLock ) {
 - LineSet lineSet = myLineSet ;
 - if ( lineSet != null ) {
 - lineSet . changedUpdate ( event ) ;
 - }
 - }
 + getLineSet ( ) . changedUpdate ( event ) ;
 setModificationStamp ( newModificationStamp ) ;

 if ( ! ShutDownTracker . isShutdownHookRunning ( ) ) {

initialize LineSet before document changes


<nb> public class XmppConnectionService extends Service {
 }
 for ( Account account : getAccounts ( ) ) {
 if ( account . getStatus ( ) == Account . State . ONLINE ) {
 + account . deactivateGracePeriod ( ) ;
 XmppConnection connection = account . getXmppConnection ( ) ;
 if ( connection != null && connection . getFeatures ( ) . csi ( ) ) {
 connection . sendActive ( ) ;

deactivate grace period when coming to foreground


<nb> public class UnusedDefInspection extends GroovyLocalInspectionBase {
 PsiElement parent = element . getParent ( ) ;
 PsiElement toHighlight = null ;
 if ( parent instanceof GrAssignmentExpression ) {
 - toHighlight = ( ( GrAssignmentExpression ) parent ) . getRValue ( ) ;
 + toHighlight = ( ( GrAssignmentExpression ) parent ) . getLValue ( ) ;
 } if ( parent instanceof GrPostfixExpression ) {
 toHighlight = parent ;
 }
 if ( toHighlight == null ) toHighlight = element ;
 problemsHolder . registerProblem ( toHighlight , GroovyInspectionBundle . message ( STRING0 ) , ProblemHighlightType . LIKE_UNUSED_SYMBOL ) ;
 } else if ( element instanceof GrVariable ) {
 - problemsHolder . registerProblem ( ( ( GrVariable ) element ) . getInitializerGroovy ( ) , GroovyInspectionBundle . message ( STRING0 ) , ProblemHighlightType . LIKE_UNUSED_SYMBOL ) ;
 + problemsHolder . registerProblem ( ( ( GrVariable ) element ) . getNameIdentifierGroovy ( ) , GroovyInspectionBundle . message ( STRING0 ) , ProblemHighlightType . LIKE_UNUSED_SYMBOL ) ;
 }
 }
 return true ;

highlight assigned unused variable instead of the value


<nb> public final class Settings {
 public static final String VENDING_TOS_URL = STRING0 ;

 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + public static final String VENDING_TOS_MISSING_URL = STRING1 ;
 +
 + COMMENT
 COMMENT
 COMMENT
 COMMENT

Add gservices flag for configuring the URL vending redirects to when TOS is inaccessible due to no bad network


<nb> public class OpenStreetMapGraphBuilderImpl implements GraphBuilder {
 networkSet . addAll ( Arrays . asList ( operators . split ( STRING0 ) ) ) ;
 if ( networkSet . isEmpty ( ) ) {
 _log . warn ( STRING1 + node . getId ( ) + STRING2
 - + creativeName + STRING3 ) ;
 - continue ;
 + + creativeName + STRING4 ) ;
 + networkSet . add ( STRING5 ) ; SINGLE
 }
 BikeRentalStationVertex station = new BikeRentalStationVertex ( graph , STRING6
 + node . getId ( ) , STRING7 + node . getId ( ) , node . getLon ( ) ,
 <nb> public abstract class RentABikeAbstractEdge extends Edge {
 private boolean hasCompatibleNetworks ( Set < String > stationNetworks , Set < String > rentedNetworks ) {
 COMMENT
 COMMENT
 + COMMENT
 COMMENT
 + if ( stationNetworks . contains ( STRING5 ) || rentedNetworks . contains ( STRING5 ) )
 + return true ;
 return ! Sets . intersection ( stationNetworks , rentedNetworks ) . isEmpty ( ) ;
 }
 }


<nb> public class PsiAnonymousClassImpl extends PsiClassImpl implements PsiAnonymousC
 PsiElement lastParent ,
 @ notnull PsiElement place ) {
 if ( lastParent instanceof PsiExpressionList ) return true ;
 +
 + final PsiClassStub stub = getStub ( ) ;
 + if ( stub != null ) {
 + SINGLE
 + SINGLE
 + SINGLE
 + return true ;
 + }
 +
 COMMENT
 && lastParent . getParent ( ) == this && lastParent == getBaseClassReference ( ) ) {
 return true ;


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class SmsSync extends PreferenceActivity {
 String getLastSyncText ( final long lastSync ) {
 return getString ( R . string . status_idle_details ,
 lastSync < NUMBER0 ? getString ( R . string . status_idle_details_never ) :
 - new Date ( lastSync ) . toLocaleString ( ) ) ;
 + DateFormat . getDateTimeInstance ( ) . format ( new Date ( lastSync ) ) ) ;
 +
 }

 private void updateLastBackupTimes ( ) {


<nb> public enum ItemType {
 TYPE_DEBUG_INFO_ITEM ( NUMBER0 , STRING0 ) ,
 TYPE_ANNOTATION_ITEM ( NUMBER1 , STRING1 ) ,
 TYPE_ENCODED_ARRAY_ITEM ( NUMBER2 , STRING2 ) ,
 - TYPE_ANNOTATIONS_DIRECTORY_ITEM ( NUMBER3 , STRING3 ) ,
 - TYPE_MAP_ITEM ( - NUMBER4 , STRING4 ) ,
 - TYPE_TYPE_ITEM ( - NUMBER4 , STRING5 ) ,
 - TYPE_EXCEPTION_HANDLER_ITEM ( - NUMBER4 , STRING6 ) ,
 - TYPE_ANNOTATION_SET_REF_ITEM ( - NUMBER4 , STRING7 ) ;
 + TYPE_ANNOTATIONS_DIRECTORY_ITEM ( NUMBER3 , STRING3 ) ;

 COMMENT
 private final static TreeMap < Integer , ItemType > itemTypeIntegerMap ;

removed unneeded values in the ItemType enumeration


<nb> public class WebsocketComponent extends DefaultComponent {

 @ override
 protected Endpoint createEndpoint ( String uri , String remaining , Map < String , Object > parameters ) throws Exception {
 -
 + SINGLE
 SSLContextParameters sslContextParameters = resolveAndRemoveReferenceParameter ( parameters , STRING0 , SSLContextParameters . class ) ;
 + if ( sslContextParameters == null ) {
 + sslContextParameters = resolveAndRemoveReferenceParameter ( parameters , STRING1 , SSLContextParameters . class ) ;
 + }
 Boolean enableJmx = getAndRemoveParameter ( parameters , STRING2 , Boolean . class ) ;
 String staticResources = getAndRemoveParameter ( parameters , STRING3 , String . class ) ;
 int port = extractPortNumber ( remaining ) ;

deprecate the xxxRef options and provide the xxx option if it not exists yet work in progress


<nb> public class IngestBootstrapper extends AbstractLifecycleComponent implements Cl
 this . pipelineStore = new PipelineStore ( settings , clusterService , transportService ) ;
 this . pipelineExecutionService = new PipelineExecutionService ( pipelineStore , threadPool ) ;

 - clusterService . add ( this ) ;
 + boolean isNoTribeNode = settings . getByPrefix ( STRING0 ) . getAsMap ( ) . isEmpty ( ) ;
 + if ( isNoTribeNode ) {
 + clusterService . add ( this ) ;
 + }
 }

 SINGLE


<nb> public class AttachmentMapper extends FieldMapper {
 try {
 parsedContent = parseWithTika ( content , metadata , indexedChars ) ;
 } catch ( Throwable e ) {
 - SINGLE
 - if ( e instanceof PrivilegedActionException ) {
 - e = e . getCause ( ) ;
 - }
 SINGLE
 if ( ! ignoreErrors ) {
 logger . trace ( STRING0 , e ) ;

remove unnecessary unboxing we do that as impl detail in parseTika


<nb> public class PhoneStatusBar extends BaseStatusBar implements DemoMode {
 private boolean mAutohideSuspended ;
 private int mStatusBarMode ;
 private int mNavigationBarMode ;
 - private boolean mScreenOn ;
 + private Boolean mScreenOn ;

 private final Runnable mAutohide = new Runnable ( ) {
 @ override
 <nb> public class PhoneStatusBar extends BaseStatusBar implements DemoMode {
 private void checkBarMode ( int mode , int windowState , BarTransitions transitions ) {
 final boolean imeVisible = ( mNavigationIconHints & NAVIGATION_HINT_BACK_ALT ) != NUMBER0 ;
 final int finalMode = imeVisible ? MODE_OPAQUE : mode ;
 - final boolean animate = mScreenOn && windowState != WINDOW_STATE_HIDDEN ;
 - transitions . transitionTo ( finalMode , animate ) ;
 + final boolean anim = ( mScreenOn == null || mScreenOn ) && windowState != WINDOW_STATE_HIDDEN ;
 + transitions . transitionTo ( finalMode , anim ) ;
 }

 private final Runnable mCheckBarModes = new Runnable ( ) {

Don t disable animations until screen state is known


<nb> public class PApplet implements PConstants {
 fileChooser . setDialogTitle ( prompt ) ;
 fileChooser . setFileSelectionMode ( JFileChooser . DIRECTORIES_ONLY ) ;
 if ( defaultSelection != null ) {
 - fileChooser . setSelectedFile ( defaultSelection ) ;
 + fileChooser . setCurrentDirectory ( defaultSelection ) ;
 }

 int result = fileChooser . showOpenDialog ( parentFrame ) ;


<nb> public class HttpConnection implements Connection {
 throw new UnsupportedMimeTypeException ( STRING0 ,
 contentType , req . url ( ) . toString ( ) ) ;

 + if ( contentType . startsWith ( STRING1 ) || xmlContentTypeRxp . matcher ( contentType ) . matches ( ) )
 + req . parser ( Parser . xmlParser ( ) ) ;
 +
 res . charset = DataUtil . getCharsetFromContentType ( res . contentType ) ; SINGLE
 if ( conn . getContentLength ( ) != NUMBER0 ) { SINGLE
 InputStream bodyStream = null ;

Use the XML parser if the MIME type matches an XML type


<nb> public class XCameraDevice2 extends XHook {
 }

 public String getClassName ( ) {
 - return STRING0 ;
 + return STRING1 ;
 }

 SINGLE


<nb> import NAMESPACE ;
 import NAMESPACE ;

 import NAMESPACE ;
 +
 import NAMESPACE ;

 public class IssueUpdater implements BatchComponent , ServerComponent {
 <nb> public class IssueUpdater implements BatchComponent , ServerComponent {
 return false ;
 }

 - public DefaultIssue setManualSeverity ( DefaultIssue issue , String severity ) {
 + public boolean setManualSeverity ( DefaultIssue issue , String severity ) {
 if ( ! issue . manualSeverity ( ) || ! Objects . equal ( severity , issue . severity ( ) ) ) {
 issue . setDiff ( STRING0 , issue . severity ( ) , severity ) ;
 issue . setSeverity ( severity ) ;
 issue . setManualSeverity ( true ) ;
 + return true ;
 }
 - return issue ;
 + return false ;
 }

 public boolean assign ( DefaultIssue issue , @ nullable String assignee ) {

Fix IssueUpdater due to previous commit that forgot some files


<nb> public class Capsule implements Runnable , InvocationHandler {
 COMMENT
 COMMENT
 COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 protected JMXServiceURL startJMXServer ( ) {
 COMMENT

Deprecate startJMXServer to exclude it from JavaDoc ci skip


<nb> public class Help extends BasePresenter implements ShowHelpHandler
 view_ = view ;
 globalDisplay_ = globalDisplay ;
 events_ = events ;
 - commands_ = commands ;

 binder . bind ( commands , this ) ;

 <nb> public class Help extends BasePresenter implements ShowHelpHandler
 private boolean historyInitialized_ ;
 private GlobalDisplay globalDisplay_ ;
 private EventBus events_ ;
 - private Commands commands_ ;
 }
 <nb> import NAMESPACE
 import NAMESPACE ;
 import NAMESPACE ;

 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;

 public class PresentationDispatcher


<nb> public final class PermissionCheckTest {
 }

 public static class FakeUserGroupsMapping implements GroupMappingService {
 - private HashMap < String , String > mUserGroups = new HashMap < String , String > ( ) ;
 + private HashMap < String , String > mUserGroups = new HashMap < > ( ) ;

 public FakeUserGroupsMapping ( ) {
 mUserGroups . put ( TEST_USER_ADMIN . getUser ( ) , TEST_USER_ADMIN . getGroups ( ) ) ;

SMALLFIX Removed explicit type argument in PermissionCheckTest


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class BasicMapTest extends HazelcastTestSupport {
 }

 @ test
 - @ category ( ProblematicTest . class )
 public void testMapLoaderLoadUpdatingIndex ( ) throws Exception {
 MapConfig mapConfig = getInstance ( ) . getConfig ( ) . getMapConfig ( STRING0 ) ;
 List < MapIndexConfig > indexConfigs = mapConfig . getMapIndexConfigs ( ) ;

moved problematic marker for testMapLoaderLoadUpdatingIndex


<nb> public class TupleInfo
 public enum Type
 {
 FIXED_INT_64 ( SIZE_OF_LONG ) ,
 - DOUBLE ( SIZE_OF_DOUBLE ) ,
 - VARIABLE_BINARY ( - NUMBER0 ) ;
 + VARIABLE_BINARY ( - NUMBER0 ) ,
 + DOUBLE ( SIZE_OF_DOUBLE ) ;

 private final int size ;

Reorder TypeInfo Type enum so ordinals back to orginal value


<nb> public final class TieredBlockStoreTests {
 }

 @ test
 - public void moveNonExistingBlock ( ) throws Exception {
 + public void moveNonExistingBlockTest ( ) throws Exception {
 mThrown . expect ( NotFoundException . class ) ;
 mThrown . expectMessage ( STRING0 + BLOCK_ID1 + STRING1 ) ;

Changed test method name to be consistent


<nb> public class PooledDataSource implements DataSource {

 protected void finalize ( ) throws Throwable {
 forceCloseAll ( ) ;
 + super . finalize ( ) ;
 }

 public < T > T unwrap ( Class < T > iface ) throws SQLException {


<nb> import NAMESPACE ;
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 <nb> import NAMESPACE ;
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 public interface Promise < T > extends SuccessPromise < T > {

Add mention of the exec harness for promises


<nb> public abstract class DvcsCommitAdditionalComponent implements RefreshableOnComp

 @ nullable
 protected abstract String getLastCommitMessage ( @ notnull VirtualFile repo ) throws VcsException ;
 +
 + public boolean isAmend ( ) {
 + return myAmend . isSelected ( ) ;
 + }
 }
 <nb> public class CommitChangeListDialog extends DialogWrapper implements CheckinProj
 ProjectLevelVcsManager . getInstance ( project ) . getAllActiveVcss ( ) ) ;
 }

 + @ notnull
 + public List < RefreshableOnComponent > getAdditionalComponents ( ) {
 + return Collections . unmodifiableList ( myAdditionalComponents ) ;
 + }
 +
 public static void commitPaths ( final Project project , Collection < FilePath > paths , final LocalChangeList initialSelection ,
 @ nullable final CommitExecutor executor , final String comment ) {
 final ChangeListManager manager = ChangeListManager . getInstance ( project ) ;

extend API to add possibility for checkin handlers check if it is amending operation


<nb> public abstract class LocalWorld {
 return false ;
 }

 - private class QueuedEffect implements Comparable < QueuedEffect > {
 + private class QueuedEffect implements Comparable < QueuedEffect > {
 private final Vector position ;
 private final int blockId ;
 private final double priority ;
 <nb> public abstract class Node implements RValue {
 }

 @ override
 - public int getPosition ( ) {
 + public final int getPosition ( ) {
 return position ;
 }
 }

Made Node getPosition final and fixed some whitespace in LocalWorld


<nb> public class PyIntroduceParameterHandler extends IntroduceHandler {
 }
 } . visitElement ( element ) ;
 }
 - return function != null && ! isResolvedToParameter ( element ) && isValid [ NUMBER0 ] ;
 + final PyStatement nonlocalStatement =
 + PsiTreeUtil . getParentOfType ( element , PyNonlocalStatement . class , PyGlobalStatement . class ) ;
 + return function != null && ! isResolvedToParameter ( element ) && isValid [ NUMBER0 ] && nonlocalStatement == null ;
 }
 return false ;
 }

disable refactoring for nonlocal and global statements


<nb> public class MapRouteInfoControl extends MapControls implements IRouteInformatio
 if ( selectFromMapTouch ) {
 LatLon latlon = tileBox . getLatLonFromPixel ( point . x , point . y ) ;
 selectFromMapTouch = false ;
 - SINGLE
 + SINGLE
 if ( selectFromMapForTarget ) {
 getTargets ( ) . navigateToPoint ( latlon , true , - NUMBER0 ) ;
 } else {

amend bug description comment with latest test results


<nb> public class ImmutableEnumChecker extends BugChecker implements ClassTreeMatcher
 this ,
 state ,
 STRING0 ,
 - STRING1 )
 + STRING2 )
 . checkForImmutability ( Optional . of ( tree ) , ImmutableSet . of ( ) , getType ( tree ) ) ;

 if ( ! info . isPresent ( ) ) {
 <nb> public class ImmutableEnumCheckerTest {
 STRING3 ,
 STRING4 ,
 STRING5 ,
 - STRING6 ,
 + STRING7
 + + STRING8 ,
 STRING9 ,
 STRING10 ,
 STRING11 ,

Update warning message for ImmutableEnumChecker


<nb> public class IocVfs {
 }

 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + public static void initWithoutPassword ( Activity activity ) {
 + init ( activity , null ) ;
 + }
 + COMMENT
 COMMENT
 COMMENT
 public static void init ( Context context , String password ) throws IllegalArgumentException {
 <nb> public class LockScreenActivity extends ThemeableActivity implements ICacheWordS
 if ( passphrase . isEmpty ( ) ) {
 SINGLE
 if ( Imps . setEmptyPassphrase ( this , false ) ) {
 - IocVfs . init ( this , STRING0 ) ;
 + IocVfs . initWithoutPassword ( this ) ;
 SINGLE
 afterCacheWordOpened ( ) ;
 } else {
 <nb> public class WelcomeActivity extends ThemeableActivity implements ICacheWordSubs
 else
 {
 if ( openEncryptedStores ( null , false ) ) {
 - IocVfs . init ( this , STRING0 ) ;
 + IocVfs . initWithoutPassword ( this ) ;
 } else {
 connectToCacheWord ( ) ; SINGLE
 }

create IocVfs initWithoutPassword Context for when no password is set


<nb> public class IdeaDependenciesProvider {
 Lists . newArrayList ( new IdeaScopeMappingRule ( STRING0 ) , new IdeaScopeMappingRule ( STRING1 ) ) ) ;
 }

 - Set < Dependency > provide ( final IdeaModule ideaModule ) {
 + public Set < Dependency > provide ( final IdeaModule ideaModule ) {
 getPath = new Transformer < FilePath , File > ( ) {
 @ nullable
 public FilePath transform ( File file ) {
 <nb> public class IdeaDependenciesProvider {
 return result ;
 }

 - Collection < UnresolvedIdeRepoFileDependency > getUnresolvedDependencies ( IdeaModule ideaModule ) {
 + public Collection < UnresolvedIdeRepoFileDependency > getUnresolvedDependencies ( IdeaModule ideaModule ) {
 return dependenciesExtractor . unresolvedExternalDependencies ( ideaConfigurations ( ideaModule ) , Collections . < Configuration > emptyList ( ) ) ;
 }

Make entry methods of IdeaDependenciesProvider public


<nb> public class ShardCompactionManager
 {
 TableMetadata metadata = getTableMetadata ( tableId ) ;
 List < ShardInfo > newShards = performCompaction ( transactionId , bucketNumber , shardUuids , metadata ) ;
 - shardManager . replaceShardUuids ( transactionId , tableId , metadata . getColumns ( ) , shardUuids , newShards ) ;
 log . info ( STRING0 , shardUuids , newShards . stream ( ) . map ( ShardInfo : : getShardUuid ) . collect ( toList ( ) ) ) ;
 + shardManager . replaceShardUuids ( transactionId , tableId , metadata . getColumns ( ) , shardUuids , newShards ) ;
 }

 private List < ShardInfo > performCompaction ( long transactionId , OptionalInt bucketNumber , Set < UUID > shardUuids , TableMetadata tableMetadata )

Log compaction before we commit the transaction


<nb> public class DeviceDiscoveryService extends Service {

 List < ScanFilter > filters = new ArrayList < > ( ) ;

 - ScanFilter . Builder builder = new ScanFilter . Builder ( )
 + ScanFilter filter = new ScanFilter . Builder ( )
 . setServiceData ( UriBeacon . URI_SERVICE_UUID ,
 new byte [ ] { } ,
 - new byte [ ] { } ) ;
 - filters . add ( builder . build ( ) ) ;
 + new byte [ ] { } )
 + . build ( ) ;
 +
 + filters . add ( filter ) ;

 boolean started = getLeScanner ( ) . startScan ( filters , settings , mScanCallback ) ;
 Log . v ( TAG , started ? STRING0 : STRING1 ) ;

Initialize the filter directly instead of initializing the builder first


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 + import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> class NotificationManagerService extends INotificationManager . Stub
 mSound . setUsesWakeLock ( context ) ;
 mToastQueue = new ArrayList < ToastRecord > ( ) ;
 mHandler = new WorkerHandler ( ) ;
 +
 + StorageManager sm = ( StorageManager ) context . getSystemService ( Context . STORAGE_SERVICE ) ;
 + sm . registerListener ( new com . android . internal . app . StorageNotification ( context ) ) ;
 +
 mStatusBarService = statusBar ;
 statusBar . setNotificationCallbacks ( mNotificationCallbacks ) ;


<nb> public class UpdateEventHandler implements ISVNEventHandler {
 else if ( event . getContentsStatus ( ) == SVNStatusType . UNCHANGED &&
 ( event . getPropertiesStatus ( ) == SVNStatusType . UNCHANGED || event . getPropertiesStatus ( ) == SVNStatusType . UNKNOWN ) ) {
 myText2 = SvnBundle . message ( STRING0 , displayPath ) ;
 + } else if ( SVNStatusType . INAPPLICABLE . equals ( event . getContentsStatus ( ) ) &&
 + ( event . getPropertiesStatus ( ) == SVNStatusType . UNCHANGED || event . getPropertiesStatus ( ) == SVNStatusType . UNKNOWN ) ) {
 + myText2 = SvnBundle . message ( STRING0 , displayPath ) ;
 }
 else {
 myText2 = STRING1 ;

UpdateProject Info view shows all folders as not in repository


<nb> import NAMESPACE ;
 public class TSVSentenceIterator implements Iterator < Sentence > {

 COMMENT
 - enum SentenceField {
 + public enum SentenceField {
 ID ,
 DEPENDENCIES_STANFORD ,
 DEPENDENCIES_EXTRAS ,

Updated TSViterator and created an asrelations processor


<nb>
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 package NAMESPACE ;

 import static NAMESPACE ;
 <nb>
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 package NAMESPACE ;

 import static NAMESPACE ;


<nb> public class ZWaveConfiguration implements OpenHABConfigurationService , ZWaveEve

 if ( networkMonitor != null ) {
 record = new OpenHABConfigurationRecord ( domain , STRING0 , STRING1 , true ) ;
 - if ( node . getHealState ( ) == null )
 - record . value = STRING2 ;
 - else
 - record . value = node . getHealState ( ) ;
 -
 + record . value = networkMonitor . getNodeState ( nodeId ) ;
 records . add ( record ) ;
 }


<nb> public class DynamoBackedConfigurationTest {
 assertEquals ( STRING0 , test2 . get ( ) ) ;
 assertEquals ( STRING1 , test3 . get ( ) ) ;

 - Thread . sleep ( NUMBER0 ) ;
 + Thread . sleep ( NUMBER1 ) ;

 assertEquals ( STRING2 , test1 . get ( ) ) ;
 assertEquals ( STRING3 , test2 . get ( ) ) ;

Add longer sleep for timing sensitive test


<nb> public class MimeHeader {
 String v = field . value ;

 if ( hasToBeEncoded ( v ) ) {
 - v = EncoderUtil . encodeEncodedWord ( field . value , Charset . forName ( mCharset ) ) ;
 + Charset charset = null ;
 +
 + if ( mCharset != null ) {
 + charset = Charset . forName ( mCharset ) ;
 + }
 + v = EncoderUtil . encodeEncodedWord ( field . value , charset ) ;
 }

 writer . write ( field . name + STRING0 + v + STRING1 ) ;

Avoid IllegalArgumentException for Charset forName null


<nb> public class Wallet implements Serializable , BlockChainListener {
 newBlockHashes . add ( b . getHeader ( ) . getHash ( ) ) ;
 }

 - boolean affectedUs = false ;
 - for ( Sha256Hash hash : Iterables . concat ( oldBlockHashes , newBlockHashes ) ) {
 - if ( mapBlockTx . get ( hash ) != null ) {
 - affectedUs = true ;
 - break ;
 - }
 - }
 - log . info ( affectedUs ? STRING0 : STRING1 ) ;
 - if ( ! affectedUs ) return ;
 -
 SINGLE
 SINGLE
 onWalletChangedSuppressions ++ ;

remove the affectedUs check in reorganize


<nb> public class WindowManagerService extends IWindowManager . Stub
 try {
 mDimSurface = new Surface ( session , NUMBER0 , - NUMBER1 , NUMBER2 , NUMBER2 , PixelFormat . OPAQUE ,
 Surface . FX_SURFACE_DIM ) ;
 + mDimSurface . setAlpha ( FLOAT0 ) ;
 } catch ( Exception e ) {
 Log . e ( TAG , STRING0 , e ) ;
 }

Set alpha value for newly created dim surface


<nb> public class NotificationsPendingDraftsService extends Service {
 } else if ( draftPostsOlderThan3Days . size ( ) > NUMBER0 ) {
 long longestLivingDraft = NUMBER1 ;
 boolean onlyPagesFound = true ;
 + boolean doShowNotification = false ;
 for ( Post post : draftPostsOlderThan3Days ) {

 SINGLE
 if ( ( now - post . getDateLastNotified ( ) ) > MINIMUM_ELAPSED_TIME_BEFORE_REPEATING_NOTIFICATION ) {
 - if ( post . getDateLastNotified ( ) > longestLivingDraft ) {
 +
 + if ( post . getDateLastNotified ( ) >= longestLivingDraft ) {
 + doShowNotification = true ;
 longestLivingDraft = post . getDateLastNotified ( ) ;
 if ( ! post . isPage ( ) ) {
 onlyPagesFound = false ;
 <nb> public class NotificationsPendingDraftsService extends Service {

 SINGLE
 SINGLE
 - if ( longestLivingDraft > NUMBER1 ) {
 + if ( doShowNotification ) {
 buildPendingDraftsNotification ( draftPostsOlderThan3Days , onlyPagesFound ) ;
 }
 }

made use of an explicit flag to know when the condition to show a notification is met


<nb> public class RelativeLayout extends ViewGroup {
 SINGLE
 width += mPaddingRight ;

 - if ( mLayoutParams . width >= NUMBER0 ) {
 + if ( mLayoutParams != null && mLayoutParams . width >= NUMBER0 ) {
 width = Math . max ( width , mLayoutParams . width ) ;
 }

 <nb> public class RelativeLayout extends ViewGroup {
 SINGLE
 height += mPaddingBottom ;

 - if ( mLayoutParams . height >= NUMBER0 ) {
 + if ( mLayoutParams != null && mLayoutParams . height >= NUMBER0 ) {
 height = Math . max ( height , mLayoutParams . height ) ;
 }

Check RelativeLayout s access of its own LayoutParams during measurement


<nb> public class Transformer implements ClassFileTransformer
 byte [ ] bytes = cw . toByteArray ( ) ;
 {
 SINGLE
 + SINGLE
 }
 + return bytes ;
 + }
 +
 + private void debugSave ( final ClassNode classNode , final byte [ ] bytes )
 + {
 try
 {
 Path path = Paths . get ( STRING0 + classNode . name + STRING1 ) ;
 Files . createDirectories ( path . getParent ( ) ) ;
 Files . write ( path , bytes ) ;
 -
 }
 catch ( IOException e )
 {
 e . printStackTrace ( ) ;
 }
 - return bytes ;
 }

 -
 COMMENT
 COMMENT
 COMMENT


<nb> public class HTreeMap2Test {
 assertEquals ( ZERO , engine . get ( m . expireHeads [ NUMBER0 ] , Serializer . LONG ) ) ;
 assertEquals ( ZERO , engine . get ( m . expireTails [ NUMBER0 ] , Serializer . LONG ) ) ;
 assertArrayEquals ( new int [ ] { } , getExpireList ( m , NUMBER0 ) ) ;
 + m . segmentLocks [ NUMBER0 ] . writeLock ( ) . unlock ( ) ;
 }

 @ test public void expire_link_test ( ) {
 <nb> public class HTreeMap2Test {

 assertEquals ( NUMBER1 , m . expireLinkRemove ( NUMBER2 , recids [ NUMBER3 ] ) . hash ) ;
 assertArrayEquals ( new int [ ] { NUMBER4 , NUMBER5 , NUMBER6 , NUMBER7 } , getExpireList ( m , NUMBER2 ) ) ;
 -
 + m . segmentLocks [ NUMBER2 ] . writeLock ( ) . unlock ( ) ;

 }

add unlocking to prevent failing threads


<nb> public class BTreeMapTest {
 new Object [ ] { null , NUMBER0 , NUMBER1 , NUMBER2 , null } ,
 NUMBER3 ) ;
 long rootRecid = engine . put ( l , m . nodeSerializer ) ;
 - long rootRecidRef = engine . put ( rootRecid , Serializer . LONG_SERIALIZER ) ;
 + engine . update ( m . rootRecidRef , rootRecid , Serializer . LONG_SERIALIZER ) ;

 assertEquals ( null , m . get ( NUMBER4 ) ) ;
 assertEquals ( null , m . get ( NUMBER5 ) ) ;


<nb> public final class PrintManagerService extends IPrintManager . Stub {
 . queryIntentServicesAsUser ( intent , PackageManager . GET_SERVICES ,
 getChangingUserId ( ) ) ;

 + if ( installedServices == null ) {
 + return ;
 + }
 +
 final int installedServiceCount = installedServices . size ( ) ;
 for ( int i = NUMBER0 ; i < installedServiceCount ; i ++ ) {
 ServiceInfo serviceInfo = installedServices . get ( i ) . serviceInfo ;

PackageManager queryIntentServices breaks its contract


<nb> public class OpenStreetMapGraphBuilderImpl implements GraphBuilder {
 continue ;
 }
 if ( node == null ) {
 - throw new RuntimeException ( STRING0 ) ;
 + throw new RuntimeException ( STRING1 + nodeRef + STRING2 + areaWay . getId ( ) + STRING3 ) ;
 }
 Point point = new Point ( node . getLon ( ) , node . getLat ( ) ) ;
 nodes . add ( node ) ;


<nb> public class Etc1Texture extends ACompressedTexture {
 try {
 texture = ETC1Util . createTexture ( compressedTexture ) ;
 } catch ( IOException e ) {
 - Log . e ( STRING0 , e . getMessage ( ) ) ;
 + RajLog . e ( STRING0 , e . getMessage ( ) ) ;
 } finally {
 if ( texture == null ) {
 setBitmap ( fallbackTexture ) ;
 - Log . d ( STRING1 , STRING2 ) ;
 + RajLog . d ( STRING1 , STRING2 ) ;
 } else {
 setByteBuffer ( texture . getData ( ) ) ;
 setWidth ( texture . getWidth ( ) ) ;
 setHeight ( texture . getHeight ( ) ) ;
 - Log . d ( STRING1 , STRING3 ) ;
 + RajLog . d ( STRING1 , STRING3 ) ;
 }
 }
 }


<nb> public class ObjectRange extends AbstractList implements Range {
 }

 private void constructorHelper ( Comparable from , Comparable to ) {
 + if ( from == null ) {
 + throw new IllegalArgumentException ( STRING0 ) ;
 + }
 + if ( to == null ) {
 + throw new IllegalArgumentException ( STRING1 ) ;
 + }
 if ( from . getClass ( ) == to . getClass ( ) ) {
 this . from = from ;
 this . to = to ;

added better argument checking in range construction


<nb> public final class PowerProvider {
 private SafeTimeTracker energyLossTracker = new SafeTimeTracker ( ) ;
 private SafeTimeTracker doWorkTracker = new SafeTimeTracker ( ) ;
 public final int [ ] powerSources = { NUMBER0 , NUMBER0 , NUMBER0 , NUMBER0 , NUMBER0 , NUMBER0 } ;
 - private final IPowerReceptor receptor ;
 + public final IPowerReceptor receptor ;

 public PowerProvider ( IPowerReceptor receptor ) {
 this ( receptor , true ) ;


<nb> public final class BridgeContext extends Context {
 }
 SINGLE
 if ( typeArrayAndPropertiesPair . getSecond ( ) != null ) {
 - Object key = getCurrentParser ( ) . getViewCookie ( ) ;
 + BridgeXmlBlockParser parser = getCurrentParser ( ) ;
 + Object key = parser != null ? parser . getViewCookie ( ) : null ;
 if ( key != null ) {
 PropertiesMap defaultPropMap = mDefaultPropMaps . get ( key ) ;
 if ( defaultPropMap == null ) {


<nb> import NAMESPACE ;
 public abstract class AnimationAdapter < T > extends ArrayAdapter < T > {

 private Context mContext ;
 + private Handler mHandler ;

 private ListView mListView ;

 <nb> public abstract class AnimationAdapter < T > extends ArrayAdapter < T > {
 public AnimationAdapter ( Context context , ArrayList < T > items ) {
 super ( items ) ;
 mContext = context ;
 + mHandler = new Handler ( ) ;

 mPreviousLastVisiblePosition = - NUMBER0 ;
 mAnimationStartMillis = - NUMBER0 ;
 <nb> public abstract class AnimationAdapter < T > extends ArrayAdapter < T > {
 animation . setStartTime ( mAnimationStartMillis ) ;
 animation . setStartOffset ( delayMillis ) ;

 - new Handler ( ) . postDelayed ( new Runnable ( ) {
 + mHandler . postDelayed ( new Runnable ( ) {

 @ override
 public void run ( ) {


<nb> public class SSLCertificateSocketFactory extends SSLSocketFactory {
 }
 } ;

 - private static final HostnameVerifier HOSTNAME_VERIFIER =
 - HttpsURLConnection . getDefaultHostnameVerifier ( ) ;
 -
 private SSLSocketFactory mInsecureFactory = null ;
 private SSLSocketFactory mSecureFactory = null ;
 private TrustManager [ ] mTrustManagers = null ;
 <nb> public class SSLCertificateSocketFactory extends SSLSocketFactory {
 if ( session == null ) {
 throw new SSLException ( STRING0 ) ;
 }
 - if ( ! HOSTNAME_VERIFIER . verify ( hostname , session ) ) {
 + if ( ! HttpsURLConnection . getDefaultHostnameVerifier ( ) . verify ( hostname , session ) ) {
 throw new SSLPeerUnverifiedException ( STRING1 + hostname ) ;
 }
 }

Use hostname verifier directly instead of instance


<nb> public class AttributeBinding extends BasePrimitiveBinding {
 throw new XmlSerializationException ( STRING0 + myAccessor ) ;
 }
 }
 +
 + public String toString ( ) {
 + return STRING1 + myName + STRING2 + myBinding + STRING3 ;
 + }
 }


<nb> public class CodeColorizersTest {
 CodeColorizers codeColorizers = new CodeColorizers ( Arrays . < CodeColorizerFormat > asList ( new JavaScriptColorizerFormat ( ) ) ) ;

 File fileWithBom = temp . newFile ( ) ;
 - FileUtils . write ( fileWithBom , STRING0 ) ;
 + FileUtils . write ( fileWithBom , STRING0 , STRING1 ) ;
 File jsFile = new File ( this . getClass ( ) . getResource ( STRING2 ) . toURI ( ) ) ;
 - FileUtils . write ( fileWithBom , FileUtils . readFileToString ( jsFile ) , true ) ;
 + FileUtils . write ( fileWithBom , FileUtils . readFileToString ( jsFile ) , STRING1 , true ) ;

 SyntaxHighlightingData syntaxHighlighting = codeColorizers . toSyntaxHighlighting ( fileWithBom , STRING1 , STRING3 ) ;


<nb> public class CallPeerSipImpl
 . soundLevelChanged ( evt ) ;
 }
 }
 -
 }

 COMMENT

Adds more code on dispatching audio level events


<nb> public class FileFilter {
 } ;

 static final String [ ] ignoreTestList = {
 + STRING0 , SINGLE
 STRING1 , SINGLE
 STRING2 , SINGLE
 STRING3 , SINGLE
 + STRING4 , SINGLE
 STRING5 , SINGLE
 STRING6 , SINGLE
 - STRING7 , SINGLE
 - SINGLE
 - STRING8 , SINGLE
 - SINGLE
 - STRING9 , SINGLE
 - SINGLE
 - STRING10 , SINGLE
 - SINGLE
 - STRING0 , SINGLE
 - SINGLE
 + STRING10 , SINGLE
 + STRING7 , SINGLE
 + STRING8 , SINGLE
 + STRING9 , SINGLE
 } ;

 static void fillIgnoreResultList ( ) {


<nb> public class Utils {
 COMMENT
 COMMENT
 public static int [ ] mapping ( int [ ] dom ) {
 - assert dom . length > NUMBER0 : STRING0 ;
 + if ( dom . length == NUMBER0 ) return new int [ ] { } ;
 assert dom [ NUMBER0 ] <= dom [ dom . length - NUMBER1 ] : STRING1 ;
 int min = dom [ NUMBER0 ] ;
 int max = dom [ dom . length - NUMBER1 ] ;

Permit empty domain in transforming into enum


<nb> public class ModuleDeleteProvider implements DeleteProvider , TitledHandler {

 @ override
 public String getActionTitle ( ) {
 - return ProjectAttachProcessor . canAttachToProject ( ) ? STRING0 : STRING1 ;
 + return ProjectAttachProcessor . canAttachToProject ( ) ? STRING2 : STRING1 ;
 }

 public static void removeModule ( @ notnull final Module moduleToRemove ,


<nb> public class kNNClusterer extends Clusterer {
 }

 public boolean visit ( Project project , int rowIndex , Row row , boolean contextual ) {
 - Cell cell = row . cells . get ( _colindex ) ;
 + Cell cell = row . getCell ( _colindex ) ;
 if ( cell != null && cell . value != null ) {
 Object v = cell . value ;
 String s = ( v instanceof String ) ? ( ( String ) v ) : v . toString ( ) . intern ( ) ;


<nb> public class SvnChangeList implements CommittedChangeList {
 }

 myCommonPathRoot = commonPathSearcher . getCommon ( ) ;
 -
 - updateCachedInfo ( ) ;
 }

 public SvnChangeList ( SvnVcs vcs , @ notnull SvnRepositoryLocation location , DataInput stream , final boolean supportsCopyFromInfo ) throws IOException {
 <nb> public class SvnChangeList implements CommittedChangeList {
 commonPathSearcher . next ( path ) ;
 }
 myCommonPathRoot = commonPathSearcher . getCommon ( ) ;
 - updateCachedInfo ( ) ;
 }

 private void uploadDeletedRenamedChildren ( final List < Change > out ) {

DirectoryIndexImpl checkAvailability SVN change list will go for VCS root only on request not in init stage


<nb> public class JavaInMemoryFileManager extends ForwardingJavaFileManager < StandardJ
 private void createDirectory ( String name ) throws IOException {
 JavaFileObject fileObject = createJavaMemoryFileObject ( name , JavaFileObject . Kind . OTHER ) ;
 jarFileSemaphore . acquireUninterruptibly ( ) ;
 - jarOutputStream . putNextEntry ( createEntry ( fileObject . getName ( ) ) ) ;
 - jarOutputStream . closeEntry ( ) ;
 - jarFileSemaphore . release ( ) ;
 + try {
 + jarOutputStream . putNextEntry ( createEntry ( fileObject . getName ( ) ) ) ;
 + jarOutputStream . closeEntry ( ) ;
 + } finally {
 + jarFileSemaphore . release ( ) ;
 + }
 }

 private JavaFileObject createJavaMemoryFileObject ( String path , JavaFileObject . Kind kind ) {


<nb> public class StudyDocumentListener extends DocumentAdapter {
 if ( myTaskWindow != null ) {
 int newLength = myTaskWindow . getLength ( ) + change ;
 myTaskWindow . setLength ( newLength <= NUMBER0 ? NUMBER0 : newLength ) ;
 - if ( e . getNewFragment ( ) . equals ( STRING0 ) ) {
 - myTaskWindow . setLength ( myTaskWindow . getLength ( ) + NUMBER1 ) ;
 - }
 }
 int newEnd = offset + event . getNewLength ( ) ;
 int newLine = document . getLineNumber ( newEnd ) ;


<nb> package NAMESPACE ;

 COMMENT
 COMMENT
 - COMMENT
 - COMMENT
 - COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT

Improved documentation of LifeCycleAwareChannelHandler


<nb> public class HttpRequest {
 return this ;
 if ( multipart )
 output . write ( STRING0 + BOUNDARY + STRING1 ) ;
 - output . flush ( ) ;
 if ( ignoreCloseExceptions )
 try {
 output . close ( ) ;

Remove explicit flush of request output stream


<nb> package NAMESPACE ;

 import NAMESPACE ;

 + SINGLE
 COMMENT
 COMMENT
 COMMENT
 <nb> class ThreadLocalRandom extends Random {

 private static final long serialVersionUID = - NUMBER0 ;
 }
 + SINGLE

Disable Checkstyle for ThreadLocalRandom


<nb> public final class Protocol {
 public static final String PUBSUB_NUMSUB = STRING0 ;
 public static final String PUBSUB_NUM_PAT = STRING1 ;

 + public static final byte [ ] BYTES_TRUE = toByteArray ( NUMBER0 ) ;
 + public static final byte [ ] BYTES_FALSE = toByteArray ( NUMBER1 ) ;
 +
 private Protocol ( ) {
 SINGLE
 }
 <nb> public final class Protocol {
 }

 public static final byte [ ] toByteArray ( final boolean value ) {
 - return toByteArray ( value ? NUMBER0 : NUMBER1 ) ;
 + return value ? BYTES_TRUE : BYTES_FALSE ;
 }

 public static final byte [ ] toByteArray ( final int value ) {

Introduce two byte array for boolean true and false to avoid to create same byte array repeatedly


<nb> public class PhoneStatusBarPolicy {
 int state = intent . getIntExtra ( BluetoothAdapter . EXTRA_STATE , BluetoothAdapter . ERROR ) ;
 mBluetoothEnabled = state == BluetoothAdapter . STATE_ON ;
 } else if ( action . equals ( BluetoothAdapter . ACTION_CONNECTION_STATE_CHANGED ) ) {
 - int state = intent . getIntExtra ( BluetoothAdapter . EXTRA_STATE ,
 + int state = intent . getIntExtra ( BluetoothAdapter . EXTRA_CONNECTION_STATE ,
 BluetoothAdapter . STATE_DISCONNECTED ) ;
 if ( state == BluetoothAdapter . STATE_CONNECTED ) {
 iconId = R . drawable . stat_sys_data_bluetooth_connected ;


<nb> public class CqlNativeStorage extends CqlStorage
 CqlConfigHelper . setInputMaxConnections ( conf , nativeMaxConnections ) ;
 if ( nativeMinSimultReqs != null )
 CqlConfigHelper . setInputMinSimultReqPerConnections ( conf , nativeMinSimultReqs ) ;
 - if ( nativeMinSimultReqs != null )
 - CqlConfigHelper . setInputMaxSimultReqPerConnections ( conf , nativeMaxSimultReqs ) ;
 if ( nativeConnectionTimeout != null )
 CqlConfigHelper . setInputNativeConnectionTimeout ( conf , nativeConnectionTimeout ) ;
 if ( nativeReadConnectionTimeout != null )


<nb> public class GCInspector

 if ( previousCount == null )
 previousCount = NUMBER0 L ;
 - if ( count == previousCount )
 + if ( count . equals ( previousCount ) )
 continue ;

 gccounts . put ( gc . getName ( ) , count ) ;


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 - SINGLE
 public class QueryBuilder implements ConvertibleToDocument {
 private final Document query ;
 private String currentKey ;


<nb> public final class QuasarInstrumentor {
 }

 boolean shouldInstrument ( String className ) {
 - System . out . println ( STRING0 + className ) ;
 className = className . replace ( STRING1 , STRING2 ) ;
 if ( className . startsWith ( STRING3 ) )
 return false ;
 <nb> public final class QuasarInstrumentor {
 return false ;
 if ( MethodDatabase . isJavaCore ( className ) )
 return false ;
 - System . out . println ( STRING4 ) ;
 return true ;
 }


<nb> public class JSONObject {
 }
 if ( object instanceof JSONObject || object instanceof JSONArray
 || NULL . equals ( object ) || object instanceof JSONString
 - || object instanceof Number || object instanceof Character
 - || object instanceof Boolean || object instanceof String
 - || object instanceof Enum ) {
 + || object instanceof Byte || object instanceof Character
 + || object instanceof Short || object instanceof Integer
 + || object instanceof Long || object instanceof Boolean
 + || object instanceof Float || object instanceof Double
 + || object instanceof String || object instanceof BigInteger
 + || object instanceof BigDecimal || object instanceof Enum ) {
 return object ;
 }


<nb> package NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 <nb> package NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;

 import NAMESPACE ;


<nb>
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 package NAMESPACE ;

 import NAMESPACE ;


<nb> public class WebRuntimeActivity extends Activity

 @ override
 public void onPageFinished ( WebView view , String url ) {
 - if ( mSplashScreen . getVisibility ( ) == View . VISIBLE ) {
 + if ( mSplashScreen != null && mSplashScreen . getVisibility ( ) == View . VISIBLE ) {
 mSplashScreen . setVisibility ( View . GONE ) ;
 mSplashScreen = null ;
 }

Fix NPE in the webruntime after the splash screen has been dismissed


<nb> class DotComSiteSettings extends SiteSettingsInterface {
 mRemoteSettings . localTableId = mBlog . getRemoteBlogId ( ) ;
 deserializeDotComRestResponse ( mBlog , response ) ;
 if ( ! mRemoteSettings . equals ( mSettings ) ) {
 + SINGLE
 + final Map < String , String > currentPostFormats = mSettings . postFormats ;
 + final String [ ] currentPostFormatKeys = mSettings . postFormatKeys ;
 +
 mSettings . copyFrom ( mRemoteSettings ) ;
 +
 + mSettings . postFormats = currentPostFormats ;
 + mSettings . postFormatKeys = currentPostFormatKeys ;
 +
 SiteSettingsTable . saveSettings ( mSettings ) ;
 notifyUpdatedOnUiThread ( null ) ;
 }

Keep postFormats since api call doesn t return it


<nb> public class GroupReduceCompilationTest extends CompilerTestBase implements java

 SINGLE
 assertEquals ( DriverStrategy . ALL_GROUP_REDUCE , reduceNode . getDriverStrategy ( ) ) ;
 - assertEquals ( DriverStrategy . ALL_GROUP_REDUCE , combineNode . getDriverStrategy ( ) ) ;
 + assertEquals ( DriverStrategy . ALL_GROUP_COMBINE , combineNode . getDriverStrategy ( ) ) ;

 SINGLE
 assertEquals ( NUMBER0 , sourceNode . getDegreeOfParallelism ( ) ) ;


<nb> public final class SSL {

 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 COMMENT
 COMMENT
 COMMENT
 COMMENT
 - COMMENT
 + COMMENT
 COMMENT
 COMMENT
 public static native boolean hasOp ( int op ) ;


<nb> public class IndexCreator {

 SINGLE
 SINGLE
 - String file = STRING0 ;
 + String file = STRING1 ;
 int st = file . lastIndexOf ( STRING2 ) ;
 int e = file . indexOf ( STRING3 , st ) ;
 creator . setNodesDBFile ( new File ( STRING4 + file . substring ( st , e ) + STRING5 ) ) ;
 <nb> public class IndexRouteCreator extends AbstractIndexPartCreator {
 douglasPeukerSimplificationStep ( clusters ) ;

 SINGLE
 - removeSmall2RoadsConnectors ( clusters ) ;
 + SINGLE
 + SINGLE

 SINGLE


<nb> public abstract class SharedTreeModelBuilder < TM extends DTree . TreeModel > extends
 _validAdapted = true ;
 SINGLE
 if ( _validMap != null ) {
 + assert _modelMap != null : STRING0 ;
 String vr = model . responseName ( ) ;
 - _adaptedValidationResponse = validation . vec ( vr ) . makeTransf ( _validMap ) ; SINGLE
 + Vec tmp = validation . vec ( vr ) . toEnum ( ) ;
 + _adaptedValidationResponse = tmp . makeTransf ( _validMap ) ; SINGLE
 _toDeleteFrame . add ( STRING1 , _adaptedValidationResponse ) ; SINGLE
 + _toDeleteFrame . add ( STRING2 , tmp ) ;
 }
 }

Missing toEnum call for validation response


<nb> package NAMESPACE ;

 import NAMESPACE ;
 import NAMESPACE ;
 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 <nb> public class ProjectWizardStepFactoryImpl extends ProjectWizardStepFactory {
 }

 public ModuleWizardStep createNameAndLocationStep ( final WizardContext wizardContext ) {
 - return new ProjectNameStep ( wizardContext , null ) ;
 + return new ProjectNameStep ( wizardContext ) ;
 }

 COMMENT


<nb> public class OCommandExecutorSQLSelect extends OCommandExecutorSQLResultsetAbstr
 for ( OIdentifiable identifiable : indexResultSet ) {
 ORecord < ? > record = identifiable . getRecord ( ) ;
 SINGLE
 - if ( ( record != null ) && filter ( ( ORecordInternal < ? > ) record ) ) ) {
 + if ( ( record != null ) && filter ( ( ORecordInternal < ? > ) record ) ) {
 final boolean continueResultParsing = handleResult ( record ) ;
 if ( ! continueResultParsing )
 break ;


<nb> public class WalletProtobufSerializer {
 if ( extProto . getMandatory ( ) && requireMandatoryExtensions )
 throw new UnreadableWalletException ( STRING0 + id ) ;
 else
 - log . error ( STRING1 , id , e ) ;
 + log . error ( STRING2 , id , e ) ;
 }
 }
 }

Print full backtrace when a wallet extension fails to load


<nb> public class MavenPluginDomUtil {
 File file = MavenArtifactUtil . getArtifactFile ( MavenProjectsManager . getInstance ( p ) . getLocalRepository ( ) ,
 groupId , artifactId , version , STRING0 ) ;
 VirtualFile pluginFile = LocalFileSystem . getInstance ( ) . findFileByIoFile ( file ) ;
 - if ( pluginFile == null ) {
 - if ( ! file . exists ( ) ) return null ;
 -
 - pluginFile = LocalFileSystem . getInstance ( ) . refreshAndFindFileByIoFile ( file ) ;
 - if ( pluginFile == null ) return null ;
 - }
 + if ( pluginFile == null ) return null ;

 VirtualFile pluginJarRoot = JarFileSystem . getInstance ( ) . getJarRootForLocalFile ( pluginFile ) ;
 if ( pluginJarRoot == null ) return null ;

don t refresh plugin jar in MavenPluginDomUtil getPluginXmlFile


<nb> public abstract class ESSmokeClientTestCase extends LuceneTestCase {
 COMMENT
 public static final String TESTS_CLUSTER_DEFAULT = STRING0 ;

 - protected static ESLogger logger = ESLoggerFactory . getLogger ( ESSmokeClientTestCase . class . getName ( ) ) ;
 + protected static final ESLogger logger = ESLoggerFactory . getLogger ( ESSmokeClientTestCase . class . getName ( ) ) ;

 private static final AtomicInteger counter = new AtomicInteger ( ) ;
 private static Client client ;

make logger final so its not detected as a static leak


<nb> import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;

 - import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;
 import NAMESPACE ;


<nb> public final class AudioFormat implements Parcelable {
 public static final int ENCODING_AAC_HE_V2 = NUMBER0 ;
 COMMENT
 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 COMMENT
 public static final int ENCODING_IEC61937 = NUMBER1 ;

 <nb> public class AudioTrack implements AudioRouting
 }
 mSampleRate = sampleRateInHz ;

 + SINGLE
 + SINGLE
 + SINGLE
 + if ( audioFormat == AudioFormat . ENCODING_IEC61937
 + && channelConfig != AudioFormat . CHANNEL_OUT_STEREO ) {
 + throw new IllegalArgumentException (
 + STRING0 ) ;
 + }
 +
 SINGLE
 SINGLE
 mChannelConfiguration = channelConfig ;


<nb> public class SuperRecyclerView extends FrameLayout {
 mMoreProgress . setVisibility ( View . GONE ) ;
 }

 + public void setRefreshing ( boolean refreshing ) {
 + mPtrLayout . setRefreshing ( refreshing ) ;
 + }
 +
 COMMENT
 COMMENT
 COMMENT


<nb> public class ReactShadowNode extends CSSNode {
 return Assertions . assertNotNull ( mThemedContext ) ;
 }

 - protected void setThemedContext ( ThemedReactContext themedContext ) {
 + public void setThemedContext ( ThemedReactContext themedContext ) {
 mThemedContext = themedContext ;
 }

 <nb> public class UIImplementation {
 return mShadowNodeRegistry . getNode ( reactTag ) ;
 }

 + protected final ViewManager resolveViewManager ( String className ) {
 + return mViewManagers . get ( className ) ;
 + }
 +
 COMMENT
 COMMENT
 COMMENT
 <nb> public class ReactTextInputShadowNode extends ReactTextShadowNode implements
 }

 @ override
 - protected void setThemedContext ( ThemedReactContext themedContext ) {
 + public void setThemedContext ( ThemedReactContext themedContext ) {
 super . setThemedContext ( themedContext ) ;

 SINGLE


<nb> public class ThriftFactory < T > {
 private Amount < Long , Time > socketTimeout = null ;
 private Closure < Connection < TTransport , InetSocketAddress > > postCreateCallback = Closures . noop ( ) ;
 private StatsProvider statsProvider = Stats . STATS_PROVIDER ;
 - private Optional < String > endpointName ;
 + private Optional < String > endpointName = Optional . absent ( ) ;
 private String serviceName ;
 private boolean sslTransport ;

Initialize endpointName to an empty Optional holder


<nb> public abstract class BaseItem {
 return count ;
 }

 - public static int lengthOfUleb128 ( final int s ) {
 + public static int lengthOfUleb128 ( int value ) {
 + int remaining = value > > > NUMBER0 ;
 int length = NUMBER1 ;
 - if ( s > NUMBER2 ) {
 + while ( remaining != NUMBER3 ) {
 length ++ ;
 - if ( s > ( NUMBER4 ) ) {
 - length ++ ;
 - if ( s > ( NUMBER5 ) ) {
 - length ++ ;
 - if ( s > ( NUMBER6 ) ) {
 - length ++ ;
 - }
 - }
 - }
 + remaining > > >= NUMBER0 ;
 }
 return length ;
 }


<nb> public class AboutSettingsFragment extends Fragment {
 }

 private void viewUrl ( @ stringres int urlResId ) {
 - Utils . launchWebsite ( getActivity ( ) , getString ( urlResId ) , STRING0 , STRING1 ) ;
 + Utils . launchWebsite ( getActivity ( ) , getString ( urlResId ) ) ;
 }
 }


<nb> public class TaskAdapter extends CursorAdapter implements Filterable {
 new Class < ? > [ ] { Integer . TYPE , Integer . TYPE } ,
 R . anim . slide_left_in , R . anim . slide_left_out ) ;
 }
 - } else {
 + } else if ( ! viewHolder . task . getFlag ( Task . FLAGS , Task . FLAG_IS_READONLY ) ) {
 SINGLE
 IntentFilter filter = new IntentFilter ( AstridApiConstants . BROADCAST_REQUEST_ACTIONS ) ;
 filter . setPriority ( - NUMBER0 ) ;

Fixed a bug that could allow editing of readonly tasks suggested by Arne


<nb> public class DeepWaterModel extends Model < DeepWaterModel , DeepWaterParameters , Dee
 _mb . perRow ( new double [ ] { predFloats [ j ] } , actual , weight , NUMBER0 COMMENT , DeepWaterModel . this ) ;
 }
 }
 + if ( _makePreds ) {
 + Futures fs = new Futures ( ) ;
 + for ( int i = NUMBER0 ; i < vw . length ; ++ i )
 + vw [ i ] . close ( fs ) ;
 + fs . blockForPending ( ) ;
 + }

 obs += batch_size ;
 }


<nb> public class SingleRepositoryNode extends RepositoryNode {
 }

 @ override
 + public boolean isChecked ( ) {
 + return true ;
 + }
 +
 + @ override
 + public void fireOnSelectionChange ( boolean isSelected ) {
 + }
 +
 + @ override
 public void render ( @ notnull ColoredTreeCellRenderer renderer ) {
 renderer . setIcon ( myLoading . get ( ) ? myLoadingIcon : myEmptyIcon ) ;
 renderer . setIconOnTheRight ( false ) ;

space should not toggle invisible checkbox in single repository mode


<nb> public abstract class ExprNodeDesc implements Serializable , Node {

 SINGLE
 SINGLE
 - public static class ExprNodeDescEqualityWrapper {
 - private ExprNodeDesc exprNodeDesc ;
 + public final static class ExprNodeDescEqualityWrapper {
 + private final ExprNodeDesc exprNodeDesc ;
 + SINGLE
 + SINGLE
 + SINGLE
 + private final int hashcode ;

 public ExprNodeDescEqualityWrapper ( ExprNodeDesc exprNodeDesc ) {
 this . exprNodeDesc = exprNodeDesc ;
 + this . hashcode = exprNodeDesc == null ? NUMBER0 : exprNodeDesc . hashCode ( ) ;
 }

 public ExprNodeDesc getExprNodeDesc ( ) {
 return exprNodeDesc ;
 }

 - public void setExprNodeDesc ( ExprNodeDesc exprNodeDesc ) {
 - this . exprNodeDesc = exprNodeDesc ;
 - }
 -
 @ override
 public boolean equals ( Object other ) {

 <nb> public abstract class ExprNodeDesc implements Serializable , Node {

 @ override
 public int hashCode ( ) {
 - return exprNodeDesc == null ? NUMBER0 : exprNodeDesc . hashCode ( ) ;
 + return hashcode ;
 }

 COMMENT

Avoid right leaning tree hashCode depth in ExprNodeDescEqualityWrapper hashmaps Gopal V reviewed by Ashutosh Chauhan


<nb> public class ImmutableASTTransformation extends AbstractASTTransformation {

 private void createConstructorMap ( ClassNode cNode , List < PropertyNode > list , List < String > knownImmutableClasses , List < String > knownImmutables ) {
 final BlockStatement body = new BlockStatement ( ) ;
 + body . addStatement ( ifS ( equalsNullX ( varX ( STRING0 ) ) , assignS ( varX ( STRING0 ) , new MapExpression ( ) ) ) ) ;
 for ( PropertyNode pNode : list ) {
 body . addStatement ( createConstructorStatement ( cNode , pNode , knownImmutableClasses , knownImmutables ) ) ;
 }
 SINGLE
 body . addStatement ( stmt ( callX ( SELF_TYPE , STRING1 , args ( STRING2 , STRING0 ) ) ) ) ;
 - body . addStatement ( ifS ( equalsNullX ( varX ( STRING0 ) ) , assignS ( varX ( STRING0 ) , new MapExpression ( ) ) ) ) ;
 createConstructorMapCommon ( cNode , body ) ;
 if ( list . size ( ) > NUMBER0 ) {
 createNoArgConstructor ( cNode ) ;

minor refactor improve null handling case


<nb> public abstract class ImplTypeSchemaExtractionStrategySupport implements ModelSc
 Collection < Method > setterMethods = methodsByName . get ( setterName ) ;
 PropertyAccessorExtractionContext setterContext = ! setterMethods . isEmpty ( ) ? new PropertyAccessorExtractionContext ( setterMethods ) : null ;

 - ModelProperty < ? > property = extractPropertySchema ( extractionContext , propertyName , getterContext , setterContext , handledMethods ) ;
 + ModelProperty < ? > property = extractPropertySchema ( extractionContext , propertyName , getterContext , setterContext ) ;
 if ( property != null ) {
 properties . add ( property ) ;

 <nb> public abstract class ImplTypeSchemaExtractionStrategySupport implements ModelSc
 }

 @ nullable
 - private ModelProperty < ? > extractPropertySchema ( ModelSchemaExtractionContext < ? > extractionContext , String propertyName , PropertyAccessorExtractionContext getterContext , PropertyAccessorExtractionContext setterContext , Set < Method > handledMethods ) {
 + private < R > ModelProperty < ? > extractPropertySchema ( ModelSchemaExtractionContext < ? > extractionContext , String propertyName , PropertyAccessorExtractionContext getterContext , PropertyAccessorExtractionContext setterContext ) {
 SINGLE
 Method mostSpecificGetter = getterContext . getMostSpecificDeclaration ( ) ;
 if ( mostSpecificGetter . getParameterTypes ( ) . length != NUMBER0 ) {


<nb> public class Form {
 }

 COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + COMMENT
 + public void setDefaultAnswer ( String variable ) {
 + if ( ! isSubmitType ( ) ) {
 + throw new IllegalStateException ( STRING0 +
 + STRING1 ) ;
 + }
 + FormField field = getField ( variable ) ;
 + if ( field != null ) {
 + SINGLE
 + field . resetValues ( ) ;
 + SINGLE
 + for ( Iterator it = field . getValues ( ) ; it . hasNext ( ) ; ) {
 + field . addValue ( ( String ) it . next ( ) ) ;
 + }
 + }
 + else {
 + throw new IllegalArgumentException ( STRING2 ) ;
 + }
 + }
 +
 + COMMENT
 COMMENT
 COMMENT
 COMMENT

Added setDefaultAnswer String to set the default value as the current value of a given field