matthiasblaesing commented on a change in pull request #3868:
URL: https://github.com/apache/netbeans/pull/3868#discussion_r835746392
##########
File path:
ide/db.core/src/org/netbeans/modules/db/sql/history/SQLHistoryManager.java
##########
@@ -97,7 +101,7 @@ public int getListSize() {
return
NbPreferences.forModule(SQLHistoryPanel.class).getInt("OPT_SQL_STATEMENTS_SAVED_FOR_HISTORY",
DEFAULT_SQL_STATEMENTS_SAVED_FOR_HISTORY);
}
- protected FileObject getHistoryRoot(boolean create) throws IOException {
+ private FileObject getHistoryRoot(boolean create) throws IOException {
Review comment:
If I remember correctly there is exactly one platform, that does not use
"/" as separator, that is windows and I know, that windows also accepts forward
slashes. I have some hope that in the future people learn from past mistakes
and don't try to reinvent the wheel and either use URIs or at least use these
as base.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
