This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "The Saros Eclipse Plugin". Pusher was dpp-gerrit.
The branch, master has been updated
via c0ff1a30bfa967739a8623e7186aba0957285324 (commit)
from 4082991802675f4794c36a4b622de6b7b33ab667 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit c0ff1a30bfa967739a8623e7186aba0957285324
Author: Matthias Bohnstedt <matthias.bohnst...@gmail.com>
Date: Thu Jun 11 15:26:02 2015 +0200
[REFACTOR][I] Changes "filesystem" package naming to match common
pattern
This commit change the naming for the fs package to filesystem, since
abbreviations are discourage in our code guidlines, and to have a common
naming over all our projects (Eclipse,Core,IntelliJ).
Furthermore every class that implements an core interfaces should use a
dedicated naming to avoid name confusion. Having Workspace.java in
Eclipse,IntelliJ, Netbeans, ... make it hard to see what workspace is
actual used.
Since our eclipse version already use a sufficient pattern to name those
core implementation this commits changes the InelltiJ file classes to
use the same name pattern. The pattern
is:[IDE-Name][ImplementedCoreClass][Impl]
F.E:
Core: IFolder
IDEA: IntelliJFolderImpl
Eclipse: EclipseFolderImpl
Netbeans: NetbeanFolderImpl
TODO: Make a Guideline of how the naming pattern should be.
Change-Id: I752f08ae84d459a03945186f334056fabc169695
Reviewed-on: http://saros-build.imp.fu-berlin.de/gerrit/2252
Tested-by: Jenkins CI
Reviewed-by: Bastian Sieker <bsie...@mail.uni-paderborn.de>
Reviewed-by: Stefan Rossbach <srossb...@arcor.de>
-----------------------------------------------------------------------
Summary of changes:
.../src/de/fu_berlin/inf/dpp/core/Saros.java | 4 +-
.../concurrent/ConsistencyWatchdogHandler.java | 2 +-
.../core/concurrent/ConsistencyWatchdogServer.java | 2 +-
.../dpp/core/editor/adapter/DocumentFactory.java | 2 +-
.../inf/dpp/core/ui/util/CollaborationUtils.java | 8 +-
.../context/SarosIntellijContextFactory.java | 4 +-
.../intellij/editor/LocalEditorManipulator.java | 3 +-
.../intellij/project/FileSystemChangeListener.java | 64 ++++++++++----------
.../intellij/project/SharedResourcesManager.java | 12 ++--
.../FileContentChangedNotifierBridge.java | 2 +-
.../project/{fs => filesystem}/FileImp.java | 6 +-
.../IntelliJFileResourceAttributesImpl.java} | 6 +-
.../IntelliJFolderImpl.java} | 10 ++--
.../IntelliJPathImpl.java} | 34 +++++-----
.../IntelliJProjectImpl.java} | 22 ++++----
.../IntelliJResourceAttributesImpl.java} | 8 +-
.../IntelliJWorkspaceImpl.java} | 18 +++---
.../project/{fs => filesystem}/PathFactory.java | 4 +-
.../{fs => filesystem}/ResourceConverter.java | 2 +-
.../project/{fs => filesystem}/ResourceImp.java | 24 ++++----
.../dpp/intellij/ui/menu/SarosFileShareGroup.java | 14 ++--
.../dpp/intellij/ui/menu/ShareWithUserAction.java | 4 +-
.../ui/wizards/AddProjectToSessionWizard.java | 6 +-
.../IntelliJFileImplTest.java} | 22 ++++---
.../IntelliJPathImplTest.java} | 14 +++--
25 files changed, 152 insertions(+), 145 deletions(-)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs =>
filesystem}/FileContentChangedNotifierBridge.java (98%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs =>
filesystem}/FileImp.java (96%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs/FileResourceAttributes.java
=> filesystem/IntelliJFileResourceAttributesImpl.java} (86%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs/FolderImp.java
=> filesystem/IntelliJFolderImpl.java} (91%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs/PathImp.java
=> filesystem/IntelliJPathImpl.java} (83%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs/ProjectImp.java
=> filesystem/IntelliJProjectImpl.java} (94%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs/ResourceAttributes.java
=> filesystem/IntelliJResourceAttributesImpl.java} (83%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs/Workspace.java
=> filesystem/IntelliJWorkspaceImpl.java} (83%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs =>
filesystem}/PathFactory.java (92%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs =>
filesystem}/ResourceConverter.java (97%)
rename
de.fu_berlin.inf.dpp.intellij/src/de/fu_berlin/inf/dpp/intellij/project/{fs =>
filesystem}/ResourceImp.java (87%)
rename
de.fu_berlin.inf.dpp.intellij/test/junit/de/fu_berlin/inf/dpp/intellij/project/{fs/FileImpTest.java
=> filesystem/IntelliJFileImplTest.java} (87%)
rename
de.fu_berlin.inf.dpp.intellij/test/junit/de/fu_berlin/inf/dpp/intellij/project/{fs/PathImpTest.java
=> filesystem/IntelliJPathImplTest.java} (83%)
hooks/post-receive
--
The Saros Eclipse Plugin
------------------------------------------------------------------------------
_______________________________________________
Dpp-robot mailing list
Dpp-robot@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dpp-robot
