From a5e0ef57d7e8b9518c595b4ad3cdf005c7998b4a Mon Sep 17 00:00:00 2001 From: "csilv@chromium.org" Date: Thu, 21 Apr 2011 18:38:54 +0000 Subject: Cleanup: Fix some style issues in the printing code. Fix misuse of scoped_ptr. Patch created by arthurhsu@chromium.org BUG=none TEST=none Review URL: http://codereview.chromium.org/6880092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82523 0039d316-1c4b-4281-b951-d872f2087c98 --- printing/backend/print_backend_win.cc | 2 +- printing/printing_context_win.cc | 11 ++++------- 2 files changed, 5 insertions(+), 8 deletions(-) (limited to 'printing') diff --git a/printing/backend/print_backend_win.cc b/printing/backend/print_backend_win.cc index 480f4b8..d482a91 100644 --- a/printing/backend/print_backend_win.cc +++ b/printing/backend/print_backend_win.cc @@ -56,7 +56,7 @@ bool PrintBackendWin::EnumeratePrinters(PrinterList* printer_list) { NULL, 0, &bytes_needed, &count_returned); if (!bytes_needed) return false; - scoped_ptr printer_info_buffer(new BYTE[bytes_needed]); + scoped_array printer_info_buffer(new BYTE[bytes_needed]); ret = EnumPrinters(PRINTER_ENUM_LOCAL|PRINTER_ENUM_CONNECTIONS, NULL, 2, printer_info_buffer.get(), bytes_needed, &bytes_needed, &count_returned); diff --git a/printing/printing_context_win.cc b/printing/printing_context_win.cc index 8d6252d..884ae37 100644 --- a/printing/printing_context_win.cc +++ b/printing/printing_context_win.cc @@ -220,9 +220,8 @@ PrintingContext::Result PrintingContextWin::UseDefaultSettings() { &count_returned); if (ret && count_returned) { // have printers // Open the first successfully found printer. - for (DWORD count = 0; count < count_returned; count++) { - PRINTER_INFO_2* info_2; - info_2 = reinterpret_cast( + for (DWORD count = 0; count < count_returned; ++count) { + PRINTER_INFO_2* info_2 = reinterpret_cast( printer_info_buffer.get() + count * sizeof(PRINTER_INFO_2)); std::wstring printer_name = info_2->pPrinterName; if (info_2->pDevMode == NULL || printer_name.length() == 0) @@ -230,12 +229,10 @@ PrintingContext::Result PrintingContextWin::UseDefaultSettings() { if (!AllocateContext(printer_name, info_2->pDevMode, &context_)) break; if (InitializeSettings(*info_2->pDevMode, printer_name, - NULL, 0, false)) + NULL, 0, false)) { break; - if (context_) { - ::DeleteDC(context_); - context_ = NULL; } + ReleaseContext(); } if (context_) return OK; -- cgit v1.1