Remove redundant code and improve string formatting

Several changes were made throughout different functions in the ImGui codes. A redundant itemSize() call was removed from the inputText method to improve code performance. TextureId formatting was expanded in the debugTools method to avoid truncation of longer Ids. Inconsistent usage of white spaces and indents were adjusted for standardization purpose. Several methods in DemoWindow and ShowDemoWindowWidgets files got their string representations adjusted for better readability and to avoid unintended syntax errors like unclosed quotations. A couple of unused imports were also removed from the ShowDemoWindowPopus file to clean up the code base. The for-loop in the same file was simplified for improving readability.

Author: elect86

core/src/main/kotlin/imgui/demo/DemoWindow.kt

@@ -295,15 +295,15 @@ object DemoWindow {
             }
 
             treeNode("Capture/Logging") {
-                textWrapped("""
+                helpMarker("""
                     The logging API redirects all text output so you can easily capture the content of a window or a block. Tree nodes can be automatically expanded.
-                    Try opening any of the contents below in this window and then click one of the \"Log To\" button.""".trimIndent())
+                    Try opening any of the contents below in this window and then click one of the "Log To" button.""".trimIndent())
                 logButtons()
 
                 helpMarker("You can also call ImGui::LogText() to output directly to the log without a visual output.")
-                if (button("Copy \"Hello, world!\" to clipboard")) {
+                if (button("""Copy "Hello, world!" to clipboard""")) {
                     logToClipboard()
-                    logText("%s", "Hello, world!")
+                    logText("Hello, world!")
                     logFinish()
                 }
             }

core/src/main/kotlin/imgui/demo/ShowDemoWindowPopus.kt

@@ -16,7 +16,6 @@ import imgui.ImGui.endMenu
 import imgui.ImGui.endMenuBar
 import imgui.ImGui.endPopup
 import imgui.ImGui.inputText
-import imgui.ImGui.isItemHovered
 import imgui.ImGui.mainViewport
 import imgui.ImGui.menuItem
 import imgui.ImGui.openPopup
@@ -29,7 +28,6 @@ import imgui.ImGui.setItemDefaultFocus
 import imgui.ImGui.setItemTooltip
 import imgui.ImGui.setNextItemWidth
 import imgui.ImGui.setNextWindowPos
-import imgui.ImGui.setTooltip
 import imgui.ImGui.text
 import imgui.ImGui.textEx
 import imgui.ImGui.textWrapped
@@ -160,7 +158,7 @@ object ShowDemoWindowPopups {
         operator fun invoke() {
             treeNode("Context menus") {
 
-                helpMarker("\"Context\" functions are simple helpers to associate a Popup to a given Item or Window identifier.")
+                helpMarker(""""Context" functions are simple helpers to associate a Popup to a given Item or Window identifier.""")
 
                 // BeginPopupContextItem() is a helper to provide common/simple popup behavior of essentially doing:
                 //     if (id == 0)
@@ -176,13 +174,13 @@ object ShowDemoWindowPopups {
                 // and BeginPopupContextItem() will use the last item ID as the popup ID.
                 run {
                     val names = listOf("Label1", "Label2", "Label3", "Label4", "Label5")
-                    for (n in 0..4) {
-                        selectable(names[n])
-                        if (selectable(names[n], selected == n))
+                    for (n in names.indices) {
+                        val name = names[n]
+                        if (selectable(name, selected == n))
                             selected = n
                         popupContextItem { // <-- use last item id as popup id
                             selected = n
-                            text("This a popup for \"${names[n]}\"!")
+                            text("""This a popup for "$name"!""")
                             if (button("Close"))
                                 closeCurrentPopup()
                         }

core/src/main/kotlin/imgui/demo/ShowDemoWindowWidgets.kt

@@ -2055,7 +2055,7 @@ object ShowDemoWindowWidgets {
                     |  ""         display all lines
                     |  "xxx"      display lines containing "xxx"
                     |  "xxx,yyy"  display lines containing "xxx" or "yyy"
-                    |  "-xxx"     hide lines containing "xxx"""")
+                    |  "-xxx"     hide lines containing "xxx"""".trimMargin())
                 filter.draw()
                 val lines = listOf("aaa1.c", "bbb1.c", "ccc1.c", "aaa2.cpp", "bbb2.cpp", "ccc2.cpp", "abc.h", "hello, world")
                 for (line in lines)

core/src/main/kotlin/imgui/internal/api/debugTools.kt

@@ -322,7 +322,7 @@ internal interface debugTools {
                 continue
             }
 
-            var buf = "DrawCmd:%5d tris, Tex 0x%02d, ClipRect (%4.0f,%4.0f)-(%4.0f,%4.0f)".format(
+            var buf = "DrawCmd:%5d tris, Tex 0x%016d, ClipRect (%4.0f,%4.0f)-(%4.0f,%4.0f)".format(
                 cmd.elemCount / 3, cmd.textureId, cmd.clipRect.x, cmd.clipRect.y, cmd.clipRect.z, cmd.clipRect.w)
             val pcmdNodeOpen = treeNode(drawList.cmdBuffer.indexOf(cmd), buf)
             if (isItemHovered() && (cfg.showDrawCmdMesh || cfg.showDrawCmdBoundingBoxes) /*&& fgDrawList != null*/)

core/src/main/kotlin/imgui/internal/api/inputText.kt

@@ -136,7 +136,6 @@ internal interface inputText {
             val backupPos = Vec2(window.dc.cursorPos)
             itemSize(totalBb, style.framePadding.y)
             if (!itemAdd(totalBb, id, frameBb, ItemFlag.Inputable)) {
-                itemSize(totalBb, style.framePadding.y)
                 endGroup()
                 return false
             }