Closed Bug 489612 Opened 16 years ago Closed 16 years ago

Update mozStoragePrivateHelpers.* to follow style guidelines

Categories

(Core :: SQLite and Embedded Database Bindings, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: sdwilsh, Assigned: sdwilsh)

Details

Attachments

(2 files)

Attached patch v1.0Splinter Review
No description provided.
Attachment #374094 - Flags: review?(bugmail)
Whiteboard: [needs review asuth]
Comment on attachment 374094 [details] [diff] [review] v1.0 >diff --git a/storage/src/mozStorageAsyncStatementExecution.cpp b/storage/src/mozStorageAsyncStatementExecution.cpp >--- a/storage/src/mozStorageAsyncStatementExecution.cpp >+++ b/storage/src/mozStorageAsyncStatementExecution.cpp >@@ -443,7 +443,7 @@ Connection::databaseElementExists(enum D > sqlite3_stmt *stmt; > int srv = ::sqlite3_prepare_v2(mDBConn, query.get(), -1, &stmt, NULL); > if (srv != SQLITE_OK) >- return ConvertResultCode(srv); >+ return convertResultCode(srv); 4-space should be 2-space >diff --git a/storage/src/mozStoragePrivateHelpers.h b/storage/src/mozStoragePrivateHelpers.h >--- a/storage/src/mozStoragePrivateHelpers.h >+++ b/storage/src/mozStoragePrivateHelpers.h >@@ -65,7 +68,9 @@ nsresult ConvertResultCode(int aSQLiteRe ... >+#endif // _mozStoragePrivateHelpters_h_ ^ no t in helper
Attachment #374094 - Flags: review?(bugmail) → review+
Whiteboard: [needs review asuth] → [needs new patch]
Attached patch v1.1Splinter Review
addresses review comments
Whiteboard: [needs new patch] → [can land]
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [can land]
Target Milestone: --- → mozilla1.9.2a1
Product: Toolkit → Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: