ODFstudio/odfstudio
2
0
Fork 0
Code Issues Packages Activity

sqliterator: Use rtl::Reference::get() instead of &*

Browse Source 
Using &* can be problematic if the pointer is null. In this case it
doesn’t matter, but using Reference::get() is probably easier to
understand anyway. This also paves the way for a later compiler plugin
to check for this.

Change-Id: I2361761496648eaaa1e245912ac19aa130b38863
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/211842
Tested-by: Jenkins
Reviewed-by: Neil Roberts <nroberts@libreoffice.org>
This commit is contained in:
Neil Roberts
2026-10-07 19:14:54 +02:00
parent bd60b650fe
commit e6176fc4f0
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
connectivity/source/parse/sqliterator.cxx
+1 -1
View File
@@ -769,7 +769,7 @@ void OSQLParseTreeIterator::getColumnRange( const OSQLParseNode* _pColumnRef,
OUString& _rTableRange, OUString& _rTableRange,
OUString& _out_rColumnAliasIfPresent ) const OUString& _out_rColumnAliasIfPresent ) const
{ {
lcl_getColumnRange( _pColumnRef, m_pImpl->m_xConnection, _rColumnName, _rTableRange, &*m_aSelectColumns, _out_rColumnAliasIfPresent ); lcl_getColumnRange( _pColumnRef, m_pImpl->m_xConnection, _rColumnName, _rTableRange, m_aSelectColumns.get(), _out_rColumnAliasIfPresent );
} }