Skip to content

Commit 840ab7b

Browse files
committed
8263894: Convert defaultPrinter and printers fields to local variables
Reviewed-by: prr, azvegint, kizune
1 parent ba504fc commit 840ab7b

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/java.desktop/windows/classes/sun/print/PrintServiceLookupProvider.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,7 @@
4343

4444
public class PrintServiceLookupProvider extends PrintServiceLookup {
4545

46-
private String defaultPrinter;
4746
private PrintService defaultPrintService;
48-
private String[] printers; /* excludes the default printer */
4947
private PrintService[] printServices; /* includes the default printer */
5048

5149
static {
@@ -114,7 +112,7 @@ public synchronized PrintService[] getPrintServices() {
114112
}
115113

116114
private synchronized void refreshServices() {
117-
printers = getAllPrinterNames();
115+
String[] printers = getAllPrinterNames();
118116
if (printers == null) {
119117
// In Windows it is safe to assume no default if printers == null so we
120118
// don't get the default.
@@ -279,7 +277,7 @@ public synchronized PrintService getDefaultPrintService() {
279277

280278
// Windows does not have notification for a change in default
281279
// so we always get the latest.
282-
defaultPrinter = getDefaultPrinterName();
280+
String defaultPrinter = getDefaultPrinterName();
283281
if (defaultPrinter == null) {
284282
return null;
285283
}

0 commit comments

Comments
 (0)