From f34e660f03beea74107ed957aca032358dc6b3e4 Mon Sep 17 00:00:00 2001
From: Stephen Gold <sgold@sonic.net>
Date: Sun, 17 May 2020 23:00:02 -0700
Subject: [PATCH] AppSettings: change the "VSync" default from false to true

---
 .../src/main/java/com/jme3/system/AppSettings.java | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/jme3-core/src/main/java/com/jme3/system/AppSettings.java b/jme3-core/src/main/java/com/jme3/system/AppSettings.java
index 7919a695f..07bbe5cdd 100644
--- a/jme3-core/src/main/java/com/jme3/system/AppSettings.java
+++ b/jme3-core/src/main/java/com/jme3/system/AppSettings.java
@@ -279,7 +279,7 @@ public final class AppSettings extends HashMap<String, Object> {
         defaults.put("AudioRenderer", LWJGL_OPENAL);
         defaults.put("DisableJoysticks", true);
         defaults.put("UseInput", true);
-        defaults.put("VSync", false);
+        defaults.put("VSync", true);
         defaults.put("FrameRate", -1);
         defaults.put("SettingsDialogImage", "/com/jme3/app/Monkey.png");
         defaults.put("MinHeight", 0);
@@ -862,10 +862,10 @@ public final class AppSettings extends HashMap<String, Object> {
     }
 
     /**
-     * Set to true to enable vertical-synchronization, limiting and synchronizing
-     * every frame rendered to the monitor's refresh rate.
-     * @param value
-     * (Default: false)
+     * Enable or disable vertical synchronization. If enabled, rendering will be
+     * synchronized with the display's refresh interval.
+     *
+     * @param value true to enable, false to disable (Default : true)
      */
     public void setVSync(boolean value) {
         putBoolean("VSync", value);
@@ -1042,7 +1042,9 @@ public final class AppSettings extends HashMap<String, Object> {
     }
 
     /**
-     * Get the vsync state
+     * Test whether vertical synchronization should be enabled.
+     *
+     * @return true for enabled, false for disabled
      * @see #setVSync(boolean)
      */
     public boolean isVSync() {