https://bz.apache.org/ooo/show_bug.cgi?id=126635
[email protected] changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] Attachment #85136| |review- Flags| | --- Comment #6 from [email protected] --- Comment on attachment 85136 --> https://bz.apache.org/ooo/attachment.cgi?id=85136 patch to main/sw/source/core/doc/poolfmt.cxx thank you for the patch but I wouldn't approve the current patch 1. use spaces instead of tabs https://wiki.openoffice.org/wiki/Cpp_Coding_Standards/FORMAT https://wiki.openoffice.org/wiki/Cpp_Coding_Standards/FORMAT/NoTabs 2. the new return path when pColl is NULL breaks functionality 3. as you noted in comment 2, lcl_SetNumBul is a helper of SwDoc::GetTextCollFromPool and the null check should be there if anywhere @AppChecker review requests in bugzilla get automatic send to dev-list this has nothing to do with the issue status it's done to highlight patches and faster review process -- You are receiving this mail because: You are the assignee for the issue.
