:aceman Assignee
	Description • 11 years ago

Defined as a variable in:

    mailnews/extensions/mailviews/src/nsMsgMailViewList.cpp (View Hg log or Hg annotations)
        line 139 -- m_mailViews->Count(aCount);
        line 151 -- m_mailViews->Count(&mailViewCount);
        line 154 -- return m_mailViews->QueryElementAt(aMailViewIndex, NS_GET_IID(nsIMsgMailView),
        line 163 -- m_mailViews->AppendElement(static_cast<nsISupports*>(aMailView));
        line 172 -- m_mailViews->RemoveElement(static_cast<nsISupports*>(aMailView));
        line 213 -- m_mailViews->Count(&mailViewCount); 
    mailnews/extensions/mailviews/src/nsMsgMailViewList.h (View Hg log or Hg annotations)
        line 49 -- nsCOMPtr<nsISupportsArray> m_mailViews; 

Referenced in:

    mailnews/extensions/mailviews/src/nsMsgMailViewList.cpp (View Hg log or Hg annotations)
        line 138 -- if (m_mailViews)
        line 148 -- NS_ENSURE_TRUE(m_mailViews, NS_ERROR_FAILURE);
        line 161 -- NS_ENSURE_TRUE(m_mailViews, NS_ERROR_FAILURE);
        line 170 -- NS_ENSURE_TRUE(m_mailViews, NS_ERROR_FAILURE);
        line 212 -- if (m_mailViews)
        line 274 -- ConvertFilterListToMailView(mFilterList, getter_AddRefs(m_mailViews));

	neil@parkwaycc.co.uk
	Comment 2 • 11 years ago

Comment on attachment 731722 [details] [diff] [review]
patch

>     // now convert the filter list into our mail view objects, stripping out just the info we need
>-    ConvertFilterListToMailView(mFilterList, getter_AddRefs(m_mailViews));
>+    ConvertFilterListToMailView(mFilterList, &m_mailViews);
If this is the only caller, then we shouldn't be passing members (compare ConvertMailViewListToFilterList). [Failing that, my preference would have been to pass the nsCOMArray by reference.]

	:aceman Assignee
	Comment 3 • 11 years ago

Thanks.

	neil@parkwaycc.co.uk
	Comment 4 • 11 years ago

Comment on attachment 732464 [details] [diff] [review]
patch v2

[aFilterList could have become mFilterList too]

	:aceman Assignee
	Comment 5 • 11 years ago

OK, done.

	Ryan VanderMeulen [:RyanVM]
	Comment 6 • 11 years ago

https://hg.mozilla.org/comm-central/rev/bbdfb610ba02