On Sun, 2011-11-27 at 20:07 +0000, [email protected] wrote:
> - parent = tree_get_default_folder_node(hotlist_tree);
> + if (selected == true) {
> + parent = tree_get_selected_node(tree_get_root(hotlist_tree));
> + if (parent && (tree_node_is_folder == false)) {
tree_node_is_folder is a function. Comparing its address to false is
bogus (and generates compiler warnings here). Please make sure that when
committing changes to core code, they are warning-free. I would fix
this, but I've no understanding of what it is you're attempting to do
here.
> /**
> * Add an entry node.
> - */
> -void hotlist_add_entry(void)
> + *
> + * \param selected add the entry in the currently-selected node
> + */
> +void hotlist_add_entry(bool selected)
> {
> struct node *node, *parent;
> creating_node = true;
>
> - parent = tree_get_default_folder_node(hotlist_tree);
> + if (selected == true) {
> + parent = tree_get_selected_node(tree_get_root(hotlist_tree));
> + if (parent && (tree_node_is_folder == false)) {
Ditto.
J.
