On Thu, Jun 25, 2015 at 11:05:28AM +0200, Pavel Reichl wrote:
On 06/24/2015 06:46 PM, Michal Židek wrote:
>+static inline bool is_default_view(const char *view_name)
>+{
>+ /* NULL is treated as default */
>+ if (view_name == NULL
>+ || strcmp(view_name, SYSDB_DEFAULT_VIEW_NAME) == 0) {
>+ return true;
>+ } else {
>+ return false;
>+ }
>+}
>+
Do you think that
{
return view_name == NULL || strcmp(view_name,SYSDB_DEFAULT_VIEW_NAME) ==
0;
}
would be harder to read? (I'm just asking I'm not giving nack.)
To me, the explicit true and false returns are easier.