Merge lp:~ted/libdbusmenu/lp757569 into lp:libdbusmenu/0.5
Proposed by
Ted Gould
Status: | Merged |
---|---|
Merged at revision: | 336 |
Proposed branch: | lp:~ted/libdbusmenu/lp757569 |
Merge into: | lp:libdbusmenu/0.5 |
Diff against target: |
13 lines (+2/-0) 1 file modified
libdbusmenu-glib/server.c (+2/-0) |
To merge this branch: | bzr merge lp:~ted/libdbusmenu/lp757569 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Andrea Cimitan (community) | Approve | ||
DBus Menu Team | Pending | ||
Review via email: mp+76799@code.launchpad.net |
Description of the change
Add checkes to protect against a NULL priv? Eh, protected.
To post a comment you must log in.
I was commenting the same, how it can be NULL? SERVER_ GET_PRIVATE( server) ", and that should give better performance too.
My suggestion for the future is to store a pointer to the private structure in the DbusmenuServer struct, so you can use "server->priv" instead "DBUSMENU_