Opened 10 years ago
Closed 10 years ago
Last modified 10 years ago
#9054 closed Patch - Bug Fix (Fixed)
Remove deprecated isNull checks for QStrings
Reported by: | Owned by: | stuartm | |
---|---|---|---|
Priority: | minor | Milestone: | 0.25 |
Component: | MythTV - General | Version: | Master Head |
Severity: | low | Keywords: | |
Cc: | Ticket locked: | no |
Description
Attached patch removes unnecessary isNull checks from mythtv. I have compile tested and have not found any regressions with these changes.
Attachments (1)
Change History (4)
Changed 10 years ago by
Attachment: | remove_isNull_r26633.patch added |
---|
comment:1 Changed 10 years ago by
Owner: | set to stuartm |
---|---|
Status: | new → assigned |
comment:2 Changed 10 years ago by
Resolution: | → Fixed |
---|---|
Status: | assigned → closed |
(In [27401]) Remove a couple of redundant isNull() calls. isEmpty() returns true for null strings and strings are initialised to empty not null anyway. Patch by Ben Dailey. I've omitted the other changes in that patch, one was not as straightforward as it seems and the others were all in xmlparse.cpp which is deprecated code due for removal in 0.25. Closes #9054
comment:3 Changed 10 years ago by
Milestone: | unknown → 0.25 |
---|---|
Type: | patch → Patch - Bug Fix |
Note: See
TracTickets for help on using
tickets.
isNull removal patch against head r26633