Merge lp:~freiksenet/python-mode/hide-show-support into lp:python-mode

Proposed by Mikhail Novikov
Status: Merged
Approved by: Andreas Roehler
Approved revision: not available
Merged at revision: not available
Proposed branch: lp:~freiksenet/python-mode/hide-show-support
Merge into: lp:python-mode
Diff against target: 41 lines
1 file modified
python-mode.el (+24/-0)
To merge this branch: bzr merge lp:~freiksenet/python-mode/hide-show-support
Reviewer Review Type Date Requested Status
Barry Warsaw code Needs Information
Review via email: mp+12493@code.launchpad.net

Commit message

add support for hide-show hs-minor-mode

To post a comment you must log in.
Revision history for this message
Mikhail Novikov (freiksenet) wrote :

Fixed the XEmacs bug, seems to work alright now.

Revision history for this message
Barry Warsaw (barry) wrote :

Could you please open a bug that describes the exact problem, and link the bug to this branch? It's not clear to me what this is trying to fix.

review: Needs Information (code)
Revision history for this message
Mikhail Novikov (freiksenet) wrote :

> Could you please open a bug that describes the exact problem, and link the bug
> to this branch? It's not clear to me what this is trying to fix.

Hello!

This bug was inside this particular branch, not inside python-mode. Branch itself is dedicated to make hideshow mode work with python-mode.

Best regards, Mikhail

Revision history for this message
Barry Warsaw (barry) wrote :

From <email address hidden>:

"reconsidered these stuff, it's not ready for merge now.

Intend coming up with a new approach.

Is there a way to remove the merge-proposal? Should I delete the whole branch rather?"

Revision history for this message
Barry Warsaw (barry) wrote :

Oops, that was the wrong branch. Andreas, please review this branch yourself and mark it approved if you want. Then merge this branch and push it.

Revision history for this message
Andreas Roehler (a-roehler) wrote :

Barry Warsaw wrote:
> Oops, that was the wrong branch. Andreas, please review this branch yourself and mark it approved if you want. Then merge this branch and push it.

OK, merged locally. Got

" M python-mode.el
All changes applied successfully."

Seems I have to do a commit still.
After this: push into the trunk?

Andreas

Revision history for this message
Barry Warsaw (barry) wrote :

On Jan 18, 2010, at 2:43 AM, Andreas Roehler wrote:

> Barry Warsaw wrote:
>> Oops, that was the wrong branch. Andreas, please review this branch yourself and mark it approved if you want. Then merge this branch and push it.
>
> OK, merged locally. Got
>
> " M python-mode.el
> All changes applied successfully."
>
> Seems I have to do a commit still.
> After this: push into the trunk?

Correct.

Uses --fixes when you do a commit (see 'bzr help commit') and attach the bug number. Then when you push, Launchpad should update all the data.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'python-mode.el'
--- python-mode.el 2009-03-20 01:37:21 +0000
+++ python-mode.el 2009-09-27 18:55:18 +0000
@@ -375,6 +375,21 @@
375 :type 'boolean375 :type 'boolean
376 :group 'python)376 :group 'python)
377377
378(defcustom py-hide-show-keywords '(
379 "class" "def" "elif" "else" "except"
380 "for" "if" "while" "finally" "try"
381 "with"
382 )
383 "*Keywords that can be hiden by hide-show"
384 :type '(repeat string)
385 :group 'python)
386
387(defcustom py-hide-show-hide-docstrings t
388 "*Controls if doc strings can be hiden by hide-show"
389 :type 'boolean
390 :group 'python)
391
392
378393
379394
380;; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^395;; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
381;; NO USER DEFINABLE VARIABLES BEYOND THIS POINT396;; NO USER DEFINABLE VARIABLES BEYOND THIS POINT
@@ -1225,6 +1240,15 @@
1225 (if (fboundp 'imenu-add-to-menubar)1240 (if (fboundp 'imenu-add-to-menubar)
1226 (imenu-add-to-menubar (format "%s-%s" "IM" mode-name)))1241 (imenu-add-to-menubar (format "%s-%s" "IM" mode-name)))
1227 )1242 )
1243
1244 ;; Add support for HideShow
1245 (add-to-list 'hs-special-modes-alist (list
1246 'python-mode (concat (if py-hide-show-hide-docstrings "^\\s-*\"\"\"\\|" "") (mapconcat 'identity (mapcar #'(lambda (x) (concat "^\\s-*" x "\\>")) py-hide-show-keywords ) "\\|")) nil "#"
1247 (lambda (arg)
1248 (py-goto-beyond-block)
1249 (skip-chars-backward " \t\n"))
1250 nil))
1251
1228 ;; Run the mode hook. Note that py-mode-hook is deprecated.1252 ;; Run the mode hook. Note that py-mode-hook is deprecated.
1229 (if python-mode-hook1253 (if python-mode-hook
1230 (run-hooks 'python-mode-hook)1254 (run-hooks 'python-mode-hook)

Subscribers

People subscribed via source and target branches

to status/vote changes: