From 7b837cd707753919dea0c14207c11643f7a88e87 Mon Sep 17 00:00:00 2001 From: Lars Ingebrigtsen Date: Thu, 13 Jun 2019 18:01:42 +0200 Subject: [PATCH] Use CLOS-style accessors in oref/oset * lisp/cedet/srecode/texi.el (srecode-semantic-handle-:texitag): * lisp/cedet/srecode/table.el (srecode-mode-table-new): (srecode-dump): * lisp/cedet/srecode/srt-mode.el (srecode-parse-this-macro): (semantic-analyze-possible-completions): * lisp/cedet/srecode/semantic.el (srecode-compound-toString): (srecode-semantic-apply-tag-to-dict-default): * lisp/cedet/srecode/mode.el (srecode-minor-mode-templates-menu): (srecode-edit): * lisp/cedet/srecode/insert.el (srecode-insert-method): (srecode-inserter-apply-state): (srecode-insert-ask-default): (srecode-insert-method-ask): (srecode-insert-method-field): (srecode-insert-method-helper): (srecode-parse-input): (srecode-match-end): (srecode-insert-include-lookup): (srecode-insert-method): * lisp/cedet/srecode/find.el (srecode-template-table-in-project-p): (srecode-template-get-table): (srecode-template-get-table-for-binding): (srecode-all-template-hash): * lisp/cedet/srecode/extract.el (srecode-inserter-extract): (srecode-inserter-extract): * lisp/cedet/srecode/el.el (srecode-semantic-apply-tag-to-dict): * lisp/cedet/srecode/dictionary.el (initialize-instance): (srecode-dictionary-add-template-table): (srecode-compound-toString): * lisp/cedet/srecode/cpp.el (srecode-semantic-apply-tag-to-dict): * lisp/cedet/srecode/compile.el (srecode-compile-template-table): (srecode-dump): * lisp/cedet/srecode/args.el (srecode-semantic-handle-:project): * lisp/cedet/semantic/texi.el (semantic-analyze-possible-completions): * lisp/cedet/semantic/symref/list.el (semantic-symref-results-dump): (semantic-symref-list-create-macro-on-open-hit): (semantic-symref-list-rename-open-hits): (semantic-symref-list-map-open-hits): * lisp/cedet/semantic/symref/idutils.el (semantic-symref-perform-search): (semantic-symref-parse-tool-output-one-line): * lisp/cedet/semantic/symref/global.el (semantic-symref-perform-search): (semantic-symref-parse-tool-output-one-line): * lisp/cedet/semantic/symref/cscope.el (semantic-symref-perform-search): (semantic-symref-parse-tool-output-one-line): * lisp/cedet/semantic/mru-bookmark.el (semantic-mrub-ring-to-assoc-list): * lisp/cedet/semantic/grammar.el (semantic-analyze-possible-completions): * lisp/cedet/semantic/decorate/include.el (semantic-decoration-include-describe): * lisp/cedet/semantic/debug.el (semantic-debug-highlight-lexical-token): (semantic-debug-highlight-rule): * lisp/cedet/semantic/db-typecache.el (semanticdb-typecache-for-database): * lisp/cedet/semantic/complete.el (semantic-collector-calculate-completions-raw): * lisp/cedet/semantic/bovine/make.el (semantic-analyze-possible-completions): * lisp/cedet/semantic/analyze/refs.el (semantic-analyze-refs-impl): (semantic-analyze-refs-proto): * lisp/cedet/semantic/analyze/debug.el (semantic-analyzer-debug-describe-scope): * lisp/cedet/semantic/analyze.el (semantic-analyze-interesting-tag): (semantic-analyze-interesting-tag): (semantic-analyze-pulse): * lisp/cedet/ede/util.el (ede-update-version): * lisp/cedet/ede/shell.el (ede-shell-run-something): * lisp/cedet/ede/project-am.el (project-am-load-makefile): (project-rescan): (project-am-macro): (ede-buffer-mine): (project-compile-target-command): (project-am-package-info): * lisp/cedet/ede/files.el (ede--project-inode): (ede-directory-get-open-project): (ede-directory-get-toplevel-open-project): (ede-find-subproject-for-directory): (ede-toplevel-project): * lisp/cedet/ede/cpp-root.el (initialize-instance): (project-compile-project): (project-compile-target): * lisp/cedet/ede/config.el (ede-config-get-configuration): (project-debug-target): (project-run-target): (project-compile-project): (ede-preprocessor-map): (ede-java-classpath): * lisp/cedet/ede/base.el (ede-normalize-file/directory): Use CLOS-style oref/oset accessors instead of eieio-style :colon accessors. This avoids compilation warnings and is allegedly faster. --- lisp/cedet/ede/base.el | 12 ++--- lisp/cedet/ede/config.el | 14 +++--- lisp/cedet/ede/cpp-root.el | 16 +++--- lisp/cedet/ede/files.el | 18 +++---- lisp/cedet/ede/project-am.el | 67 +++++++++++++------------ lisp/cedet/ede/shell.el | 2 +- lisp/cedet/ede/util.el | 4 +- lisp/cedet/semantic/analyze.el | 10 ++-- lisp/cedet/semantic/analyze/debug.el | 2 +- lisp/cedet/semantic/analyze/refs.el | 4 +- lisp/cedet/semantic/bovine/make.el | 4 +- lisp/cedet/semantic/complete.el | 2 +- lisp/cedet/semantic/db-typecache.el | 2 +- lisp/cedet/semantic/debug.el | 4 +- lisp/cedet/semantic/decorate/include.el | 6 +-- lisp/cedet/semantic/grammar.el | 2 +- lisp/cedet/semantic/mru-bookmark.el | 2 +- lisp/cedet/semantic/symref/cscope.el | 21 ++++---- lisp/cedet/semantic/symref/global.el | 19 +++---- lisp/cedet/semantic/symref/idutils.el | 19 +++---- lisp/cedet/semantic/symref/list.el | 14 +++--- lisp/cedet/semantic/texi.el | 6 +-- lisp/cedet/srecode/args.el | 4 +- lisp/cedet/srecode/compile.el | 10 ++-- lisp/cedet/srecode/cpp.el | 2 +- lisp/cedet/srecode/dictionary.el | 8 +-- lisp/cedet/srecode/el.el | 2 +- lisp/cedet/srecode/extract.el | 14 +++--- lisp/cedet/srecode/find.el | 18 +++---- lisp/cedet/srecode/insert.el | 53 ++++++++++--------- lisp/cedet/srecode/mode.el | 10 ++-- lisp/cedet/srecode/semantic.el | 7 ++- lisp/cedet/srecode/srt-mode.el | 10 ++-- lisp/cedet/srecode/table.el | 22 ++++---- lisp/cedet/srecode/texi.el | 2 +- 35 files changed, 200 insertions(+), 212 deletions(-) diff --git a/lisp/cedet/ede/base.el b/lisp/cedet/ede/base.el index 41ab17f078d..57973ffd13b 100644 --- a/lisp/cedet/ede/base.el +++ b/lisp/cedet/ede/base.el @@ -628,15 +628,15 @@ instead of the current project." The other slot will be used to calculate values. PROJECT-FILE-NAME is a name of project file (short name, like `pom.xml', etc." (when (and (or (not (slot-boundp this :file)) - (not (oref this :file))) + (not (oref this file))) (slot-boundp this :directory) - (oref this :directory)) - (oset this :file (expand-file-name project-file-name (oref this :directory)))) + (oref this directory)) + (oset this file (expand-file-name project-file-name (oref this directory)))) (when (and (or (not (slot-boundp this :directory)) - (not (oref this :directory))) + (not (oref this directory))) (slot-boundp this :file) - (oref this :file)) - (oset this :directory (file-name-directory (oref this :file)))) + (oref this file)) + (oset this directory (file-name-directory (oref this file)))) ) diff --git a/lisp/cedet/ede/config.el b/lisp/cedet/ede/config.el index c94fdabf50e..f4fc1c2832b 100644 --- a/lisp/cedet/ede/config.el +++ b/lisp/cedet/ede/config.el @@ -139,7 +139,7 @@ the directory isn't on the `safe' list, ask to add it to the safe list." (setq config nil)) (when (not config) - (let* ((top (oref proj :directory)) + (let* ((top (oref proj directory)) (fname (expand-file-name (oref proj config-file-basename) top)) (class (oref proj config-class)) (ignore-type nil)) @@ -258,7 +258,7 @@ programs from a project.") "Run the current project derived from TARGET in a debugger." (let* ((proj (ede-target-parent target)) (config (ede-config-get-configuration proj t)) - (debug (oref config :debug-command)) + (debug (oref config debug-command)) (cmd (read-from-minibuffer "Debug Command: " debug)) @@ -275,7 +275,7 @@ programs from a project.") "Run the current project derived from TARGET." (let* ((proj (ede-target-parent target)) (config (ede-config-get-configuration proj t)) - (run (concat "./" (oref config :run-command))) + (run (concat "./" (oref config run-command))) (cmd (read-from-minibuffer "Run (like this): " run))) (ede-shell-run-something target cmd))) @@ -306,7 +306,7 @@ This class brings in method overloads for for building.") "Compile the entire current project PROJ. Argument COMMAND is the command to use when compiling." (let* ((config (ede-config-get-configuration proj t)) - (comp (oref config :build-command))) + (comp (oref config build-command))) (compile comp))) (cl-defmethod project-compile-target ((obj ede-target-with-config-build) &optional command) @@ -370,7 +370,7 @@ the preprocessor map, and include paths.") filemap ) ;; Preprocessor files - (dolist (G (oref config :c-preprocessor-files)) + (dolist (G (oref config c-preprocessor-files)) (let ((table (semanticdb-file-table-object (ede-expand-filename root G)))) (when table @@ -379,7 +379,7 @@ the preprocessor map, and include paths.") (setq filemap (append filemap (oref table lexical-table))) ))) ;; The core table - (setq filemap (append filemap (oref config :c-preprocessor-table))) + (setq filemap (append filemap (oref config c-preprocessor-table))) filemap )) @@ -408,7 +408,7 @@ java class path.") (cl-defmethod ede-java-classpath ((proj ede-project-with-config-java)) "Return the classpath for this project." - (oref (ede-config-get-configuration proj) :classpath)) + (oref (ede-config-get-configuration proj) classpath)) ;; Local variables: ;; generated-autoload-file: "loaddefs.el" diff --git a/lisp/cedet/ede/cpp-root.el b/lisp/cedet/ede/cpp-root.el index 83474a44dd3..0a77692dd71 100644 --- a/lisp/cedet/ede/cpp-root.el +++ b/lisp/cedet/ede/cpp-root.el @@ -279,7 +279,7 @@ Each directory needs a project file to control it.") "Make sure the :file is fully expanded." ;; Add ourselves to the master list (cl-call-next-method) - (let ((f (expand-file-name (oref this :file)))) + (let ((f (expand-file-name (oref this file)))) ;; Remove any previous entries from the main list. (let ((old (eieio-instance-tracker-find (file-name-directory f) :directory 'ede-cpp-root-project-list))) @@ -292,8 +292,8 @@ Each directory needs a project file to control it.") (file-directory-p f)) (delete-instance this) (error ":file for ede-cpp-root-project must be a file")) - (oset this :file f) - (oset this :directory (file-name-directory f)) + (oset this file f) + (oset this directory (file-name-directory f)) (ede-project-directory-remove-hash (file-name-directory f)) ;; NOTE: We must add to global list here because these classes are not ;; created via the typical loader, but instead via calls from a .emacs @@ -301,7 +301,7 @@ Each directory needs a project file to control it.") (ede-add-project-to-global-list this) (unless (slot-boundp this 'targets) - (oset this :targets nil)) + (oset this targets nil)) )) ;;; SUBPROJ Management. @@ -455,8 +455,8 @@ This is for project include paths and spp source files." "Compile the entire current project PROJ. Argument COMMAND is the command to use when compiling." ;; we need to be in the proj root dir for this to work - (let* ((cmd (oref proj :compile-command)) - (ov (oref proj :local-variables)) + (let* ((cmd (oref proj compile-command)) + (ov (oref proj local-variables)) (lcmd (when ov (cdr (assoc 'compile-command ov)))) (cmd-str (cond ((stringp cmd) cmd) @@ -470,8 +470,8 @@ Argument COMMAND is the command to use when compiling." (cl-defmethod project-compile-target ((obj ede-cpp-root-target) &optional command) "Compile the current target OBJ. Argument COMMAND is the command to use for compiling the target." - (when (oref obj :project) - (project-compile-project (oref obj :project) command))) + (when (oref obj project) + (project-compile-project (oref obj project) command))) (cl-defmethod project-rescan ((this ede-cpp-root-project)) diff --git a/lisp/cedet/ede/files.el b/lisp/cedet/ede/files.el index fc74c0a07e9..a6a0af100f5 100644 --- a/lisp/cedet/ede/files.el +++ b/lisp/cedet/ede/files.el @@ -120,7 +120,7 @@ of the anchor file for the project." "Get the inode of the directory project PROJ is in." (if (slot-boundp proj 'dirinode) (oref proj dirinode) - (oset proj dirinode (ede--inode-for-dir (oref proj :directory))))) + (oset proj dirinode (ede--inode-for-dir (oref proj directory))))) (defun ede--inode-get-toplevel-open-project (inode) "Return an already open toplevel project that is managing INODE. @@ -159,7 +159,8 @@ If DIR is the root project, then it is the same." (when rootreturn (set rootreturn proj)) ;; Find subprojects. (when (and proj (if ede--disable-inode - (not (string= ft (expand-file-name (oref proj :directory)))) + (not (string= ft (expand-file-name + (oref proj directory)))) (not (equal inode (ede--project-inode proj))))) (setq ans (ede-find-subproject-for-directory proj ft))) ans)) @@ -175,8 +176,7 @@ If optional EXACT is non-nil, only return exact matches for DIR." (shortans nil)) (while (and all (not ans)) ;; Do the check. - (let ((pd (expand-file-name (oref (car all) :directory))) - ) + (let ((pd (expand-file-name (oref (car all) directory)))) (cond ;; Exact text match. ((string= pd ft) @@ -187,7 +187,7 @@ If optional EXACT is non-nil, only return exact matches for DIR." (setq shortans (car all)) ;; We already have a short answer, so see if pd (the match we found) ;; is longer. If it is longer, then it is more precise. - (when (< (length (oref shortans :directory)) + (when (< (length (oref shortans directory)) (length pd)) (setq shortans (car all)))) ) @@ -208,7 +208,7 @@ If optional EXACT is non-nil, only return exact matches for DIR." (setq shortans (car all)) ;; We already have a short answer, so see if pd (the match we found) ;; is longer. If it is longer, then it is more precise. - (when (< (length (expand-file-name (oref shortans :directory))) + (when (< (length (expand-file-name (oref shortans directory))) (length pd)) (setq shortans (car all)))) ))) @@ -228,7 +228,7 @@ If optional EXACT is non-nil, only return exact matches for DIR." proj (lambda (SP) (when (not ans) - (if (string= fulldir (file-truename (oref SP :directory))) + (if (string= fulldir (file-truename (oref SP directory))) (setq ans SP) (ede-find-subproject-for-directory SP dir))))) ans) @@ -358,11 +358,11 @@ If DIR is not part of a project, return nil." ((and (string= dir default-directory) ede-object-root-project) ;; Try the local buffer cache first. - (oref ede-object-root-project :directory)) + (oref ede-object-root-project directory)) ;; See if there is an existing project in DIR. ((setq ans (ede-directory-get-toplevel-open-project dir)) - (oref ans :directory)) + (oref ans directory)) ;; Detect using our file system detector. ((setq ans (ede-detect-directory-for-project dir)) diff --git a/lisp/cedet/ede/project-am.el b/lisp/cedet/ede/project-am.el index 12db0a43ce7..fbbbe6871b6 100644 --- a/lisp/cedet/ede/project-am.el +++ b/lisp/cedet/ede/project-am.el @@ -491,7 +491,7 @@ This is used when subprojects are made in named subdirectories." :version ver :mailinglist (or bug "") :file fn))) - (oset ampf :directory (file-name-directory fn)) + (oset ampf directory (file-name-directory fn)) (oset ampf configureoutputfiles cof) (make-local-variable 'ede-object) (setq ede-object ampf) @@ -659,11 +659,11 @@ Strip out duplicates, and recurse on variables." (mapc (lambda (sp) (let* ((subdir (file-name-as-directory (expand-file-name - sp (file-name-directory (oref this :file))))) + sp (file-name-directory (oref this file))))) (submake (expand-file-name "Makefile.am" subdir))) - (if (string= submake (oref this :file)) + (if (string= submake (oref this file)) nil ;; don't recurse.. please! ;; For each project id found, see if we need to recycle, ;; and if we do not, then make a new one. Check the deep @@ -694,45 +694,46 @@ Strip out duplicates, and recurse on variables." (cl-defmethod project-rescan ((this project-am-program)) "Rescan object THIS." - (oset this :source (makefile-macro-file-list (project-am-macro this))) - (unless (oref this :source) - (oset this :source (list (concat (oref this :name) ".c")))) - (oset this :ldadd (makefile-macro-file-list - (concat (oref this :name) "_LDADD")))) + (oset this source (makefile-macro-file-list (project-am-macro this))) + (unless (oref this source) + (oset this source (list (concat (oref this name) ".c")))) + (oset this ldadd (makefile-macro-file-list + (concat (oref this name) "_LDADD")))) (cl-defmethod project-rescan ((this project-am-lib)) "Rescan object THIS." - (oset this :source (makefile-macro-file-list (project-am-macro this))) - (unless (oref this :source) - (oset this :source (list (concat (file-name-sans-extension (oref this :name)) ".c"))))) + (oset this source (makefile-macro-file-list (project-am-macro this))) + (unless (oref this source) + (oset this source (list (concat (file-name-sans-extension + (oref this name)) ".c"))))) (cl-defmethod project-rescan ((this project-am-texinfo)) "Rescan object THIS." - (oset this :include (makefile-macro-file-list (project-am-macro this)))) + (oset this include (makefile-macro-file-list (project-am-macro this)))) (cl-defmethod project-rescan ((this project-am-man)) "Rescan object THIS." - (oset this :source (makefile-macro-file-list (project-am-macro this)))) + (oset this source (makefile-macro-file-list (project-am-macro this)))) (cl-defmethod project-rescan ((this project-am-lisp)) "Rescan the lisp sources." - (oset this :source (makefile-macro-file-list (project-am-macro this)))) + (oset this source (makefile-macro-file-list (project-am-macro this)))) (cl-defmethod project-rescan ((this project-am-header)) "Rescan the Header sources for object THIS." - (oset this :source (makefile-macro-file-list (project-am-macro this)))) + (oset this source (makefile-macro-file-list (project-am-macro this)))) (cl-defmethod project-rescan ((this project-am-built-src)) "Rescan built sources for object THIS." - (oset this :source (makefile-macro-file-list "BUILT_SOURCES"))) + (oset this source (makefile-macro-file-list "BUILT_SOURCES"))) (cl-defmethod project-rescan ((this project-am-extra-dist)) "Rescan object THIS." - (oset this :source (makefile-macro-file-list "EXTRA_DIST"))) + (oset this source (makefile-macro-file-list "EXTRA_DIST"))) (cl-defmethod project-am-macro ((this project-am-objectcode)) "Return the default macro to `edit' for this object type." - (concat (subst-char-in-string ?- ?_ (oref this :name)) "_SOURCES")) + (concat (subst-char-in-string ?- ?_ (oref this name)) "_SOURCES")) (cl-defmethod project-am-macro ((this project-am-header-noinst)) "Return the default macro to `edit' for this object." @@ -752,11 +753,11 @@ Strip out duplicates, and recurse on variables." (cl-defmethod project-am-macro ((this project-am-texinfo)) "Return the default macro to `edit' for this object type." - (concat (file-name-sans-extension (oref this :name)) "_TEXINFOS")) + (concat (file-name-sans-extension (oref this name)) "_TEXINFOS")) (cl-defmethod project-am-macro ((this project-am-man)) "Return the default macro to `edit' for this object type." - (oref this :name)) + (oref this name)) (cl-defmethod project-am-macro ((this project-am-lisp)) "Return the default macro to `edit' for this object." @@ -784,7 +785,7 @@ nil means that this buffer belongs to no-one." (cl-defmethod ede-buffer-mine ((this project-am-makefile) buffer) "Return t if object THIS lays claim to the file in BUFFER." (let ((efn (expand-file-name (buffer-file-name buffer)))) - (or (string= (oref this :file) efn) + (or (string= (oref this file) efn) (string-match "/configure\\.ac$" efn) (string-match "/configure\\.in$" efn) (string-match "/configure$" efn) @@ -798,25 +799,25 @@ nil means that this buffer belongs to no-one." (cl-defmethod ede-buffer-mine ((this project-am-objectcode) buffer) "Return t if object THIS lays claim to the file in BUFFER." - (member (file-relative-name (buffer-file-name buffer) (oref this :path)) - (oref this :source))) + (member (file-relative-name (buffer-file-name buffer) (oref this path)) + (oref this source))) (cl-defmethod ede-buffer-mine ((this project-am-texinfo) buffer) "Return t if object THIS lays claim to the file in BUFFER." (let ((bfn (file-relative-name (buffer-file-name buffer) - (oref this :path)))) - (or (string= (oref this :name) bfn) - (member bfn (oref this :include))))) + (oref this path)))) + (or (string= (oref this name) bfn) + (member bfn (oref this include))))) (cl-defmethod ede-buffer-mine ((this project-am-man) buffer) "Return t if object THIS lays claim to the file in BUFFER." - (string= (oref this :name) - (file-relative-name (buffer-file-name buffer) (oref this :path)))) + (string= (oref this name) + (file-relative-name (buffer-file-name buffer) (oref this path)))) (cl-defmethod ede-buffer-mine ((this project-am-lisp) buffer) "Return t if object THIS lays claim to the file in BUFFER." - (member (file-relative-name (buffer-file-name buffer) (oref this :path)) - (oref this :source))) + (member (file-relative-name (buffer-file-name buffer) (oref this path)) + (oref this source))) (cl-defmethod project-am-subtree ((ampf project-am-makefile) subdir) "Return the sub project in AMPF specified by SUBDIR." @@ -829,11 +830,11 @@ nil means that this buffer belongs to no-one." (cl-defmethod project-compile-target-command ((this project-am-objectcode)) "Default target to use when compiling an object code target." - (oref this :name)) + (oref this name)) (cl-defmethod project-compile-target-command ((this project-am-texinfo)) "Default target t- use when compiling a texinfo file." - (let ((n (oref this :name))) + (let ((n (oref this name))) (if (string-match "\\.texi?\\(nfo\\)?" n) (setq n (replace-match ".info" t t n))) n)) @@ -993,7 +994,7 @@ Kill the Configure buffer if it was not already in a buffer." "Get the package information for directory topmost project dir over DIR. Calculates the info with `project-am-extract-package-info'." (let ((top (ede-toplevel))) - (when top (setq dir (oref top :directory))) + (when top (setq dir (oref top directory))) (project-am-extract-package-info dir))) ;; for simple per project include path extension diff --git a/lisp/cedet/ede/shell.el b/lisp/cedet/ede/shell.el index 11a5eacaef0..3641803e1e0 100644 --- a/lisp/cedet/ede/shell.el +++ b/lisp/cedet/ede/shell.el @@ -38,7 +38,7 @@ COMMAND is a text string representing the thing to be run." (let* ((buff (ede-shell-buffer target)) (cp (ede-target-parent target)) - (dd (oref cp :directory))) + (dd (oref cp directory))) ;; Show the new buffer. (when (not (get-buffer-window buff)) (switch-to-buffer-other-window buff t)) diff --git a/lisp/cedet/ede/util.el b/lisp/cedet/ede/util.el index 29b8fbffade..37d6b18233f 100644 --- a/lisp/cedet/ede/util.el +++ b/lisp/cedet/ede/util.el @@ -41,8 +41,8 @@ Argument NEWVERSION is the version number to use in the current project." v nil v)))) (let ((ede-object (ede-toplevel))) ;; Don't update anything if there was no change. - (unless (string= (oref ede-object :version) newversion) - (oset ede-object :version newversion) + (unless (string= (oref ede-object version) newversion) + (oset ede-object version newversion) (project-update-version ede-object) (ede-update-version-in-source ede-object newversion)))) diff --git a/lisp/cedet/semantic/analyze.el b/lisp/cedet/semantic/analyze.el index 9ea7ebf7c1b..8f1d92822e6 100644 --- a/lisp/cedet/semantic/analyze.el +++ b/lisp/cedet/semantic/analyze.el @@ -201,7 +201,7 @@ Optional argument DESIRED-TYPE may be a non-type tag to analyze." (cl-defmethod semantic-analyze-interesting-tag ((context semantic-analyze-context)) "Return a tag from CONTEXT that would be most interesting to a user." - (let ((prefix (reverse (oref context :prefix)))) + (let ((prefix (reverse (oref context prefix)))) ;; Go back through the prefix until we find a tag we can return. (while (and prefix (not (semantic-tag-p (car prefix)))) (setq prefix (cdr prefix))) @@ -211,12 +211,12 @@ Optional argument DESIRED-TYPE may be a non-type tag to analyze." (cl-defmethod semantic-analyze-interesting-tag ((context semantic-analyze-context-functionarg)) "Try the base, and if that fails, return what we are assigning into." - (or (cl-call-next-method) (car-safe (oref context :function)))) + (or (cl-call-next-method) (car-safe (oref context function)))) (cl-defmethod semantic-analyze-interesting-tag ((context semantic-analyze-context-assignment)) "Try the base, and if that fails, return what we are assigning into." - (or (cl-call-next-method) (car-safe (oref context :assignee)))) + (or (cl-call-next-method) (car-safe (oref context assignee)))) ;;; ANALYSIS ;; @@ -738,8 +738,8 @@ Optional argument CTXT is the context to show." (cl-defmethod semantic-analyze-pulse ((context semantic-analyze-context)) "Pulse the region that CONTEXT affects." (require 'pulse) - (with-current-buffer (oref context :buffer) - (let ((bounds (oref context :bounds))) + (with-current-buffer (oref context buffer) + (let ((bounds (oref context bounds))) (when bounds (pulse-momentary-highlight-region (car bounds) (cdr bounds)))))) diff --git a/lisp/cedet/semantic/analyze/debug.el b/lisp/cedet/semantic/analyze/debug.el index 8ad9c22216f..0cd8e44f588 100644 --- a/lisp/cedet/semantic/analyze/debug.el +++ b/lisp/cedet/semantic/analyze/debug.el @@ -478,7 +478,7 @@ variable `semantic-dependency-system-include-path'.")) (defun semantic-analyzer-debug-describe-scope (ctxt &optional classconstraint) "Describe the scope in CTXT for finding a global symbol. Optional argument CLASSCONSTRAINT says to output to tags of that class." - (let* ((scope (oref ctxt :scope)) + (let* ((scope (oref ctxt scope)) (parents (oref scope parents)) (cc (or classconstraint (oref ctxt prefixclass))) ) diff --git a/lisp/cedet/semantic/analyze/refs.el b/lisp/cedet/semantic/analyze/refs.el index 773210698ad..cb0f4645320 100644 --- a/lisp/cedet/semantic/analyze/refs.el +++ b/lisp/cedet/semantic/analyze/refs.el @@ -104,7 +104,7 @@ Use `semantic-analyze-current-tag' to debug this fcn." "Return the implementations derived in the reference analyzer REFS. Optional argument IN-BUFFER indicates that the returned tag should be in an active buffer." (let ((allhits (oref refs rawsearchdata)) - (tag (oref refs :tag)) + (tag (oref refs tag)) (impl nil) ) (semanticdb-find-result-mapc @@ -129,7 +129,7 @@ Optional argument IN-BUFFER indicates that the returned tag should be in an acti "Return the prototypes derived in the reference analyzer REFS. Optional argument IN-BUFFER indicates that the returned tag should be in an active buffer." (let ((allhits (oref refs rawsearchdata)) - (tag (oref refs :tag)) + (tag (oref refs tag)) (proto nil)) (semanticdb-find-result-mapc (lambda (T DB) diff --git a/lisp/cedet/semantic/bovine/make.el b/lisp/cedet/semantic/bovine/make.el index 4fe33345d73..3676c6972f2 100644 --- a/lisp/cedet/semantic/bovine/make.el +++ b/lisp/cedet/semantic/bovine/make.el @@ -181,10 +181,10 @@ Uses default implementation, and also gets a list of filenames." (require 'semantic/analyze/complete) (with-current-buffer (oref context buffer) (let* ((normal (semantic-analyze-possible-completions-default context)) - (classes (oref context :prefixclass)) + (classes (oref context prefixclass)) (filetags nil)) (when (memq 'filename classes) - (let* ((prefix (car (oref context :prefix))) + (let* ((prefix (car (oref context prefix))) (completetext (cond ((semantic-tag-p prefix) (semantic-tag-name prefix)) ((stringp prefix) diff --git a/lisp/cedet/semantic/complete.el b/lisp/cedet/semantic/complete.el index ea2e2dbd1af..3c950230559 100644 --- a/lisp/cedet/semantic/complete.el +++ b/lisp/cedet/semantic/complete.el @@ -1281,7 +1281,7 @@ Uses semanticdb for searching all tags in the current project." (localstuff (oref scope scope))) (list (cons - (oref scope :table) + (oref scope table) (semantic-find-tags-for-completion prefix localstuff))))) ;(semanticdb-brute-deep-find-tags-for-completion prefix (oref obj path)))) diff --git a/lisp/cedet/semantic/db-typecache.el b/lisp/cedet/semantic/db-typecache.el index 2c99b2cef3b..fe9d2d21056 100644 --- a/lisp/cedet/semantic/db-typecache.el +++ b/lisp/cedet/semantic/db-typecache.el @@ -554,7 +554,7 @@ If there isn't one, create it. (stream nil) ) (dolist (table (semanticdb-get-database-tables db)) - (when (eq lmode (oref table :major-mode)) + (when (eq lmode (oref table major-mode)) (setq stream (semanticdb-typecache-merge-streams stream diff --git a/lisp/cedet/semantic/debug.el b/lisp/cedet/semantic/debug.el index e5c04d1b8d7..b5bb00d8cda 100644 --- a/lisp/cedet/semantic/debug.el +++ b/lisp/cedet/semantic/debug.el @@ -170,7 +170,7 @@ These buffers are brought into view when layout occurs.") (cl-defmethod semantic-debug-highlight-lexical-token ((iface semantic-debug-interface) token) "For IFACE, highlight TOKEN in the source buffer . TOKEN is a lexical token." - (set-buffer (oref iface :source-buffer)) + (set-buffer (oref iface source-buffer)) (object-add-to-list iface 'overlays (semantic-lex-highlight-token token)) @@ -183,7 +183,7 @@ TOKEN is a lexical token." NONTERM is the name of the rule currently being processed that shows up as a nonterminal (or tag) in the source buffer. If RULE and MATCH indices are specified, highlight those also." - (set-buffer (oref iface :parser-buffer)) + (set-buffer (oref iface parser-buffer)) (let* ((rules (semantic-find-tags-by-class 'nonterminal (current-buffer))) (nt (semantic-find-first-tag-by-name nonterm rules)) diff --git a/lisp/cedet/semantic/decorate/include.el b/lisp/cedet/semantic/decorate/include.el index eaf2425c53f..a8f3f2a87d1 100644 --- a/lisp/cedet/semantic/decorate/include.el +++ b/lisp/cedet/semantic/decorate/include.el @@ -437,11 +437,11 @@ its contents. ;; Get the semanticdb statement, and display it's contents. (princ "\nDetails for header file...\n") (princ "\nMajor Mode: ") - (princ (oref table :major-mode)) + (princ (oref table major-mode)) (princ "\nTags: ") - (princ (format "%s entries" (length (oref table :tags)))) + (princ (format "%s entries" (length (oref table tags)))) (princ "\nFile Size: ") - (princ (format "%s chars" (oref table :pointmax))) + (princ (format "%s chars" (oref table pointmax))) (princ "\nSave State: ") (cond ((oref table dirty) (princ "Table needs to be saved.")) diff --git a/lisp/cedet/semantic/grammar.el b/lisp/cedet/semantic/grammar.el index 2af7a7fa826..0db675b8f23 100644 --- a/lisp/cedet/semantic/grammar.el +++ b/lisp/cedet/semantic/grammar.el @@ -1919,7 +1919,7 @@ Optional argument COLOR determines if color is added to the text." (with-mode-local emacs-lisp-mode (semantic-analyze-possible-completions context)) (with-current-buffer (oref context buffer) - (let* ((prefix (car (oref context :prefix))) + (let* ((prefix (car (oref context prefix))) (completetext (cond ((semantic-tag-p prefix) (semantic-tag-name prefix)) ((stringp prefix) diff --git a/lisp/cedet/semantic/mru-bookmark.el b/lisp/cedet/semantic/mru-bookmark.el index a15ea9b316a..627c71a01b8 100644 --- a/lisp/cedet/semantic/mru-bookmark.el +++ b/lisp/cedet/semantic/mru-bookmark.el @@ -315,7 +315,7 @@ non-nil if the minor mode is enabled." (al nil)) (while (< idx len) (let ((r (ring-ref ring idx))) - (setq al (cons (cons (oref r :object-name) r) + (setq al (cons (cons (oref r object-name) r) al))) (setq idx (1+ idx))) (nreverse al))) diff --git a/lisp/cedet/semantic/symref/cscope.el b/lisp/cedet/semantic/symref/cscope.el index 89545965bf1..39f86611316 100644 --- a/lisp/cedet/semantic/symref/cscope.el +++ b/lisp/cedet/semantic/symref/cscope.el @@ -51,14 +51,11 @@ See the function `cedet-cscope-search' for more details.") default-directory)) ;; CScope has to be run from the project root where ;; cscope.out is. - (b (cedet-cscope-search (oref tool :searchfor) - (oref tool :searchtype) - (oref tool :resulttype) - (oref tool :searchscope) - )) - ) - (semantic-symref-parse-tool-output tool b) - )) + (b (cedet-cscope-search (oref tool searchfor) + (oref tool searchtype) + (oref tool resulttype) + (oref tool searchscope)))) + (semantic-symref-parse-tool-output tool b))) (defconst semantic-symref-cscope--line-re "^\\([^ ]+\\) [^ ]+ \\([0-9]+\\) ") @@ -66,22 +63,22 @@ See the function `cedet-cscope-search' for more details.") (cl-defmethod semantic-symref-parse-tool-output-one-line ((tool semantic-symref-tool-cscope)) "Parse one line of grep output, and return it as a match list. Moves cursor to end of the match." - (cond ((eq (oref tool :resulttype) 'file) + (cond ((eq (oref tool resulttype) 'file) ;; Search for files (when (re-search-forward "^\\([^\n]+\\)$" nil t) (match-string 1))) - ((eq (oref tool :searchtype) 'tagcompletions) + ((eq (oref tool searchtype) 'tagcompletions) ;; Search for files (when (re-search-forward "^[^ ]+ [^ ]+ [^ ]+ \\(.*\\)$" nil t) (let ((subtxt (match-string 1)) - (searchtxt (oref tool :searchfor))) + (searchtxt (oref tool searchfor))) (if (string-match (concat "\\<" searchtxt "\\(\\w\\|\\s_\\)*\\>") subtxt) (match-string 0 subtxt) ;; We have to return something at this point. subtxt))) ) - ((eq (oref tool :resulttype) 'line-and-text) + ((eq (oref tool resulttype) 'line-and-text) (when (re-search-forward semantic-symref-cscope--line-re nil t) (list (string-to-number (match-string 2)) (expand-file-name (match-string 1)) diff --git a/lisp/cedet/semantic/symref/global.el b/lisp/cedet/semantic/symref/global.el index 2eb0ab091cf..d1ea921302f 100644 --- a/lisp/cedet/semantic/symref/global.el +++ b/lisp/cedet/semantic/symref/global.el @@ -40,14 +40,11 @@ See the function `cedet-gnu-global-search' for more details.") (cl-defmethod semantic-symref-perform-search ((tool semantic-symref-tool-global)) "Perform a search with GNU Global." - (let ((b (cedet-gnu-global-search (oref tool :searchfor) - (oref tool :searchtype) - (oref tool :resulttype) - (oref tool :searchscope) - )) - ) - (semantic-symref-parse-tool-output tool b) - )) + (let ((b (cedet-gnu-global-search (oref tool searchfor) + (oref tool searchtype) + (oref tool resulttype) + (oref tool searchscope)))) + (semantic-symref-parse-tool-output tool b))) (defconst semantic-symref-global--line-re "^\\([^ ]+\\) +\\([0-9]+\\) \\([^ ]+\\) ") @@ -55,12 +52,12 @@ See the function `cedet-gnu-global-search' for more details.") (cl-defmethod semantic-symref-parse-tool-output-one-line ((tool semantic-symref-tool-global)) "Parse one line of grep output, and return it as a match list. Moves cursor to end of the match." - (cond ((or (eq (oref tool :resulttype) 'file) - (eq (oref tool :searchtype) 'tagcompletions)) + (cond ((or (eq (oref tool resulttype) 'file) + (eq (oref tool searchtype) 'tagcompletions)) ;; Search for files (when (re-search-forward "^\\([^\n]+\\)$" nil t) (match-string 1))) - ((eq (oref tool :resulttype) 'line-and-text) + ((eq (oref tool resulttype) 'line-and-text) (when (re-search-forward semantic-symref-global--line-re nil t) (list (string-to-number (match-string 2)) (match-string 3) diff --git a/lisp/cedet/semantic/symref/idutils.el b/lisp/cedet/semantic/symref/idutils.el index 9f28e16c348..489bcec060d 100644 --- a/lisp/cedet/semantic/symref/idutils.el +++ b/lisp/cedet/semantic/symref/idutils.el @@ -40,14 +40,11 @@ See the function `cedet-idutils-search' for more details.") (cl-defmethod semantic-symref-perform-search ((tool semantic-symref-tool-idutils)) "Perform a search with IDUtils." - (let ((b (cedet-idutils-search (oref tool :searchfor) - (oref tool :searchtype) - (oref tool :resulttype) - (oref tool :searchscope) - )) - ) - (semantic-symref-parse-tool-output tool b) - )) + (let ((b (cedet-idutils-search (oref tool searchfor) + (oref tool searchtype) + (oref tool resulttype) + (oref tool searchscope)))) + (semantic-symref-parse-tool-output tool b))) (defconst semantic-symref-idutils--line-re "^\\(\\(?:[a-zA-Z]:\\)?[^:\n]+\\):\\([0-9]+\\):") @@ -55,14 +52,14 @@ See the function `cedet-idutils-search' for more details.") (cl-defmethod semantic-symref-parse-tool-output-one-line ((tool semantic-symref-tool-idutils)) "Parse one line of grep output, and return it as a match list. Moves cursor to end of the match." - (cond ((eq (oref tool :resulttype) 'file) + (cond ((eq (oref tool resulttype) 'file) ;; Search for files (when (re-search-forward "^\\([^\n]+\\)$" nil t) (match-string 1))) - ((eq (oref tool :searchtype) 'tagcompletions) + ((eq (oref tool searchtype) 'tagcompletions) (when (re-search-forward "^\\([^ ]+\\) " nil t) (match-string 1))) - ((eq (oref tool :resulttype) 'line-and-text) + ((eq (oref tool resulttype) 'line-and-text) (when (re-search-forward semantic-symref-idutils--line-re nil t) (list (string-to-number (match-string 2)) (expand-file-name (match-string 1) default-directory) diff --git a/lisp/cedet/semantic/symref/list.el b/lisp/cedet/semantic/symref/list.el index c11eb310a0f..f6417b1c6b7 100644 --- a/lisp/cedet/semantic/symref/list.el +++ b/lisp/cedet/semantic/symref/list.el @@ -205,7 +205,7 @@ Some useful functions are found in `semantic-format-tag-functions'." (erase-buffer) ;; Insert the contents. (let ((lastfile nil)) - (dolist (T (oref results :hit-tags)) + (dolist (T (oref results hit-tags)) (unless (equal lastfile (semantic-tag-file-name T)) (setq lastfile (semantic-tag-file-name T)) (insert-button lastfile @@ -402,8 +402,8 @@ cursor to the beginning of that symbol, then record a macro as if {kmacro-end-macro} to end the macro, and return to the symbol found list." (interactive) (let* ((oldsym (oref (oref semantic-symref-current-results - :created-by) - :searchfor)) + created-by) + searchfor)) (ol (save-excursion (end-of-line) (car (semantic-overlays-at (1- (point)))))) @@ -448,8 +448,8 @@ Closed items will be skipped." (interactive (list (read-string "Rename to: " (oref (oref semantic-symref-current-results - :created-by) - :searchfor)))) + created-by) + searchfor)))) (let ((count (semantic-symref-list-map-open-hits (lambda () (replace-match newname nil t))))) (semantic-symref-list-update-open-hits) @@ -469,8 +469,8 @@ Return the number of occurrences FUNCTION was operated upon." ;; class members. (Not Done) (let ((oldsym (oref (oref semantic-symref-current-results - :created-by) - :searchfor)) + created-by) + searchfor)) (count 0)) (save-excursion (goto-char (point-min)) diff --git a/lisp/cedet/semantic/texi.el b/lisp/cedet/semantic/texi.el index f079bf201c8..3a0050b920c 100644 --- a/lisp/cedet/semantic/texi.el +++ b/lisp/cedet/semantic/texi.el @@ -419,9 +419,9 @@ Since texinfo is not a programming language the default version is not useful. Instead, look at the current symbol. If it is a command do primitive texinfo built ins. If not, use ispell to lookup words that start with that symbol." - (let ((prefix (car (oref context :prefix))) + (let ((prefix (car (oref context prefix))) ) - (cond ((member 'function (oref context :prefixclass)) + (cond ((member 'function (oref context prefixclass)) ;; Do completion for texinfo commands (let* ((cmd (substring prefix 1)) (lst (all-completions @@ -429,7 +429,7 @@ that start with that symbol." (mapcar (lambda (f) (semantic-tag (concat "@" f) 'function)) lst)) ) - ((member 'word (oref context :prefixclass)) + ((member 'word (oref context prefixclass)) ;; Do completion for words via ispell. (require 'ispell) (let ((word-list (ispell-lookup-words prefix))) diff --git a/lisp/cedet/srecode/args.el b/lisp/cedet/srecode/args.el index 7f5600083ee..086f369b7f2 100644 --- a/lisp/cedet/srecode/args.el +++ b/lisp/cedet/srecode/args.el @@ -174,8 +174,8 @@ do not contain any text from preceding or following text." (srecode-dictionary-set-value dict "PROJECT_FILENAME" relfname) (srecode-dictionary-set-value dict "PROJECT_DIRECTORY" reldir) (srecode-dictionary-set-value dict "PROJECT_NAME" (ede-name (ede-toplevel))) - (srecode-dictionary-set-value dict "PROJECT_VERSION" (oref (ede-toplevel) :version)) - ) + (srecode-dictionary-set-value dict "PROJECT_VERSION" + (oref (ede-toplevel) version))) ;; If there is no EDE project, then put in some base values. (srecode-dictionary-set-value dict "PROJECT_FILENAME" bfn) (srecode-dictionary-set-value dict "PROJECT_DIRECTORY" dir) diff --git a/lisp/cedet/srecode/compile.el b/lisp/cedet/srecode/compile.el index 204211b5e53..ff640c020f3 100644 --- a/lisp/cedet/srecode/compile.el +++ b/lisp/cedet/srecode/compile.el @@ -547,8 +547,8 @@ A list of defined variables VARS provides a variable table." (while lp - (let* ((objname (oref (car lp) :object-name)) - (context (oref (car lp) :context)) + (let* ((objname (oref (car lp) object-name)) + (context (oref (car lp) context)) (globalname (concat context ":" objname)) ) @@ -583,7 +583,7 @@ A list of defined variables VARS provides a variable table." (tmpl (oref table templates))) ;; Loop over all the templates, and xref. (while tmpl - (oset (car tmpl) :table table) + (oset (car tmpl) table table) (setq tmpl (cdr tmpl)))) )) @@ -644,9 +644,9 @@ Argument INDENT specifies the indentation level for the list." "Dump the state of the SRecode template inserter INS." (princ "INS: \"") (princ (eieio-object-name-string ins)) - (when (oref ins :secondname) + (when (oref ins secondname) (princ "\" : \"") - (princ (oref ins :secondname))) + (princ (oref ins secondname))) (princ "\" type \"") (let* ((oc (symbol-name (eieio-object-class ins))) (junk (string-match "srecode-template-inserter-" oc)) diff --git a/lisp/cedet/srecode/cpp.el b/lisp/cedet/srecode/cpp.el index cedc195433c..306c60f1b61 100644 --- a/lisp/cedet/srecode/cpp.el +++ b/lisp/cedet/srecode/cpp.el @@ -122,7 +122,7 @@ specified in a C file." (srecode-semantic-apply-tag-to-dict-default tag-wrapper dict) ;; Pull out the tag for the individual pieces. - (let* ((tag (oref tag-wrapper :prime)) + (let* ((tag (oref tag-wrapper prime)) (class (semantic-tag-class tag))) ;; Add additional information based on the class of the tag. diff --git a/lisp/cedet/srecode/dictionary.el b/lisp/cedet/srecode/dictionary.el index 4b1f28babd5..8508371d53b 100644 --- a/lisp/cedet/srecode/dictionary.el +++ b/lisp/cedet/srecode/dictionary.el @@ -122,7 +122,7 @@ Makes sure that :value is compiled." (cl-call-next-method this (nreverse newfields)) (when (not (slot-boundp this 'compiled)) - (let ((val (oref this :value)) + (let ((val (oref this value)) (comp nil)) (while val (let ((nval (car val)) @@ -141,7 +141,7 @@ Makes sure that :value is compiled." (error "Don't know how to handle variable value %S" nval))) ) (setq val (cdr val))) - (oset this :compiled (nreverse comp)))))) + (oset this compiled (nreverse comp)))))) ;;; DICTIONARY METHODS ;; @@ -223,7 +223,7 @@ TPL is an object representing a compiled template file." ;; Tables are sorted with highest priority first, useful for looking ;; up templates, but this means we need to install the variables in ;; reverse order so higher priority variables override lower ones. - (let ((tabs (reverse (oref tpl :tables)))) + (let ((tabs (reverse (oref tpl tables)))) (require 'srecode/find) ; For srecode-template-table-in-project-p (while tabs (when (srecode-template-table-in-project-p (car tabs)) @@ -518,7 +518,7 @@ inserted with a new editable field.") (let* ((dv (oref cp defaultvalue)) (sti (oref cp firstinserter)) (start (point)) - (name (oref sti :object-name))) + (name (oref sti object-name))) (cond ;; No default value. diff --git a/lisp/cedet/srecode/el.el b/lisp/cedet/srecode/el.el index 0bc8e1f450c..33b75cad692 100644 --- a/lisp/cedet/srecode/el.el +++ b/lisp/cedet/srecode/el.el @@ -86,7 +86,7 @@ Calls `srecode-semantic-apply-tag-to-dict-default' first." (srecode-semantic-apply-tag-to-dict-default tagobj dict) ;; Pull out the tag for the individual pieces. - (let* ((tag (oref tagobj :prime)) + (let* ((tag (oref tagobj prime)) (doc (semantic-tag-docstring tag))) ;; It is much more common to have doc on ELisp. diff --git a/lisp/cedet/srecode/extract.el b/lisp/cedet/srecode/extract.el index acb200a46ec..c46ff7e38f7 100644 --- a/lisp/cedet/srecode/extract.el +++ b/lisp/cedet/srecode/extract.el @@ -161,10 +161,9 @@ Return nil as this inserter will extract nothing." Return t if something was extracted. Return nil if this inserter doesn't need to extract anything." (srecode-dictionary-set-value vdict - (oref ins :object-name) + (oref ins object-name) (buffer-substring-no-properties - start end) - ) + start end)) t) ;;; Section Inserter @@ -178,10 +177,9 @@ Return nil if this inserter doesn't need to extract anything." "Extract text from START/END and store in INDICT. Return the starting location of the first plain-text match. Return nil if nothing was extracted." - (let ((name (oref ins :object-name)) + (let ((name (oref ins object-name)) (subdict (srecode-create-dictionary indict)) - (allsubdict nil) - ) + (allsubdict nil)) ;; Keep extracting till we can extract no more. (while (condition-case nil @@ -217,10 +215,10 @@ Return nil if nothing was extracted." ;; There are two modes for includes. One is with no dict, ;; so it is inserted straight. If the dict has a name, then ;; we need to run once per dictionary occurrence. - (if (not (string= (oref ins :object-name) "")) + (if (not (string= (oref ins object-name) "")) ;; With a name, do the insertion. (let ((subdict (srecode-dictionary-add-section-dictionary - dict (oref ins :object-name)))) + dict (oref ins object-name)))) (error "Need to implement include w/ name extractor") ;; Recurse into the new template while no errors. (while (condition-case nil diff --git a/lisp/cedet/srecode/find.el b/lisp/cedet/srecode/find.el index f5ddc619b8c..f130edcee8d 100644 --- a/lisp/cedet/srecode/find.el +++ b/lisp/cedet/srecode/find.el @@ -100,7 +100,7 @@ all template files for that application will be loaded." "Return non-nil if the table TAB can be used in the current project. If TAB has a :project set, check that the directories match. If TAB is nil, then always return t." - (let ((proj (oref tab :project))) + (let ((proj (oref tab project))) ;; Return t if the project wasn't set. (if (not proj) t ;; If the project directory was set, let's check it. @@ -139,10 +139,10 @@ Optional argument APPLICATION restricts searches to only template tables belonging to a specific application. If APPLICATION is nil, then only tables that do not belong to an application will be searched." (let* ((mt tab) - (tabs (oref mt :tables)) + (tabs (oref mt tables)) (ans nil)) (while (and (not ans) tabs) - (let ((app (oref (car tabs) :application))) + (let ((app (oref (car tabs) application))) (when (or (and (not application) (null app)) (and application (eq app application))) (setq ans (srecode-template-get-table (car tabs) template-name @@ -150,7 +150,7 @@ tables that do not belong to an application will be searched." (setq tabs (cdr tabs)))) (or ans ;; Recurse to the default. - (when (not (equal (oref tab :major-mode) 'default)) + (when (not (equal (oref tab major-mode) 'default)) (srecode-template-get-table (srecode-get-mode-table 'default) template-name context application))))) @@ -199,10 +199,10 @@ Optional argument APPLICATION restricts searches to only template tables belonging to a specific application. If APPLICATION is nil, then only tables that do not belong to an application will be searched." (let* ((mt tab) - (tabs (oref mt :tables)) + (tabs (oref mt tables)) (ans nil)) (while (and (not ans) tabs) - (let ((app (oref (car tabs) :application))) + (let ((app (oref (car tabs) application))) (when (or (and (not application) (null app)) (and application (eq app application))) (setq ans (srecode-template-get-table-for-binding @@ -210,7 +210,7 @@ tables that do not belong to an application will be searched." (setq tabs (cdr tabs)))) (or ans ;; Recurse to the default. - (when (not (equal (oref tab :major-mode) 'default)) + (when (not (equal (oref tab major-mode) 'default)) (srecode-template-get-table-for-binding (srecode-get-mode-table 'default) binding context))))) ;;; Interactive @@ -241,10 +241,10 @@ templates." ;; Load up the hash table for our current mode. (let* ((mt (srecode-get-mode-table mmode)) - (tabs (when mt (oref mt :tables)))) + (tabs (when mt (oref mt tables)))) (dolist (tab tabs) ;; Exclude templates for a particular application. - (when (and (not (oref tab :application)) + (when (and (not (oref tab application)) (srecode-template-table-in-project-p tab)) (maphash (lambda (key temp) (when (or (not predicate) diff --git a/lisp/cedet/srecode/insert.el b/lisp/cedet/srecode/insert.el index 66bdf785c63..d23597ec479 100644 --- a/lisp/cedet/srecode/insert.el +++ b/lisp/cedet/srecode/insert.el @@ -471,12 +471,11 @@ If SECONDNAME is nil, return VALUE." dictionary) "Insert the STI inserter." ;; Convert the name into a name/fcn pair - (let* ((name (oref sti :object-name)) - (fcnpart (oref sti :secondname)) + (let* ((name (oref sti object-name)) + (fcnpart (oref sti secondname)) (val (srecode-dictionary-lookup-name dictionary name)) - (do-princ t) - ) + (do-princ t)) ;; Alert if a macro wasn't found. (when (not val) (message "Warning: macro %S was not found in the dictionary." name) @@ -545,12 +544,12 @@ Loop over the prompts to see if we have a match." ) (while prompts (when (string= (semantic-tag-name (car prompts)) - (oref ins :object-name)) - (oset ins :prompt + (oref ins object-name)) + (oset ins prompt (semantic-tag-get-attribute (car prompts) :text)) - (oset ins :defaultfcn + (oset ins defaultfcn (semantic-tag-get-attribute (car prompts) :default)) - (oset ins :read-fcn + (oset ins read-fcn (or (semantic-tag-get-attribute (car prompts) :read) 'read-string)) ) @@ -561,7 +560,7 @@ Loop over the prompts to see if we have a match." dictionary) "Insert the STI inserter." (let ((val (srecode-dictionary-lookup-name - dictionary (oref sti :object-name)))) + dictionary (oref sti object-name)))) (if val ;; Does some extra work. Oh well. (cl-call-next-method) @@ -577,7 +576,7 @@ Loop over the prompts to see if we have a match." ;; the user can use the same name again later. (srecode-dictionary-set-value (srecode-root-dictionary dictionary) - (oref sti :object-name) val) + (oref sti object-name) val) ;; Now that this value is safely stowed in the dictionary, ;; we can do what regular inserters do. @@ -587,7 +586,7 @@ Loop over the prompts to see if we have a match." dictionary) "Derive the default value for an askable inserter STI. DICTIONARY is used to derive some values." - (let ((defaultfcn (oref sti :defaultfcn))) + (let ((defaultfcn (oref sti defaultfcn))) (cond ((stringp defaultfcn) defaultfcn) @@ -614,13 +613,13 @@ DICTIONARY is used to derive some values." Use DICTIONARY to resolve values." (let* ((prompt (oref sti prompt)) (default (srecode-insert-ask-default sti dictionary)) - (reader (oref sti :read-fcn)) + (reader (oref sti read-fcn)) (val nil) ) (cond ((eq reader 'y-or-n-p) (if (y-or-n-p (or prompt (format "%s? " - (oref sti :object-name)))) + (oref sti object-name)))) (setq val default) (setq val ""))) ((eq reader 'read-char) @@ -628,14 +627,14 @@ Use DICTIONARY to resolve values." "%c" (read-char (or prompt (format "Char for %s: " - (oref sti :object-name)))))) + (oref sti object-name)))))) ) (t (save-excursion (setq val (funcall reader (or prompt (format "Specify %s: " - (oref sti :object-name))) + (oref sti object-name))) default ))))) ;; Return our derived value. @@ -648,7 +647,7 @@ Use DICTIONARY to resolve values." Use DICTIONARY to resolve values." (let* ((default (srecode-insert-ask-default sti dictionary)) (compound-value - (srecode-field-value (oref sti :object-name) + (srecode-field-value (oref sti object-name) :firstinserter sti :defaultvalue default)) ) @@ -816,12 +815,12 @@ Arguments ESCAPE-START and ESCAPE-END are the current escape sequences in use." Loops over the embedded CODE which was saved here during compilation. The template to insert is stored in SLOT." (let ((dicts (srecode-dictionary-lookup-name - dictionary (oref sti :object-name)))) + dictionary (oref sti object-name)))) (when (not (listp dicts)) (srecode-insert-report-error dictionary "Cannot insert section %S from non-section variable." - (oref sti :object-name))) + (oref sti object-name))) ;; If there is no section dictionary, then don't output anything ;; from this section. (while dicts @@ -829,7 +828,7 @@ The template to insert is stored in SLOT." (srecode-insert-report-error dictionary "Cannot insert section %S from non-section variable." - (oref sti :object-name))) + (oref sti object-name))) (srecode-insert-subtemplate sti (car dicts) slot) (setq dicts (cdr dicts))))) @@ -860,7 +859,7 @@ applied to the text between the section start and the Shorten input until the END token is found. Return the remains of INPUT." (let* ((out (srecode-compile-split-code tag input STATE - (oref ins :object-name)))) + (oref ins object-name)))) (oset ins template (srecode-template (eieio-object-name-string ins) :context nil @@ -893,7 +892,7 @@ are treated specially.") (cl-defmethod srecode-match-end ((ins srecode-template-inserter-section-end) name) "For the template inserter INS, do I end a section called NAME?" - (string= name (oref ins :object-name))) + (string= name (oref ins object-name))) (defclass srecode-template-inserter-include (srecode-template-inserter-subtemplate) ((key :initform ?> @@ -924,13 +923,13 @@ Arguments ESCAPE-START and ESCAPE-END are the current escape sequences in use." "For the template inserter STI, lookup the template to include. Finds the template with this macro function part and stores it in this template instance." - (let ((templatenamepart (oref sti :secondname))) + (let ((templatenamepart (oref sti secondname))) ;; If there was no template name, throw an error. (unless templatenamepart (srecode-insert-report-error dictionary "Include macro `%s' needs a template name" - (oref sti :object-name))) + (oref sti object-name))) ;; NOTE: We used to cache the template and not look it up a second time, ;; but changes in the template tables can change which template is @@ -968,14 +967,14 @@ this template instance." ) ;; Store the found template into this object for later use. - (oset sti :includedtemplate tmpl)) + (oset sti includedtemplate tmpl)) (unless (oref sti includedtemplate) ;; @todo - Call into a debugger to help find the template in question. (srecode-insert-report-error dictionary "No template \"%s\" found for include macro `%s'" - templatenamepart (oref sti :object-name))))) + templatenamepart (oref sti object-name))))) (cl-defmethod srecode-insert-method ((sti srecode-template-inserter-include) dictionary) @@ -985,7 +984,7 @@ with the dictionaries found in the dictionary." (srecode-insert-include-lookup sti dictionary) ;; Insert the template. ;; Our baseclass has a simple way to do this. - (if (srecode-dictionary-lookup-name dictionary (oref sti :object-name)) + (if (srecode-dictionary-lookup-name dictionary (oref sti object-name)) ;; If we have a value, then call the next method (srecode-insert-method-helper sti dictionary 'includedtemplate) ;; If we don't have a special dictionary, then just insert with the @@ -1053,7 +1052,7 @@ template where a ^ inserter occurs." (lambda (dict) (let ((srecode-template-inserter-point-override nil)) (if (srecode-dictionary-lookup-name - dict (oref sti :object-name)) + dict (oref sti object-name)) ;; Insert our sectional part with looping. (srecode-insert-method-helper sti dict 'template) diff --git a/lisp/cedet/srecode/mode.el b/lisp/cedet/srecode/mode.el index a47520e013a..a40b9d09a6f 100644 --- a/lisp/cedet/srecode/mode.el +++ b/lisp/cedet/srecode/mode.el @@ -195,7 +195,7 @@ MENU-DEF is the menu to bind this into." ;;(srecode-load-tables-for-mode major-mode) (let* ((modetable (srecode-get-mode-table major-mode)) - (subtab (when modetable (oref modetable :tables))) + (subtab (when modetable (oref modetable tables))) (context nil) (active nil) (ltab nil) @@ -318,17 +318,17 @@ Template is chosen based on the mode of the starting buffer." (if (not temp) (error "No Template named %s" template-name)) ;; We need a template specific table, since tables chain. - (let ((tab (oref temp :table)) + (let ((tab (oref temp table)) (names nil) ) - (find-file (oref tab :file)) - (setq names (semantic-find-tags-by-name (oref temp :object-name) + (find-file (oref tab file)) + (setq names (semantic-find-tags-by-name (oref temp object-name) (current-buffer))) (cond ((= (length names) 1) (semantic-go-to-tag (car names)) (semantic-momentary-highlight-tag (car names))) ((> (length names) 1) - (let* ((ctxt (semantic-find-tags-by-name (oref temp :context) + (let* ((ctxt (semantic-find-tags-by-name (oref temp context) (current-buffer))) (cls (semantic-find-tags-by-class 'context ctxt)) ) diff --git a/lisp/cedet/srecode/semantic.el b/lisp/cedet/srecode/semantic.el index ea11b0e4838..1e1a60e0245 100644 --- a/lisp/cedet/srecode/semantic.el +++ b/lisp/cedet/srecode/semantic.el @@ -63,10 +63,9 @@ If FUNCTION is non-nil, then FUNCTION is somehow applied to an aspect of the compound value." (if (not function) ;; Just format it in some handy dandy way. - (semantic-format-tag-prototype (oref cp :prime)) + (semantic-format-tag-prototype (oref cp prime)) ;; Otherwise, apply the function to the tag itself. - (funcall function (oref cp :prime)) - )) + (funcall function (oref cp prime)))) ;;; Managing the `current' tag @@ -106,7 +105,7 @@ variable default values, and other things." (srecode-dictionary-set-value dict "TAG" tagobj) ;; Pull out the tag for the individual pieces. - (let ((tag (oref tagobj :prime))) + (let ((tag (oref tagobj prime))) (srecode-dictionary-set-value dict "NAME" (semantic-tag-name tag)) (srecode-dictionary-set-value dict "TYPE" (semantic-format-tag-type tag nil)) diff --git a/lisp/cedet/srecode/srt-mode.el b/lisp/cedet/srecode/srt-mode.el index 6bf2d51ab41..a4c8cbd1d01 100644 --- a/lisp/cedet/srecode/srt-mode.el +++ b/lisp/cedet/srecode/srt-mode.el @@ -502,10 +502,10 @@ section or ? for an ask variable." ) (when inserter (let ((base - (cons (oref inserter :object-name) + (cons (oref inserter object-name) (if (and (slot-boundp inserter :secondname) - (oref inserter :secondname)) - (split-string (oref inserter :secondname) + (oref inserter secondname)) + (split-string (oref inserter secondname) ":") nil))) (key (oref inserter key))) @@ -627,7 +627,7 @@ section or ? for an ask variable." srecode-template-mode (context) "Return a list of possible completions based on NONTEXT." (with-current-buffer (oref context buffer) - (let* ((prefix (car (last (oref context :prefix)))) + (let* ((prefix (car (last (oref context prefix)))) (prefixstr (cond ((stringp prefix) prefix) ((semantic-tag-p prefix) @@ -638,7 +638,7 @@ section or ? for an ask variable." ; prefix) ; ((stringp (car prefix)) ; (car prefix)))) - (argtype (car (oref context :argument))) + (argtype (car (oref context argument))) (matches nil)) ;; Depending on what the analyzer is, we have different ways diff --git a/lisp/cedet/srecode/table.el b/lisp/cedet/srecode/table.el index 95d8bfa1bc5..7a0600c3f81 100644 --- a/lisp/cedet/srecode/table.el +++ b/lisp/cedet/srecode/table.el @@ -201,8 +201,8 @@ INIT are the initialization parameters for the new template table." ;; into the search table first, allowing lower priority items ;; to be the items found in the search table. (object-sort-list mt 'modetables (lambda (a b) - (> (oref a :priority) - (oref b :priority)))) + (> (oref a priority) + (oref b priority)))) ;; Return it. new)) @@ -239,9 +239,9 @@ Use PREDICATE is the same as for the `sort' function." (cl-defmethod srecode-dump ((tab srecode-mode-table)) "Dump the contents of the SRecode mode table TAB." (princ "MODE TABLE FOR ") - (princ (oref tab :major-mode)) + (princ (oref tab major-mode)) (princ "\n--------------------------------------------\n\nNumber of tables: ") - (let ((subtab (oref tab :tables))) + (let ((subtab (oref tab tables))) (princ (length subtab)) (princ "\n\n") (while subtab @@ -254,17 +254,17 @@ Use PREDICATE is the same as for the `sort' function." (princ "Template Table for ") (princ (eieio-object-name-string tab)) (princ "\nPriority: ") - (prin1 (oref tab :priority)) - (when (oref tab :application) + (prin1 (oref tab priority)) + (when (oref tab application) (princ "\nApplication: ") - (princ (oref tab :application))) - (when (oref tab :framework) + (princ (oref tab application))) + (when (oref tab framework) (princ "\nFramework: ") - (princ (oref tab :framework))) - (when (oref tab :project) + (princ (oref tab framework))) + (when (oref tab project) (require 'srecode/find) ; For srecode-template-table-in-project-p (princ "\nProject Directory: ") - (princ (oref tab :project)) + (princ (oref tab project)) (when (not (srecode-template-table-in-project-p tab)) (princ "\n ** Not Usable in this file. **"))) (princ "\n\nVariables:\n") diff --git a/lisp/cedet/srecode/texi.el b/lisp/cedet/srecode/texi.el index 212b56c6ca0..bb2b7c91316 100644 --- a/lisp/cedet/srecode/texi.el +++ b/lisp/cedet/srecode/texi.el @@ -156,7 +156,7 @@ Adds the following: (error "No tag to insert for :texitag template argument")) ;; Extract the tag out of the compound object. - (setq tag (oref tag :prime)) + (setq tag (oref tag prime)) ;; Extract the doc string (setq doc (semantic-documentation-for-tag tag)) -- 2.39.5