No need to be this paranoid, checking for pointers being null once is enough! Checking twise in a row is just overkill.
git-svn-id: svn://10.0.0.236/trunk@106321 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
@@ -539,9 +539,6 @@ FrameManager::GetPrimaryFrameFor(nsIContent* aContent, nsIFrame** aResult)
|
||||
NS_ENSURE_TRUE(mPresShell, NS_ERROR_NOT_AVAILABLE);
|
||||
NS_ENSURE_ARG_POINTER(aResult);
|
||||
NS_ENSURE_ARG_POINTER(aContent);
|
||||
if (!aContent || !aResult) {
|
||||
return NS_ERROR_NULL_POINTER;
|
||||
}
|
||||
*aResult = nsnull; // initialize out param
|
||||
|
||||
nsresult rv;
|
||||
|
||||
@@ -539,9 +539,6 @@ FrameManager::GetPrimaryFrameFor(nsIContent* aContent, nsIFrame** aResult)
|
||||
NS_ENSURE_TRUE(mPresShell, NS_ERROR_NOT_AVAILABLE);
|
||||
NS_ENSURE_ARG_POINTER(aResult);
|
||||
NS_ENSURE_ARG_POINTER(aContent);
|
||||
if (!aContent || !aResult) {
|
||||
return NS_ERROR_NULL_POINTER;
|
||||
}
|
||||
*aResult = nsnull; // initialize out param
|
||||
|
||||
nsresult rv;
|
||||
|
||||
Reference in New Issue
Block a user