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.)