chore: remove unnecessary comments

Author: ZigTag

examples/gauge/Components/List.tsx

@@ -27,32 +27,18 @@ interface ListProps extends ComponentProps {
   /** A function defining how to render each list item. */
   renderItem: (data: any, index: number) => VNode
 
-  /** A callback function to be called when a new item is selected. */
-  // onItemSelected?: (data: any, element: ScrollableControl | null, index: number) => void
-
-  /** The container used for scrolling. */
-  // scrollContainer?: NodeReference<HTMLElement>
-
   /** CSS class(es) to add to the root of the list component. */
   class?: string
 }
 
 /** The List component. */
 export class List extends DisplayComponent<ListProps> {
   private readonly _listContainer = FSComponent.createRef<HTMLElement>()
-  // private readonly _itemInstanceRefs: ScrollableControl[] = []
-
-  // private previousSelectedIndex = -1
-  // private previousSelectedItem: any = undefined
 
   /** @inheritdoc */
   public onAfterRender(): void {
     this.renderList()
-    // if (this.props.scrollContainer) {
-    //   this.scrollController.registerScrollContainer(this.props.scrollContainer.instance)
-    // }
     this.props.data.sub(this.onDataChanged.bind(this))
-    // this.scrollController.onScroll = this.onScroll.bind(this)
   }
 
   /**
@@ -73,7 +59,6 @@ export class List extends DisplayComponent<ListProps> {
           } else {
             this.addDomNode(item, index, el)
           }
-          // this.refreshRegistrations()
         }
         break
       case SubscribableArrayEventType.Removed:
@@ -85,16 +70,10 @@ export class List extends DisplayComponent<ListProps> {
           } else {
             this.removeDomNode(index)
           }
-          // this.refreshRegistrations()
         }
         break
       case SubscribableArrayEventType.Cleared:
-        // this._itemInstanceRefs.length = 0
         this._listContainer.instance.innerHTML = ""
-        // this.scrollController.resetCtrls()
-        // if (this.props.onItemSelected) {
-        //   this.props.onItemSelected(null, null, -1)
-        // }
         break
     }
   }
@@ -106,15 +85,8 @@ export class List extends DisplayComponent<ListProps> {
   private removeDomNode(index: number): void {
     const child = this._listContainer.instance.childNodes.item(index)
     this._listContainer.instance.removeChild(child)
-    // const removed = this._itemInstanceRefs.splice(index, 1)[0]
-    // removed.destroy()
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  // private registerListItem = (ctrl: UiControl): void => {
-  //   // noop here, we will refresh all registrations on our own
-  // }
-
   /**
    * Adds a list rendered dom node to the collection.
    * @param item Item to render and add.
@@ -129,145 +101,8 @@ export class List extends DisplayComponent<ListProps> {
       el = this._listContainer.instance
       node && el && FSComponent.render(node, el as any)
     }
-
-    // if (node !== undefined && node.instance !== null) {
-    //   this._itemInstanceRefs.splice(index, 0, node.instance as ScrollableControl)
-    // }
   }
 
-  /**
-   * Refreshs control registrations of all list elements.
-   */
-  // private refreshRegistrations(): void {
-  //   this.scrollController.resetCtrls()
-  //   for (let i = 0; i < this._itemInstanceRefs.length; i++) {
-  //     const instance = this._itemInstanceRefs[i]
-  //     if (instance instanceof UiControlGroup) {
-  //       this.register(instance)
-  //     } else if (instance instanceof UiControl) {
-  //       instance.props.onRegister = this.register
-  //       instance.onRegister()
-  //       this.register(instance);
-  //     }
-  //   }
-  //   if (this.getIsFocused()) {
-  //     if (
-  //       this.previousSelectedIndex < 0
-  //         ? this.scrollController.gotoFirst()
-  //         : this.previousSelectedIndex >= this.props.data.length
-  //           ? this.scrollController.gotoLast()
-  //           : this.scrollController.scrollTo("prev", this.previousSelectedIndex + 1)
-  //     ) {
-  //       return
-  //     } else {
-  //       this.previousSelectedIndex = -1
-  //     }
-  //   }
-  //   if (this.previousSelectedIndex < 0 || this.previousSelectedIndex >= this.props.data.length) {
-  //     this.previousSelectedIndex = -1
-  //   }
-  //   const item = this.props.data.tryGet(this.previousSelectedIndex) ?? null
-  //   if (item !== this.previousSelectedItem) {
-  //     this.props.onItemSelected &&
-  //       this.props.onItemSelected(
-  //         item,
-  //         this.getListItemInstance(this.previousSelectedIndex),
-  //         this.previousSelectedIndex,
-  //       )
-  //   }
-  //   this.previousSelectedItem = item
-  // }
-
-  /**
-   * Scrolls to an item.
-   * @param index is the index of the list item to scroll to.
-   */
-  // public scrollToIndex(index: number): void {
-  //   this.scrollController.gotoIndex(index)
-  // }
-
-  /**
-   * Ensures an indexed list item is in view.
-   * @param index The index of the list item.
-   */
-  //
-
-  /**
-   * Gets the data object related to the selected DOM element.
-   * @returns The selected item, if found.
-   */
-  // public getSelectedItem(): any | null {
-  //   const selectedIndex = this.getSelectedIndex()
-  //   if (selectedIndex > -1) {
-  //     return this.props.data.get(selectedIndex)
-  //   }
-  //   return null
-  // }
-
-  /**
-   * Get the selected HTMLElement.
-   * @returns The selected element, if found.
-   */
-  // public getSelectedElement(): HTMLElement | null {
-  //   return (this._listContainer.instance.children[this.getSelectedIndex()] as HTMLElement) ?? null
-  // }
-
-  /**
-   * Gets the index of the currently selected element.
-   * @returns Selected element index. Returns -1 if nothing found.
-   */
-  // public getSelectedIndex(): number {
-  //   const focusedControl = this.scrollController.getFocusedUiControl()
-  //   return focusedControl ? this._itemInstanceRefs.indexOf(focusedControl) : -1
-  // }
-
-  /**
-   * Gets the instance of the node at the specified index.
-   * @param index The index to get the instance for.
-   * @returns The node instance of specified type.
-   */
-  // public getListItemInstance<T>(index: number): T | null {
-  //   return (this._itemInstanceRefs[index] as unknown as T) ?? null
-  // }
-
-  /**
-   * Gets index of a item in the list by its node instance.
-   * @param nodeInstance The node instance to look for.
-   * @returns list item index
-   */
-  // public getListItemIndex(nodeInstance: ScrollableControl): number {
-  //   return this._itemInstanceRefs.indexOf(nodeInstance)
-  // }
-
-  /** @inheritdoc */
-  // protected onBlurred(): void {
-  //   if (this.previousSelectedItem && this.props.onItemSelected) {
-  //     this.props.onItemSelected(null, null, -1)
-  //   }
-
-  //   this.previousSelectedItem = null
-  //   this.previousSelectedIndex = -1
-  // }
-
-  /** @inheritdoc */
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  // protected onScroll(ctrl: ScrollableControl): void {
-  //   const el = this.getSelectedElement()
-  //   if (el !== null && this.props.scrollContainer) {
-  //     ScrollUtils.ensureInView(el, this.props.scrollContainer.instance)
-  //   }
-
-  //   const index = this.getSelectedIndex()
-  //   const item = this.props.data.tryGet(index) ?? null
-
-  //   if (this.props.onItemSelected) {
-  //     item !== this.previousSelectedItem && this.props.onItemSelected(item, this.getListItemInstance(index), index)
-  //   }
-
-  //   this.previousSelectedItem = item
-  //   this.previousSelectedIndex = index
-  // }
-
   /**
    * Renders a list item
    * @param dataItem The data item to render.
@@ -277,11 +112,6 @@ export class List extends DisplayComponent<ListProps> {
    */
   private renderListItem(dataItem: any, index: number): VNode {
     return this.props.renderItem(dataItem, index)
-    // if (renderedNode.instance instanceof UiControl || renderedNode.instance instanceof UiControlGroup) {
-    //   return renderedNode
-    // } else {
-    //   throw new Error("A ListItem must be of type UiControl or UiControlGroup!")
-    // }
   }
   /** Renders the list of data items. */
   private renderList(): void {
@@ -294,10 +124,8 @@ export class List extends DisplayComponent<ListProps> {
       const vnode = this.renderListItem(this.props.data.get(i), i)
       if (vnode !== undefined) {
         FSComponent.render(vnode, this._listContainer.instance)
-        // this._itemInstanceRefs.push(vnode.instance as ScrollableControl)
       }
     }
-    // this.refreshRegistrations()
   }
 
   /** @inheritdoc */

src/database/src/database.rs

@@ -568,5 +568,3 @@ impl DatabaseTrait for DatabaseV1 {
         Ok(gates_data.into_iter().map(PathPoint::from).collect())
     }
 }
-
-// Empty Connection

src/database/src/manual/database.rs

@@ -60,5 +60,3 @@ impl DatabaseTrait for DatabaseManual {
         Ok(header_data)
     }
 }
-
-// Empty Connection

src/database/src/util.rs

@@ -117,15 +117,3 @@ where
     }
     Ok(data)
 }
-
-// pub fn map_rows<T>(stmt: &mut rusqlite::Statement, params: [Rc<Vec<Value>>; 9]) -> Result<Vec<T>, Box<dyn Error>>
-// where
-//     T: for<'r> serde::Deserialize<'r>,
-// {
-//     let mut rows = stmt.query_and_then(params, |r| serde_rusqlite::from_row::<T>(r))?;
-//     let mut data = Vec::new();
-//     while let Some(row) = rows.next() {
-//         data.push(row.map_err(|e| e.to_string())?);
-//     }
-//     Ok(data)
-// }