emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* Bug: org-agenda-query-and-cmd results in (wrong-type-argument char-or-string-p nil) when org-agenda-query-string is unset [6.33trans (release_6.33c.27.g1bb0d)]
@ 2009-11-19 22:59 Friedrich Delgado Friedrichs
       [not found] ` <friedel@nomaden.org>
  0 siblings, 1 reply; 13+ messages in thread
From: Friedrich Delgado Friedrichs @ 2009-11-19 22:59 UTC (permalink / raw)
  To: emacs-orgmode


[-- Attachment #1.1: Type: text/plain, Size: 8770 bytes --]

Hiho!

I can't use the '/' command from the agenda.

Here's a backtrace:

Debugger entered--Lisp error: (wrong-type-argument char-or-string-p nil)
  insert("Query:    " nil "\n")
  (save-window-excursion (if expert (set-buffer ...) (delete-other-windows) (split-window-vertically) (org-switch-to-buffer-other-window ...)) (erase-buffer) (org-set-local (quote org-done-keywords) done-keywords) (insert "Query:    " current "\n") (org-agenda-query-op-line op) (insert "\n\n") (org-fast-tag-show-exit exit-after-next) (setq tbl fulltable char 97 cnt 0) (while (setq e ...) (cond ... ... ...)) (setq ntable (nreverse ntable)) (insert "\n") (goto-char (point-min)) (if (and ... ...) (fit-window-to-buffer)) (setq rtn (catch ... ...)) (if rtn current nil))
  (let* ((fulltable ...) (maxlen ...) (fwidth ...) (ncol ...) (expert ...) (exit-after-next org-agenda-query-selection-single-key) (done-keywords org-done-keywords) tbl char cnt e groups ingroup tg c2 c c1 ntable rtn) (save-window-excursion (if expert ... ... ... ...) (erase-buffer) (org-set-local ... done-keywords) (insert "Query:    " current "\n") (org-agenda-query-op-line op) (insert "\n\n") (org-fast-tag-show-exit exit-after-next) (setq tbl fulltable char 97 cnt 0) (while ... ...) (setq ntable ...) (insert "\n") (goto-char ...) (if ... ...) (setq rtn ...) (if rtn current nil)))
  org-agenda-query-selection(nil "+" nil (("REMINDER" . 114) ("DONE" . 107) ("CANCELLED" . 99) ("DOING" . 100) ("TODO" . 116)))
  (let ((q ...)) (when q (setq org-agenda-query-string q) (org-agenda-redo)))
  org-agenda-query-generic-cmd("+")
  org-agenda-query-and-cmd()
  call-interactively(org-agenda-query-and-cmd nil nil)

Not sure what's wrong here, because it looks like this works for other
people.

org-agenda-query-string seems to be nil by default and it's only set
on the first successful query

Emacs  : GNU Emacs 23.1.50.1 (x86_64-unknown-linux-gnu, GTK+ Version 2.18.3)
 of 2009-11-11 on abrasax
Package: Org-mode version 6.33trans (release_6.33c.27.g1bb0d)

current state (redacted):
=========================
(setq
 org-mutt-link-terminal-emulator "runinscreen"
 org-log-done '(state)
 org-clock-in-switch-to-state "DOING"
 org-export-with-LaTeX-fragments t
 org-archive-default-command 'org-toggle-archive-tag
 org-todo-keyword-faces '(("REMINDER" :foreground "green4"))
 org-special-ctrl-a/e t
 org-agenda-files-common-someday '("~/Org/Someday.org")
 org-agenda-custom-commands '(("d" todo "DOING" nil)
                              ("S" tags "STALLED" ((org-use-tag-inheritance nil)))
                              ("w" tags "WAITING" ((org-use-tag-inheritance nil)))
                              ("r" "Describe command here" todo "REMINDER" nil)
                              ("l" "\"Shopping List\" (just the headings)"
                               tags-todo "List"
                               ((org-agenda-remove-tags t)
                                (org-agenda-prefix-format "")
                                (org-agenda-todo-keyword-format "")
                                (org-agenda-sorting-strategy
                                 (quote (category-up priority-down)))
                                (org-habit-graph-column nil))
                               )
                              ("p" "Project List" tags "Project"
                               ((org-use-tag-inheritance nil)))
                              )
 org-agenda-include-diary t
 org-blocker-hook '(org-depend-block-todo)
 org-export-with-sub-superscripts '{}
 org-hide-leading-stars t
 org-mairix-display-hook 'org-mairix-gnus-display-results
 org-agenda-exporter-settings '((htmlize-output-type (quote inline-css)))
 org-checklist-export-function 'org-export-as-ascii
 org-completion-use-ido t
 org-startup-truncated nil
 org-agenda-skip-timestamp-if-done t
 org-after-todo-state-change-hook '(org-checklist org-clock-out-if-current)
 org-combined-agenda-icalendar-file "~/git/Org/home/org.ics"
 org-agenda-todo-ignore-scheduled t
 org-clock-history-length 15
 org-icalendar-timezone "Europe/Berlin"
 org-startup-folded 'content
 org-odd-levels-only t
 org-use-fast-tag-selection nil
 outline-minor-mode-prefix "\x03\x11"
 org-special-ctrl-k t
 org-id-include-domain t
 org-agenda-remove-times-when-in-prefix 'beg
 org-agenda-sorting-strategy '((agenda time-up priority-down category-keep)
                               (todo priority-down category-keep tag-down)
                               (tags priority-down category-keep tag-down)
                               (search category-keep))
 org-agenda-columns-add-appointments-to-effort-sum t
 org-keep-stored-link-after-insertion t
 org-agenda-skip-scheduled-if-done t
 org-stuck-projects '("+Project/-DONE" ("TODO" "DOING") ("WAITING") "")
 org-clock-in-resume t
 org-trigger-hook '(org-depend-trigger-todo)
 org-export-preprocess-hook '(org-export-blocks-preprocess)
 org-tags-exclude-from-inheritance '("Project")
 org-tab-first-hook '(org-hide-block-toggle-maybe)
 org-src-mode-hook '(org-src-mode-configure-edit-buffer)
 org-export-with-priority t
 org-confirm-shell-link-function 'yes-or-no-p
 org-clock-persist t
 org-agenda-before-write-hook '(org-agenda-add-entry-text)
 org-icalendar-include-sexps nil
 org-effort-property "EFFORT"
 org-icalendar-use-scheduled '(event-if-not-todo event-if-todo)
 org-default-notes-file "~/Org/Notes.org"
 org-use-property-inheritance '("DEADLINE PRIORITY")
 org-directory "~/Org"
 org-insert-heading-hook '((lambda nil
                            (save-excursion (insert "\n")
                             (org-insert-time-stamp (current-time) t t)
                             (org-indent-line-function))
                            )
                           )
 org-export-mark-todo-in-toc t
 org-export-html-use-infojs t
 org-blank-before-new-entry '((heading) (plain-list-item))
 org-habit-graph-column 50
 org-use-extra-keys t
 org-icalendar-verify-function 'my-icalendar-verify
 org-n-level-faces 12
 org-cycle-hook '(org-cycle-hide-archived-subtrees org-cycle-hide-drawers
                  org-cycle-show-empty-lines
                  org-optimize-window-after-visibility-change)
 org-link-to-org-use-id t
 org-mode-hook '((lambda nil (setf comment-start nil)) imenu-add-menubar-index
                 (lambda nil
                  (org-add-hook (quote change-major-mode-hook)
                   (quote org-show-block-all) (quote append) (quote local))
                  )
                 )
 org-clock-out-remove-zero-time-clocks t
 org-mairix-gnus-select-display-group-function 'org-mairix-gnus-select-display-group-function-gg
 org-global-properties '(("EFFORT_ALL" .
                          "0 0:02:00 0:05:00 0:10:00 0:20:0 0:30:00 1 2 3 4 5 6 7 8 16 24 32 40 48 56 64")
                         )
 org-refile-targets '((org-agenda-files :maxlevel . 5))
 org-return-follows-link t
 org-confirm-elisp-link-function 'yes-or-no-p
 org-icalendar-use-deadline '(event-if-not-todo event-if-todo)
 org-refile-use-outline-path 'full-file-path
 org-fast-tag-selection-single-key t
 org-icalendar-categories '(all-tags category todo-state)
 org-log-into-drawer t
 org-agenda-start-on-weekday nil
 org-agenda-skip-deadline-if-done t
 org-refile-allow-creating-parent-nodes 'confirm
 org-insert-heading-respect-content t
 org-icalendar-store-UID t
 org-occur-hook '(org-first-headline-recenter)
 org-icalendar-include-body 1000
 org-icalendar-use-plain-timestamp nil
 org-drawers '("PROPERTIES" "CLOCK" "LOGBOOK" "SCHEDULE" "HIDDEN")
 org-agenda-todo-ignore-deadlines t
 org-modules '(org-bbdb org-bibtex org-gnus org-id org-info org-jsinfo org-habit
               org-irc org-mew org-mhe org-protocol org-rmail org-vm org-wl org-w3m
               org-annotate-file org-bookmark org-depend org-elisp-symbol
               org-interactive-query org-mairix org-man org-toc)
 org-columns-default-format "%66ITEM %8TODO %3PRIORITY %SCHEDULED %DEADLINE %6EFFORT{:} %TAGS %5CLOCKSUM{:}"
 org-agenda-auto-exclude-function 'org-my-auto-exclude-function
 org-agenda-after-show-hook '(show-all)
 org-remember-templates '(("" 119 "* %?\n%u\n%c\n%a\n%i" "~/Org/Collect.org"
                           "Web links" nil)
                          ("" 116 "* TODO %?\n  %u\n  %a\n  %c\n  %i"
                           "~/Org/Collect.org" "Tasks" nil)
                          ("" 110 "* %?\n  %u\n  %a\n  %c\n  %i"
                           "~/Org/Collect.org" "Notes" nil)
                          )
 org-ellipsis 'org-ellipsis
 )

-- 
        Friedrich Delgado Friedrichs <friedel@nomaden.org>
                             TauPan on Ircnet and Freenode ;)

[-- Attachment #1.2: Type: application/pgp-signature, Size: 198 bytes --]

[-- Attachment #2: Type: text/plain, Size: 204 bytes --]

_______________________________________________
Emacs-orgmode mailing list
Remember: use `Reply All' to send replies to the list.
Emacs-orgmode@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-orgmode

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Bug: org-agenda-query-and-cmd results in (wrong-type-argument char-or-string-p nil) when org-agenda-query-string is unset [6.33trans (release_6.33c.27.g1bb0d)]
       [not found] ` <friedel@nomaden.org>
@ 2009-11-19 23:17   ` Nick Dokos
  2009-11-20 11:00     ` Friedrich Delgado Friedrichs
  2010-07-15 22:39   ` Patchwork: Patch 138 Accepted Nick Dokos
  1 sibling, 1 reply; 13+ messages in thread
From: Nick Dokos @ 2009-11-19 23:17 UTC (permalink / raw)
  To: emacs-orgmode

Friedrich Delgado Friedrichs <friedel@nomaden.org> wrote:

> Hiho!
> 
> I can't use the '/' command from the agenda.
> 
> Here's a backtrace:
> 
> Debugger entered--Lisp error: (wrong-type-argument char-or-string-p nil)
>   insert("Query:    " nil "\n")
>   (save-window-excursion (if expert (set-buffer ...) (delete-other-windows) (split-window-vertically) (org-switch-to-buffer-other-window ...)) (erase-buffer) (org-set-local (quote org-done-keywords) done-keywords) (insert "Query:    " current "\n") (org-agenda-query-op-line op) (insert "\n\n") (org-fast-tag-show-exit exit-after-next) (setq tbl fulltable char 97 cnt 0) (while (setq e ...) (cond ... ... ...)) (setq ntable (nreverse ntable)) (insert "\n") (goto-char (point-min)) (if (and ... ...) (fit-window-to-buffer)) (setq rtn (catch ... ...)) (if rtn current nil))
>   (let* ((fulltable ...) (maxlen ...) (fwidth ...) (ncol ...) (expert ...) (exit-after-next org-agenda-query-selection-single-key) (done-keywords org-done-keywords) tbl char cnt e groups ingroup tg c2 c c1 ntable rtn) (save-window-excursion (if expert ... ... ... ...) (erase-buffer) (org-set-local ... done-keywords) (insert "Query:    " current "\n") (org-agenda-query-op-line op) (insert "\n\n") (org-fast-tag-show-exit exit-after-next) (setq tbl fulltable char 97 cnt 0) (while ... ...) (setq ntable ...) (insert "\n") (goto-char ...) (if ... ...) (setq rtn ...) (if rtn current nil)))
>   org-agenda-query-selection(nil "+" nil (("REMINDER" . 114) ("DONE" . 107) ("CANCELLED" . 99) ("DOING" . 100) ("TODO" . 116)))
>   (let ((q ...)) (when q (setq org-agenda-query-string q) (org-agenda-redo)))
>   org-agenda-query-generic-cmd("+")
>   org-agenda-query-and-cmd()
>   call-interactively(org-agenda-query-and-cmd nil nil)
> 
> Not sure what's wrong here, because it looks like this works for other
> people.
> 

Perhaps that's because you are loading contrib/lisp/org-interactive-query.el,
which redefines keys in the agenda mode map:

(org-defkey org-agenda-mode-map "=" 'org-agenda-query-clear-cmd)
(org-defkey org-agenda-mode-map "/" 'org-agenda-query-and-cmd)
(org-defkey org-agenda-mode-map ";" 'org-agenda-query-or-cmd)
(org-defkey org-agenda-mode-map "\\" 'org-agenda-query-not-cmd)

HTH,
Nick

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Bug: org-agenda-query-and-cmd results in (wrong-type-argument char-or-string-p nil) when org-agenda-query-string is unset [6.33trans (release_6.33c.27.g1bb0d)]
  2009-11-19 23:17   ` Nick Dokos
@ 2009-11-20 11:00     ` Friedrich Delgado Friedrichs
  0 siblings, 0 replies; 13+ messages in thread
From: Friedrich Delgado Friedrichs @ 2009-11-20 11:00 UTC (permalink / raw)
  To: emacs-orgmode

Hey!

Case closed! Thanks a lot for spotting this!

Nick Dokos schrieb:
> Perhaps that's because you are loading contrib/lisp/org-interactive-query.el,
> which redefines keys in the agenda mode map:
---Zitatende---

Exactly that was the problem... I think there was a lazy day in the past where
I just said "let's check this out", tried it, never used it again and forgot
about it.

;-}

Best regards
	Friedel
-- 
        Friedrich Delgado Friedrichs <friedel@nomaden.org>
                             TauPan on Ircnet and Freenode ;)

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH] Correct some docstrings [part 2]
@ 2010-07-15 17:54 Philip Rooke
  2010-07-15 18:27 ` Patchwork: Patch 138 Accepted Carsten Dominik
  2010-07-15 18:28 ` [PATCH] Correct some docstrings [part 2] Carsten Dominik
  0 siblings, 2 replies; 13+ messages in thread
From: Philip Rooke @ 2010-07-15 17:54 UTC (permalink / raw)
  To: emacs-orgmode


This is a second patch in a series that makes some straightforward
corrections to a number of docstrings.  Each change is normally to:

- correct a typo, or
- fix up hyperlinks to function or variable names, or
- ensure slightly better conformance with the documentation guidelines
  and tips given in the Elisp manual

No attempt is made to provide missing docstrings or document arguments.

Cheers,

Phil

-- 

 lisp/org-docbook.el    |    8 +++---
 lisp/org-docview.el    |   11 ++++++--
 lisp/org-entities.el   |    8 ++++--
 lisp/org-exp-blocks.el |   58 ++++++++++++++++++++++++------------------------
 lisp/org-exp.el        |   29 ++++++++++++++----------
 lisp/org-feed.el       |    4 +-
 lisp/org-freemind.el   |   14 +++++-----
 lisp/org-habit.el      |    3 ++
 lisp/org-html.el       |   56 +++++++++++++++++++++++++--------------------
 9 files changed, 106 insertions(+), 85 deletions(-)

diff --git a/lisp/org-docbook.el b/lisp/org-docbook.el
index bbd0bed..c136d20 100644
--- a/lisp/org-docbook.el
+++ b/lisp/org-docbook.el
@@ -26,7 +26,7 @@
 ;; You should have received a copy of the GNU General Public License
 ;; along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.
 
-;; Commentary:
+;;; Commentary:
 ;;
 ;; This library implements a DocBook exporter for org-mode.  The basic
 ;; idea and design is very similar to what `org-export-as-html' has.
@@ -142,8 +142,8 @@ people work on the same document."
   :type 'string)
 
 (defcustom org-export-docbook-footnote-id-prefix "fn-"
-  "The prefix of footnote IDs used during exporting.  Like
-`org-export-docbook-section-id-prefix', this variable can help
+  "The prefix of footnote IDs used during exporting.
+Like `org-export-docbook-section-id-prefix', this variable can help
 avoid same set of footnote IDs being used multiple times."
   :group 'org-export-docbook
   :type 'string)
@@ -155,7 +155,7 @@ avoid same set of footnote IDs being used multiple times."
     ("=" "<code>" "</code>")
     ("~" "<literal>" "</literal>")
     ("+" "<emphasis role=\"strikethrough\">" "</emphasis>"))
-  "Alist of DocBook expressions to convert emphasis fontifiers.
+  "A list of DocBook expressions to convert emphasis fontifiers.
 Each element of the list is a list of three elements.
 The first element is the character used as a marker for fontification.
 The second element is a formatting string to wrap fontified text with.
diff --git a/lisp/org-docview.el b/lisp/org-docview.el
index 5db6594..b3a5178 100644
--- a/lisp/org-docview.el
+++ b/lisp/org-docview.el
@@ -62,7 +62,7 @@
       )))
 
 (defun org-docview-store-link ()
-  "Store a link to a docview buffer"
+  "Store a link to a docview buffer."
   (when (eq major-mode 'doc-view-mode)
     ;; This buffer is in doc-view-mode
     (let* ((path buffer-file-name)
@@ -75,11 +75,16 @@
        :description path))))
 
 (defun org-docview-complete-link ()
-  "Use the existing file name completion for file: links to get the file name,
-   then ask the user for the page number and append it."
+  "Use the existing file name completion for file.
+Links to get the file name, then ask the user for the page number
+and append it."
   (concat (replace-regexp-in-string "^file:" "docview:" (org-file-complete-link))
 	  "::"
 	  (read-from-minibuffer "Page:" "1")))
 
 
 (provide 'org-docview)
+
+(provide 'org-docview)
+
+;;; org-docview.el ends here
diff --git a/lisp/org-entities.el b/lisp/org-entities.el
index 59734dc..501eb1c 100644
--- a/lisp/org-entities.el
+++ b/lisp/org-entities.el
@@ -26,6 +26,8 @@
 ;;
 ;;; Commentary:
 
+;;; Code:
+
 (require 'org-macs)
 
 (declare-function org-table-align "org-table" ())
@@ -459,7 +461,7 @@ loaded, add these packages to `org-export-latex-packages-alist'."
     ("Diamond" "\\diamond" t "&diamond;" "[diamond]" "[diamond]" "⋄")
     ("loz" "\\diamond" t "&loz;" "[lozenge]" "[lozenge]" "◊")
     )
-  "Default entities used in Org-mode to preduce special characters.
+  "Default entities used in Org-mode to produce special characters.
 For details see `org-entities-user'.")
 
 (defsubst org-entity-get (name)
@@ -521,7 +523,7 @@ Kind can be any of `latex', `html', `ascii', `latin1', or `utf8'."
     (org-table-align)))
 
 (defun org-entities-help ()
-  "Create a Help buffer with all available entities"
+  "Create a Help buffer with all available entities."
   (interactive)
   (with-output-to-temp-buffer "*Org Entity Help*"
     (princ "Org-mode entities\n=================\n\n")
@@ -555,7 +557,7 @@ Kind can be any of `latex', `html', `ascii', `latin1', or `utf8'."
 
 
 (defun replace-amp ()
-  "Postprocess HTML file to unescape the ampersant."
+  "Postprocess HTML file to unescape the ampersand."
   (interactive)
   (while (re-search-forward "<td>&amp;\\([^<;]+;\\)" nil t)
     (replace-match (concat "<td>&" (match-string 1)) t t)))
diff --git a/lisp/org-exp-blocks.el b/lisp/org-exp-blocks.el
index 4bf5db6..bbe245c 100644
--- a/lisp/org-exp-blocks.el
+++ b/lisp/org-exp-blocks.el
@@ -68,6 +68,8 @@
 ;; `org-export-blocks-add-block' to add your block type to
 ;; `org-export-blocks'.
 
+;;; Code:
+
 (eval-when-compile
   (require 'cl))
 (require 'org)
@@ -93,10 +95,10 @@
   '((comment org-export-blocks-format-comment t)
     (ditaa org-export-blocks-format-ditaa nil)
     (dot org-export-blocks-format-dot nil))
-  "Use this a-list to associate block types with block exporting
-functions.  The type of a block is determined by the text
-immediately following the '#+BEGIN_' portion of the block header.
-Each block export function should accept three argumets..."
+  "Use this alist to associate block types with block exporting functions.
+The type of a block is determined by the text immediately
+following the '#+BEGIN_' portion of the block header.  Each block
+export function should accept three arguments."
   :group 'org-export-general
   :type '(repeat
 	  (list
@@ -106,14 +108,14 @@ Each block export function should accept three argumets..."
   :set 'org-export-blocks-set)
 
 (defun org-export-blocks-add-block (block-spec)
-  "Add a new block type to `org-export-blocks'.  BLOCK-SPEC
-should be a three element list the first element of which should
-indicate the name of the block, the second element should be the
-formatting function called by `org-export-blocks-preprocess' and
-the third element a flag indicating whether these types of blocks
-should be fontified in org-mode buffers (see
-`org-protecting-blocks').  For example the BLOCK-SPEC for ditaa
-blocks is as follows...
+  "Add a new block type to `org-export-blocks'.
+BLOCK-SPEC should be a three element list the first element of
+which should indicate the name of the block, the second element
+should be the formatting function called by
+`org-export-blocks-preprocess' and the third element a flag
+indicating whether these types of blocks should be fontified in
+org-mode buffers (see `org-protecting-blocks').  For example the
+BLOCK-SPEC for ditaa blocks is as follows.
 
   (ditaa org-export-blocks-format-ditaa nil)"
   (unless (member block-spec org-export-blocks)
@@ -122,29 +124,28 @@ blocks is as follows...
 
 (defcustom org-export-interblocks
   '()
-  "Use this a-list to associate block types with block exporting
-functions.  The type of a block is determined by the text
-immediately following the '#+BEGIN_' portion of the block header.
-Each block export function should accept three argumets..."
+  "Use this a-list to associate block types with block exporting functions.
+The type of a block is determined by the text immediately
+following the '#+BEGIN_' portion of the block header.  Each block
+export function should accept three arguments."
   :group 'org-export-general
   :type 'alist)
 
 (defcustom org-export-blocks-witheld
   '(hidden)
-  "List of block types (see `org-export-blocks') which should not
-be exported."
+  "List of block types (see `org-export-blocks') which should not be exported."
   :group 'org-export-general
   :type 'list)
 
 (defcustom org-export-blocks-postblock-hook nil
-  "Run after blocks have been processed with
-`org-export-blocks-preprocess'."
+  "Run after blocks have been processed with `org-export-blocks-preprocess'."
   :group 'org-export-general
   :type 'hook)
 
 (defun org-export-blocks-html-quote (body &optional open close)
-  "Protext BODY from org html export.  The optional OPEN and
-CLOSE tags will be inserted around BODY."
+  "Protect BODY from org html export.
+The optional OPEN and CLOSE tags will be inserted around BODY."
+
   (concat
    "\n#+BEGIN_HTML\n"
    (or open "")
@@ -153,8 +154,8 @@ CLOSE tags will be inserted around BODY."
    "#+END_HTML\n"))
 
 (defun org-export-blocks-latex-quote (body &optional open close)
-  "Protext BODY from org latex export.  The optional OPEN and
-CLOSE tags will be inserted around BODY."
+  "Protect BODY from org latex export.
+The optional OPEN and CLOSE tags will be inserted around BODY."
   (concat
    "\n#+BEGIN_LaTeX\n"
    (or open "")
@@ -163,10 +164,9 @@ CLOSE tags will be inserted around BODY."
    "#+END_LaTeX\n"))
 
 (defun org-export-blocks-preprocess ()
-  "Export all blocks according to the `org-export-blocks' block
-exportation alist.  Does not export block types specified in
-specified in BLOCKS which default to the value of
-`org-export-blocks-witheld'."
+  "Export all blocks according to the `org-export-blocks' block export alist.
+Does not export block types specified in specified in BLOCKS
+which defaults to the value of `org-export-blocks-witheld'."
   (interactive)
   (save-window-excursion
     (let ((case-fold-search t)
@@ -217,7 +217,7 @@ specified in BLOCKS which default to the value of
 			       (expand-file-name
 				"../contrib"
 				(file-name-directory (or load-file-name buffer-file-name)))))))
-  "Path to the ditaa jar executable")
+  "Path to the ditaa jar executable.")
 
 (defun org-export-blocks-format-ditaa (body &rest headers)
   "Pass block BODY to the ditaa utility creating an image.
diff --git a/lisp/org-exp.el b/lisp/org-exp.el
index 55b8812..ae10cc4 100644
--- a/lisp/org-exp.el
+++ b/lisp/org-exp.el
@@ -26,12 +26,15 @@
 ;;
 ;;; Commentary:
 
+;;; Code:
+
 (require 'org)
 (require 'org-macs)
 (require 'org-agenda)
 (require 'org-exp-blocks)
 (require 'ob-exp)
 (require 'org-src)
+
 (eval-when-compile
   (require 'cl))
 
@@ -96,9 +99,10 @@ This works by starting up a separate Emacs process visiting the same file
 and doing the export from there.
 Not all export commands are affected by this - only the ones which
 actually write to a file, and that do not depend on the buffer state.
-
+\\<org-mode-map>
 If this option is nil, you can still get background export by calling
-`org-export' with a double prefix arg: `C-u C-u C-c C-e'.
+`org-export' with a double prefix arg: \
+\\[universal-argument] \\[universal-argument] \\[org-export].
 
 If this option is t, the double prefix can be used to exceptionally
 force an export command into the current process."
@@ -843,7 +847,8 @@ in the background.  This will be done only for commands that write
 to a file.  For details see the docstring of `org-export-run-in-background'.
 
 The prefix argument ARG will be passed to the exporter.  However, if
-ARG is a double universal prefix `C-u C-u', that means to inverse the
+ARG is a double universal prefix \\[universal-argument] \\[universal-argument], \
+that means to inverse the
 value of `org-export-run-in-background'."
   (interactive "P")
   (let* ((bg (org-xor (equal arg '(16)) org-export-run-in-background))
@@ -977,7 +982,7 @@ value of `org-export-run-in-background'."
 (defvar org-export-id-target-alist nil
   "Alist of section id's with preferred aliases.")
 (defvar org-export-code-refs nil
-  "Alist of code references and line numbers")
+  "Alist of code references and line numbers.")
 
 (defun org-export-preprocess-string (string &rest parameters)
   "Cleanup STRING so that that the true exported has a more consistent source.
@@ -1183,7 +1188,7 @@ on this string to produce the exported version."
 (defun org-export-define-heading-targets (target-alist)
   "Find all headings and define the targets for them.
 The new targets are added to TARGET-ALIST, which is also returned.
-Also find all ID and CUSTOM_ID propertiess and store them."
+Also find all ID and CUSTOM_ID properties and store them."
   (goto-char (point-min))
   (org-init-section-numbers)
   (let ((re (concat "^" org-outline-regexp
@@ -2051,7 +2056,7 @@ TYPE must be a string, any of:
 	     (delq nil
 		   (mapcar (lambda (el) (when (member el all) el)) now))))
 	(when intersection
-	  (error "recursive #+INCLUDE: %S" intersection))))))
+	  (error "Recursive #+INCLUDE: %S" intersection))))))
 
 (defun org-get-file-contents (file &optional prefix prefix1 markup)
   "Get the contents of FILE and return them as a string.
@@ -2111,7 +2116,7 @@ in the list) and remove property and value from the list in LISTVAR."
 	    nil t)
       (if (match-end 1)
 	  (if (not (match-string 4))
-	      (error "source block missing language specification: %s"
+	      (error "Source block missing language specification: %s"
 		     (let* ((body (match-string 6))
 			    (nothing (message "body:%s" body))
 			    (preview (or (and (string-match
@@ -2406,8 +2411,8 @@ INDENT was the original indentation of the block."
 (defun org-export-visible (type arg)
   "Create a copy of the visible part of the current buffer, and export it.
 The copy is created in a temporary buffer and removed after use.
-TYPE is the final key (as a string) that also select the export command in
-the `C-c C-e' export dispatcher.
+TYPE is the final key (as a string) that also selects the export command in
+the \\<org-mode-map>\\[org-export] export dispatcher.
 As a special case, if the you type SPC at the prompt, the temporary
 org-mode file will not be removed but presented to you so that you can
 continue to use it.  The prefix arg ARG is passed through to the exporting
@@ -2597,8 +2602,8 @@ Does include HTML export options as well as TODO and CATEGORY stuff."
 #+AUTHOR:    %s
 #+EMAIL:     %s
 #+DATE:      %s
-#+DESCRIPTION: 
-#+KEYWORDS: 
+#+DESCRIPTION:
+#+KEYWORDS:
 #+LANGUAGE:  %s
 #+OPTIONS:   H:%d num:%s toc:%s \\n:%s @:%s ::%s |:%s ^:%s -:%s f:%s *:%s <:%s
 #+OPTIONS:   TeX:%s LaTeX:%s skip:%s d:%s todo:%s pri:%s tags:%s
@@ -2607,7 +2612,7 @@ Does include HTML export options as well as TODO and CATEGORY stuff."
 #+EXPORT_EXCLUDE_TAGS: %s
 #+LINK_UP:   %s
 #+LINK_HOME: %s
-#+XSLT: 
+#+XSLT:
 #+CATEGORY:  %s
 #+SEQ_TODO:  %s
 #+TYP_TODO:  %s
diff --git a/lisp/org-feed.el b/lisp/org-feed.el
index 2af97e5..58c552b 100644
--- a/lisp/org-feed.el
+++ b/lisp/org-feed.el
@@ -83,8 +83,8 @@
 ;;
 ;;       #+DRAWERS: PROPERTIES LOGBOOK FEEDSTATUS
 ;;
-;;  Acknowledgements
-;;  ----------------
+;;  Acknowledgments
+;;  ---------------
 ;;
 ;;  org-feed.el is based on ideas by Brad Bozarth who implemented a
 ;;  similar mechanism using shell and awk scripts.
diff --git a/lisp/org-freemind.el b/lisp/org-freemind.el
index 538b404..d066960 100644
--- a/lisp/org-freemind.el
+++ b/lisp/org-freemind.el
@@ -196,7 +196,7 @@ NOT READY YET."
 
 ;;;###autoload
 (defun org-freemind-show (mm-file)
-  "Show file MM-FILE in Freemind."
+  "Show file MM-FILE in FreeMind."
   (interactive
    (list
     (save-match-data
@@ -276,7 +276,7 @@ will also unescape &#nn;."
     ))
 
 (defun org-freemind-convert-links-from-org (org-str)
-  "Convert org links in ORG-STR to freemind links and return the result."
+  "Convert org links in ORG-STR to FreeMind links and return the result."
   (let ((fm-str (replace-regexp-in-string
                  (rx (not (any "[\""))
                      (submatch
@@ -297,7 +297,7 @@ will also unescape &#nn;."
 
 ;;(org-freemind-convert-links-to-org "<a href=\"http://www.somewhere/\">link-text</a>")
 (defun org-freemind-convert-links-to-org (fm-str)
-  "Convert freemind links in FM-STR to org links and return the result."
+  "Convert FreeMind links in FM-STR to org links and return the result."
   (let ((org-str (replace-regexp-in-string
                   (rx "<a"
                       space
@@ -344,10 +344,10 @@ will also unescape &#nn;."
           "</p>\n"))
 
 (defun org-freemind-org-text-to-freemind-subnode/note (node-name start end drawers-regexp)
-  "Convert text part of org node to freemind subnode or note.
-Convert the text part of the org node named NODE-NAME. The text
-is in the current buffer between START and END. Drawers matching
-DRAWERS-REGEXP are converted to freemind notes."
+  "Convert text part of org node to FreeMind subnode or note.
+Convert the text part of the org node named NODE-NAME.  The text
+is in the current buffer between START and END.  Drawers matching
+DRAWERS-REGEXP are converted to FreeMind notes."
   ;; fix-me: doc
   (let ((text (buffer-substring-no-properties start end))
         (node-res "")
diff --git a/lisp/org-habit.el b/lisp/org-habit.el
index 97c2bf7..f4b990e 100644
--- a/lisp/org-habit.el
+++ b/lisp/org-habit.el
@@ -27,8 +27,11 @@
 
 ;; This file contains the habit tracking code for Org-mode
 
+;;; Code:
+
 (require 'org)
 (require 'org-agenda)
+
 (eval-when-compile
   (require 'cl))
 
diff --git a/lisp/org-html.el b/lisp/org-html.el
index d972c58..3889778 100644
--- a/lisp/org-html.el
+++ b/lisp/org-html.el
@@ -26,7 +26,10 @@
 ;;
 ;;; Commentary:
 
+;;; Code:
+
 (require 'org-exp)
+
 (eval-when-compile (require 'cl))
 
 (declare-function org-id-find-id-file "org-id" (id))
@@ -57,7 +60,7 @@ by the footnotes themselves."
   :type 'string)
 
 (defcustom org-export-html-coding-system nil
-  "Coding system for HTML export, defaults to buffer-file-coding-system."
+  "Coding system for HTML export, defaults to `buffer-file-coding-system'."
   :group 'org-export-html
   :type 'coding-system)
 
@@ -81,7 +84,7 @@ and corresponding declarations."
 			(string :tag "Declaration")))))
 
 (defcustom org-export-html-style-include-scripts t
-  "Non-nil means include the javascript snippets in exported HTML files.
+  "Non-nil means include the JavaScript snippets in exported HTML files.
 The actual script is defined in `org-export-html-scripts' and should
 not be modified."
   :group 'org-export-html
@@ -110,7 +113,7 @@ not be modified."
  }
 /*]]>*///-->
 </script>"
-"Basic javascript that is needed by HTML files produced by Org-mode.")
+"Basic JavaScript that is needed by HTML files produced by Org-mode.")
 
 (defconst org-export-html-style-default
 "<style type=\"text/css\">
@@ -207,20 +210,20 @@ settings with <style>...</style> tags."
 (put 'org-export-html-style-extra 'safe-local-variable 'stringp)
 
 (defcustom org-export-html-tag-class-prefix ""
-  "Prefix to clas names for TODO keywords.
+  "Prefix to class names for TODO keywords.
 Each tag gets a class given by the tag itself, with this prefix.
 The default prefix is empty because it is nice to just use the keyword
 as a class name.  But if you get into conflicts with other, existing
-CSS classes, then this prefic can be very useful."
+CSS classes, then this prefix can be very useful."
   :group 'org-export-html
   :type 'string)
 
 (defcustom org-export-html-todo-kwd-class-prefix ""
-  "Prefix to clas names for TODO keywords.
+  "Prefix to class names for TODO keywords.
 Each TODO keyword gets a class given by the keyword itself, with this prefix.
 The default prefix is empty because it is nice to just use the keyword
 as a class name.  But if you get into conflicts with other, existing
-CSS classes, then this prefic can be very useful."
+CSS classes, then this prefix can be very useful."
   :group 'org-export-html
   :type 'string)
 
@@ -235,10 +238,11 @@ CSS classes, then this prefic can be very useful."
  |
  <a accesskey=\"H\" href=\"%s\"> HOME </a>
 </div>"
-  "Snippet used to insert the HOME and UP links.  This is a format,
-the first %s will receive the UP link, the second the HOME link.
-If both `org-export-html-link-up' and `org-export-html-link-home' are
-empty, the entire snippet will be ignored."
+  "Snippet used to insert the HOME and UP links.
+This is a format string, the first %s will receive the UP link,
+the second the HOME link.  If both `org-export-html-link-up' and
+`org-export-html-link-home' are empty, the entire snippet will be
+ignored."
   :group 'org-export-html
   :type 'string)
 
@@ -340,7 +344,7 @@ When nil, also column one will use data tags."
   :type 'boolean)
 
 (defcustom org-export-html-validation-link nil
-  "Non-nil means add validationlink to postamble of HTML exported files."
+  "Non-nil means add validation link to postamble of HTML exported files."
   :group 'org-export-html
   :type '(choice
 	  (const :tag "Nothing" nil)
@@ -349,9 +353,10 @@ When nil, also column one will use data tags."
 
 
 (defcustom org-export-html-with-timestamp nil
-  "If non-nil, write `org-export-html-html-helper-timestamp'
-into the exported HTML text.  Otherwise, the buffer will just be saved
-to a file."
+  "If non-nil, write timestamp into the exported HTML text.
+If non-nil Write `org-export-html-html-helper-timestamp' into the
+exported HTML text.  Otherwise, the buffer will just be saved to
+a file."
   :group 'org-export-html
   :type 'boolean)
 
@@ -426,7 +431,7 @@ This may also be a function, building and inserting the postamble.")
 ;;; HTML export
 
 (defun org-export-html-preprocess (parameters)
-  ;; Convert LaTeX fragments to images
+  "Convert LaTeX fragments to images."
   (when (and org-current-export-file
 	     (plist-get parameters :LaTeX-fragments))
     (org-format-latex
@@ -460,7 +465,8 @@ headlines.  The default is 3.  Lower levels will become bulleted lists."
 
 ;;;###autoload
 (defun org-export-as-html-batch ()
-  "Call `org-export-as-html', may be used in batch processing as
+  "Call the function `org-export-as-html'.
+This function can be used in batch processing as:
 emacs   --batch
         --load=$HOME/lib/emacs/org.el
         --eval \"(setq org-export-headline-levels 2)\"
@@ -540,12 +546,12 @@ in a window.  A non-interactive call will only return the buffer."
    nil
    "Function to convert link URLs to exportable URLs.
 Takes two arguments, TYPE and PATH.
-Returns exportable url as (TYPE PATH), or `nil' to signal that it
+Returns exportable url as (TYPE PATH), or nil to signal that it
 didn't handle this case.
 Intended to be locally bound around a call to `org-export-as-html'." )
 
 (defun org-html-cvt-org-as-html (opt-plist type path)
-   "Convert and org filename to an equivalent html filename.
+   "Convert an org filename to an equivalent html filename.
 If TYPE is not file, just return `nil'.
 See variable `org-export-html-link-org-files-as-html'"
 
@@ -565,10 +571,10 @@ See variable `org-export-html-link-org-files-as-html'"
 
 ;;; org-html-should-inline-p
 (defun org-html-should-inline-p (filename descp)
-   "Return non-nil if link FILENAME should be inlined, according to
-current settings.
-DESCP is the boolean of whether there was a link description.
-See variables `org-export-html-inline-images' and
+   "Return non-nil if link FILENAME should be inlined.
+The decision to inline the FILENAME link is based on the current
+settings.  DESCP is the boolean of whether there was a link
+description.  See variables `org-export-html-inline-images' and
 `org-export-html-inline-image-extensions'."
    (declare (special
 	     org-export-html-inline-images
@@ -2018,7 +2024,7 @@ that uses these same face definitions."
   (goto-char (point-min)))
 
 (defun org-html-protect (s)
-  ;; convert & to &amp;, < to &lt; and > to &gt;
+  "convert & to &amp;, < to &lt; and > to &gt;"
   (let ((start 0))
     (while (string-match "&" s start)
       (setq s (replace-match "&amp;" t t s)
@@ -2033,7 +2039,7 @@ that uses these same face definitions."
   s)
 
 (defun org-html-expand (string)
-  "Prepare STRING for HTML export.  Applies all active conversions.
+  "Prepare STRING for HTML export.  Apply all active conversions.
 If there are links in the string, don't modify these."
   (let* ((re (concat org-bracket-link-regexp "\\|"
 		     (org-re "[ \t]+\\(:[[:alnum:]_@:]+:\\)[ \t]*$")))

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* Patchwork: Patch 138 Accepted
  2010-07-15 17:54 [PATCH] Correct some docstrings [part 2] Philip Rooke
@ 2010-07-15 18:27 ` Carsten Dominik
  2010-07-15 21:28   ` Friedrich Delgado
  2010-07-15 18:28 ` [PATCH] Correct some docstrings [part 2] Carsten Dominik
  1 sibling, 1 reply; 13+ messages in thread
From: Carsten Dominik @ 2010-07-15 18:27 UTC (permalink / raw)
  To: emacs-orgmode

Patch 138 (http://patchwork.newartisans.com/patch/138/) is now Accepted.

This relates to the following submission:

http://mid.gmane.org/%3Cm21vb4bpg2.fsf%40bo.yax.org.uk%3E

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH] Correct some docstrings [part 2]
  2010-07-15 17:54 [PATCH] Correct some docstrings [part 2] Philip Rooke
  2010-07-15 18:27 ` Patchwork: Patch 138 Accepted Carsten Dominik
@ 2010-07-15 18:28 ` Carsten Dominik
  1 sibling, 0 replies; 13+ messages in thread
From: Carsten Dominik @ 2010-07-15 18:28 UTC (permalink / raw)
  To: Philip Rooke; +Cc: emacs-orgmode

Accepted, with minor reversals.

- Carsten

On Jul 15, 2010, at 7:54 PM, Philip Rooke wrote:

>
> This is a second patch in a series that makes some straightforward
> corrections to a number of docstrings.  Each change is normally to:
>
> - correct a typo, or
> - fix up hyperlinks to function or variable names, or
> - ensure slightly better conformance with the documentation guidelines
>  and tips given in the Elisp manual
>
> No attempt is made to provide missing docstrings or document  
> arguments.
>
> Cheers,
>
> Phil
>
> -- 
>
> lisp/org-docbook.el    |    8 +++---
> lisp/org-docview.el    |   11 ++++++--
> lisp/org-entities.el   |    8 ++++--
> lisp/org-exp-blocks.el |   58 +++++++++++++++++++++++ 
> +------------------------
> lisp/org-exp.el        |   29 ++++++++++++++----------
> lisp/org-feed.el       |    4 +-
> lisp/org-freemind.el   |   14 +++++-----
> lisp/org-habit.el      |    3 ++
> lisp/org-html.el       |   56 ++++++++++++++++++++++++ 
> +--------------------
> 9 files changed, 106 insertions(+), 85 deletions(-)
>
> diff --git a/lisp/org-docbook.el b/lisp/org-docbook.el
> index bbd0bed..c136d20 100644
> --- a/lisp/org-docbook.el
> +++ b/lisp/org-docbook.el
> @@ -26,7 +26,7 @@
> ;; You should have received a copy of the GNU General Public License
> ;; along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.
>
> -;; Commentary:
> +;;; Commentary:
> ;;
> ;; This library implements a DocBook exporter for org-mode.  The basic
> ;; idea and design is very similar to what `org-export-as-html' has.
> @@ -142,8 +142,8 @@ people work on the same document."
>   :type 'string)
>
> (defcustom org-export-docbook-footnote-id-prefix "fn-"
> -  "The prefix of footnote IDs used during exporting.  Like
> -`org-export-docbook-section-id-prefix', this variable can help
> +  "The prefix of footnote IDs used during exporting.
> +Like `org-export-docbook-section-id-prefix', this variable can help
> avoid same set of footnote IDs being used multiple times."
>   :group 'org-export-docbook
>   :type 'string)
> @@ -155,7 +155,7 @@ avoid same set of footnote IDs being used  
> multiple times."
>     ("=" "<code>" "</code>")
>     ("~" "<literal>" "</literal>")
>     ("+" "<emphasis role=\"strikethrough\">" "</emphasis>"))
> -  "Alist of DocBook expressions to convert emphasis fontifiers.
> +  "A list of DocBook expressions to convert emphasis fontifiers.
> Each element of the list is a list of three elements.
> The first element is the character used as a marker for fontification.
> The second element is a formatting string to wrap fontified text with.
> diff --git a/lisp/org-docview.el b/lisp/org-docview.el
> index 5db6594..b3a5178 100644
> --- a/lisp/org-docview.el
> +++ b/lisp/org-docview.el
> @@ -62,7 +62,7 @@
>       )))
>
> (defun org-docview-store-link ()
> -  "Store a link to a docview buffer"
> +  "Store a link to a docview buffer."
>   (when (eq major-mode 'doc-view-mode)
>     ;; This buffer is in doc-view-mode
>     (let* ((path buffer-file-name)
> @@ -75,11 +75,16 @@
>        :description path))))
>
> (defun org-docview-complete-link ()
> -  "Use the existing file name completion for file: links to get the  
> file name,
> -   then ask the user for the page number and append it."
> +  "Use the existing file name completion for file.
> +Links to get the file name, then ask the user for the page number
> +and append it."
>   (concat (replace-regexp-in-string "^file:" "docview:" (org-file- 
> complete-link))
> 	  "::"
> 	  (read-from-minibuffer "Page:" "1")))
>
>
> (provide 'org-docview)
> +
> +(provide 'org-docview)
> +
> +;;; org-docview.el ends here
> diff --git a/lisp/org-entities.el b/lisp/org-entities.el
> index 59734dc..501eb1c 100644
> --- a/lisp/org-entities.el
> +++ b/lisp/org-entities.el
> @@ -26,6 +26,8 @@
> ;;
> ;;; Commentary:
>
> +;;; Code:
> +
> (require 'org-macs)
>
> (declare-function org-table-align "org-table" ())
> @@ -459,7 +461,7 @@ loaded, add these packages to `org-export-latex- 
> packages-alist'."
>     ("Diamond" "\\diamond" t "&diamond;" "[diamond]" "[diamond]"  
> "⋄")
>     ("loz" "\\diamond" t "&loz;" "[lozenge]" "[lozenge]" "◊")
>     )
> -  "Default entities used in Org-mode to preduce special characters.
> +  "Default entities used in Org-mode to produce special characters.
> For details see `org-entities-user'.")
>
> (defsubst org-entity-get (name)
> @@ -521,7 +523,7 @@ Kind can be any of `latex', `html', `ascii',  
> `latin1', or `utf8'."
>     (org-table-align)))
>
> (defun org-entities-help ()
> -  "Create a Help buffer with all available entities"
> +  "Create a Help buffer with all available entities."
>   (interactive)
>   (with-output-to-temp-buffer "*Org Entity Help*"
>     (princ "Org-mode entities\n=================\n\n")
> @@ -555,7 +557,7 @@ Kind can be any of `latex', `html', `ascii',  
> `latin1', or `utf8'."
>
>
> (defun replace-amp ()
> -  "Postprocess HTML file to unescape the ampersant."
> +  "Postprocess HTML file to unescape the ampersand."
>   (interactive)
>   (while (re-search-forward "<td>&amp;\\([^<;]+;\\)" nil t)
>     (replace-match (concat "<td>&" (match-string 1)) t t)))
> diff --git a/lisp/org-exp-blocks.el b/lisp/org-exp-blocks.el
> index 4bf5db6..bbe245c 100644
> --- a/lisp/org-exp-blocks.el
> +++ b/lisp/org-exp-blocks.el
> @@ -68,6 +68,8 @@
> ;; `org-export-blocks-add-block' to add your block type to
> ;; `org-export-blocks'.
>
> +;;; Code:
> +
> (eval-when-compile
>   (require 'cl))
> (require 'org)
> @@ -93,10 +95,10 @@
>   '((comment org-export-blocks-format-comment t)
>     (ditaa org-export-blocks-format-ditaa nil)
>     (dot org-export-blocks-format-dot nil))
> -  "Use this a-list to associate block types with block exporting
> -functions.  The type of a block is determined by the text
> -immediately following the '#+BEGIN_' portion of the block header.
> -Each block export function should accept three argumets..."
> +  "Use this alist to associate block types with block exporting  
> functions.
> +The type of a block is determined by the text immediately
> +following the '#+BEGIN_' portion of the block header.  Each block
> +export function should accept three arguments."
>   :group 'org-export-general
>   :type '(repeat
> 	  (list
> @@ -106,14 +108,14 @@ Each block export function should accept three  
> argumets..."
>   :set 'org-export-blocks-set)
>
> (defun org-export-blocks-add-block (block-spec)
> -  "Add a new block type to `org-export-blocks'.  BLOCK-SPEC
> -should be a three element list the first element of which should
> -indicate the name of the block, the second element should be the
> -formatting function called by `org-export-blocks-preprocess' and
> -the third element a flag indicating whether these types of blocks
> -should be fontified in org-mode buffers (see
> -`org-protecting-blocks').  For example the BLOCK-SPEC for ditaa
> -blocks is as follows...
> +  "Add a new block type to `org-export-blocks'.
> +BLOCK-SPEC should be a three element list the first element of
> +which should indicate the name of the block, the second element
> +should be the formatting function called by
> +`org-export-blocks-preprocess' and the third element a flag
> +indicating whether these types of blocks should be fontified in
> +org-mode buffers (see `org-protecting-blocks').  For example the
> +BLOCK-SPEC for ditaa blocks is as follows.
>
>   (ditaa org-export-blocks-format-ditaa nil)"
>   (unless (member block-spec org-export-blocks)
> @@ -122,29 +124,28 @@ blocks is as follows...
>
> (defcustom org-export-interblocks
>   '()
> -  "Use this a-list to associate block types with block exporting
> -functions.  The type of a block is determined by the text
> -immediately following the '#+BEGIN_' portion of the block header.
> -Each block export function should accept three argumets..."
> +  "Use this a-list to associate block types with block exporting  
> functions.
> +The type of a block is determined by the text immediately
> +following the '#+BEGIN_' portion of the block header.  Each block
> +export function should accept three arguments."
>   :group 'org-export-general
>   :type 'alist)
>
> (defcustom org-export-blocks-witheld
>   '(hidden)
> -  "List of block types (see `org-export-blocks') which should not
> -be exported."
> +  "List of block types (see `org-export-blocks') which should not  
> be exported."
>   :group 'org-export-general
>   :type 'list)
>
> (defcustom org-export-blocks-postblock-hook nil
> -  "Run after blocks have been processed with
> -`org-export-blocks-preprocess'."
> +  "Run after blocks have been processed with `org-export-blocks- 
> preprocess'."
>   :group 'org-export-general
>   :type 'hook)
>
> (defun org-export-blocks-html-quote (body &optional open close)
> -  "Protext BODY from org html export.  The optional OPEN and
> -CLOSE tags will be inserted around BODY."
> +  "Protect BODY from org html export.
> +The optional OPEN and CLOSE tags will be inserted around BODY."
> +
>   (concat
>    "\n#+BEGIN_HTML\n"
>    (or open "")
> @@ -153,8 +154,8 @@ CLOSE tags will be inserted around BODY."
>    "#+END_HTML\n"))
>
> (defun org-export-blocks-latex-quote (body &optional open close)
> -  "Protext BODY from org latex export.  The optional OPEN and
> -CLOSE tags will be inserted around BODY."
> +  "Protect BODY from org latex export.
> +The optional OPEN and CLOSE tags will be inserted around BODY."
>   (concat
>    "\n#+BEGIN_LaTeX\n"
>    (or open "")
> @@ -163,10 +164,9 @@ CLOSE tags will be inserted around BODY."
>    "#+END_LaTeX\n"))
>
> (defun org-export-blocks-preprocess ()
> -  "Export all blocks according to the `org-export-blocks' block
> -exportation alist.  Does not export block types specified in
> -specified in BLOCKS which default to the value of
> -`org-export-blocks-witheld'."
> +  "Export all blocks according to the `org-export-blocks' block  
> export alist.
> +Does not export block types specified in specified in BLOCKS
> +which defaults to the value of `org-export-blocks-witheld'."
>   (interactive)
>   (save-window-excursion
>     (let ((case-fold-search t)
> @@ -217,7 +217,7 @@ specified in BLOCKS which default to the value of
> 			       (expand-file-name
> 				"../contrib"
> 				(file-name-directory (or load-file-name buffer-file-name)))))))
> -  "Path to the ditaa jar executable")
> +  "Path to the ditaa jar executable.")
>
> (defun org-export-blocks-format-ditaa (body &rest headers)
>   "Pass block BODY to the ditaa utility creating an image.
> diff --git a/lisp/org-exp.el b/lisp/org-exp.el
> index 55b8812..ae10cc4 100644
> --- a/lisp/org-exp.el
> +++ b/lisp/org-exp.el
> @@ -26,12 +26,15 @@
> ;;
> ;;; Commentary:
>
> +;;; Code:
> +
> (require 'org)
> (require 'org-macs)
> (require 'org-agenda)
> (require 'org-exp-blocks)
> (require 'ob-exp)
> (require 'org-src)
> +
> (eval-when-compile
>   (require 'cl))
>
> @@ -96,9 +99,10 @@ This works by starting up a separate Emacs  
> process visiting the same file
> and doing the export from there.
> Not all export commands are affected by this - only the ones which
> actually write to a file, and that do not depend on the buffer state.
> -
> +\\<org-mode-map>
> If this option is nil, you can still get background export by calling
> -`org-export' with a double prefix arg: `C-u C-u C-c C-e'.
> +`org-export' with a double prefix arg: \
> +\\[universal-argument] \\[universal-argument] \\[org-export].
>
> If this option is t, the double prefix can be used to exceptionally
> force an export command into the current process."
> @@ -843,7 +847,8 @@ in the background.  This will be done only for  
> commands that write
> to a file.  For details see the docstring of `org-export-run-in- 
> background'.
>
> The prefix argument ARG will be passed to the exporter.  However, if
> -ARG is a double universal prefix `C-u C-u', that means to inverse the
> +ARG is a double universal prefix \\[universal-argument] \\ 
> [universal-argument], \
> +that means to inverse the
> value of `org-export-run-in-background'."
>   (interactive "P")
>   (let* ((bg (org-xor (equal arg '(16)) org-export-run-in-background))
> @@ -977,7 +982,7 @@ value of `org-export-run-in-background'."
> (defvar org-export-id-target-alist nil
>   "Alist of section id's with preferred aliases.")
> (defvar org-export-code-refs nil
> -  "Alist of code references and line numbers")
> +  "Alist of code references and line numbers.")
>
> (defun org-export-preprocess-string (string &rest parameters)
>   "Cleanup STRING so that that the true exported has a more  
> consistent source.
> @@ -1183,7 +1188,7 @@ on this string to produce the exported version."
> (defun org-export-define-heading-targets (target-alist)
>   "Find all headings and define the targets for them.
> The new targets are added to TARGET-ALIST, which is also returned.
> -Also find all ID and CUSTOM_ID propertiess and store them."
> +Also find all ID and CUSTOM_ID properties and store them."
>   (goto-char (point-min))
>   (org-init-section-numbers)
>   (let ((re (concat "^" org-outline-regexp
> @@ -2051,7 +2056,7 @@ TYPE must be a string, any of:
> 	     (delq nil
> 		   (mapcar (lambda (el) (when (member el all) el)) now))))
> 	(when intersection
> -	  (error "recursive #+INCLUDE: %S" intersection))))))
> +	  (error "Recursive #+INCLUDE: %S" intersection))))))
>
> (defun org-get-file-contents (file &optional prefix prefix1 markup)
>   "Get the contents of FILE and return them as a string.
> @@ -2111,7 +2116,7 @@ in the list) and remove property and value  
> from the list in LISTVAR."
> 	    nil t)
>       (if (match-end 1)
> 	  (if (not (match-string 4))
> -	      (error "source block missing language specification: %s"
> +	      (error "Source block missing language specification: %s"
> 		     (let* ((body (match-string 6))
> 			    (nothing (message "body:%s" body))
> 			    (preview (or (and (string-match
> @@ -2406,8 +2411,8 @@ INDENT was the original indentation of the  
> block."
> (defun org-export-visible (type arg)
>   "Create a copy of the visible part of the current buffer, and  
> export it.
> The copy is created in a temporary buffer and removed after use.
> -TYPE is the final key (as a string) that also select the export  
> command in
> -the `C-c C-e' export dispatcher.
> +TYPE is the final key (as a string) that also selects the export  
> command in
> +the \\<org-mode-map>\\[org-export] export dispatcher.
> As a special case, if the you type SPC at the prompt, the temporary
> org-mode file will not be removed but presented to you so that you can
> continue to use it.  The prefix arg ARG is passed through to the  
> exporting
> @@ -2597,8 +2602,8 @@ Does include HTML export options as well as  
> TODO and CATEGORY stuff."
> #+AUTHOR:    %s
> #+EMAIL:     %s
> #+DATE:      %s
> -#+DESCRIPTION:
> -#+KEYWORDS:
> +#+DESCRIPTION:
> +#+KEYWORDS:
> #+LANGUAGE:  %s
> #+OPTIONS:   H:%d num:%s toc:%s \\n:%s @:%s ::%s |:%s ^:%s -:%s f:%s  
> *:%s <:%s
> #+OPTIONS:   TeX:%s LaTeX:%s skip:%s d:%s todo:%s pri:%s tags:%s
> @@ -2607,7 +2612,7 @@ Does include HTML export options as well as  
> TODO and CATEGORY stuff."
> #+EXPORT_EXCLUDE_TAGS: %s
> #+LINK_UP:   %s
> #+LINK_HOME: %s
> -#+XSLT:
> +#+XSLT:
> #+CATEGORY:  %s
> #+SEQ_TODO:  %s
> #+TYP_TODO:  %s
> diff --git a/lisp/org-feed.el b/lisp/org-feed.el
> index 2af97e5..58c552b 100644
> --- a/lisp/org-feed.el
> +++ b/lisp/org-feed.el
> @@ -83,8 +83,8 @@
> ;;
> ;;       #+DRAWERS: PROPERTIES LOGBOOK FEEDSTATUS
> ;;
> -;;  Acknowledgements
> -;;  ----------------
> +;;  Acknowledgments
> +;;  ---------------
> ;;
> ;;  org-feed.el is based on ideas by Brad Bozarth who implemented a
> ;;  similar mechanism using shell and awk scripts.
> diff --git a/lisp/org-freemind.el b/lisp/org-freemind.el
> index 538b404..d066960 100644
> --- a/lisp/org-freemind.el
> +++ b/lisp/org-freemind.el
> @@ -196,7 +196,7 @@ NOT READY YET."
>
> ;;;###autoload
> (defun org-freemind-show (mm-file)
> -  "Show file MM-FILE in Freemind."
> +  "Show file MM-FILE in FreeMind."
>   (interactive
>    (list
>     (save-match-data
> @@ -276,7 +276,7 @@ will also unescape &#nn;."
>     ))
>
> (defun org-freemind-convert-links-from-org (org-str)
> -  "Convert org links in ORG-STR to freemind links and return the  
> result."
> +  "Convert org links in ORG-STR to FreeMind links and return the  
> result."
>   (let ((fm-str (replace-regexp-in-string
>                  (rx (not (any "[\""))
>                      (submatch
> @@ -297,7 +297,7 @@ will also unescape &#nn;."
>
> ;;(org-freemind-convert-links-to-org "<a href=\"http://www.somewhere/ 
> \">link-text</a>")
> (defun org-freemind-convert-links-to-org (fm-str)
> -  "Convert freemind links in FM-STR to org links and return the  
> result."
> +  "Convert FreeMind links in FM-STR to org links and return the  
> result."
>   (let ((org-str (replace-regexp-in-string
>                   (rx "<a"
>                       space
> @@ -344,10 +344,10 @@ will also unescape &#nn;."
>           "</p>\n"))
>
> (defun org-freemind-org-text-to-freemind-subnode/note (node-name  
> start end drawers-regexp)
> -  "Convert text part of org node to freemind subnode or note.
> -Convert the text part of the org node named NODE-NAME. The text
> -is in the current buffer between START and END. Drawers matching
> -DRAWERS-REGEXP are converted to freemind notes."
> +  "Convert text part of org node to FreeMind subnode or note.
> +Convert the text part of the org node named NODE-NAME.  The text
> +is in the current buffer between START and END.  Drawers matching
> +DRAWERS-REGEXP are converted to FreeMind notes."
>   ;; fix-me: doc
>   (let ((text (buffer-substring-no-properties start end))
>         (node-res "")
> diff --git a/lisp/org-habit.el b/lisp/org-habit.el
> index 97c2bf7..f4b990e 100644
> --- a/lisp/org-habit.el
> +++ b/lisp/org-habit.el
> @@ -27,8 +27,11 @@
>
> ;; This file contains the habit tracking code for Org-mode
>
> +;;; Code:
> +
> (require 'org)
> (require 'org-agenda)
> +
> (eval-when-compile
>   (require 'cl))
>
> diff --git a/lisp/org-html.el b/lisp/org-html.el
> index d972c58..3889778 100644
> --- a/lisp/org-html.el
> +++ b/lisp/org-html.el
> @@ -26,7 +26,10 @@
> ;;
> ;;; Commentary:
>
> +;;; Code:
> +
> (require 'org-exp)
> +
> (eval-when-compile (require 'cl))
>
> (declare-function org-id-find-id-file "org-id" (id))
> @@ -57,7 +60,7 @@ by the footnotes themselves."
>   :type 'string)
>
> (defcustom org-export-html-coding-system nil
> -  "Coding system for HTML export, defaults to buffer-file-coding- 
> system."
> +  "Coding system for HTML export, defaults to `buffer-file-coding- 
> system'."
>   :group 'org-export-html
>   :type 'coding-system)
>
> @@ -81,7 +84,7 @@ and corresponding declarations."
> 			(string :tag "Declaration")))))
>
> (defcustom org-export-html-style-include-scripts t
> -  "Non-nil means include the javascript snippets in exported HTML  
> files.
> +  "Non-nil means include the JavaScript snippets in exported HTML  
> files.
> The actual script is defined in `org-export-html-scripts' and should
> not be modified."
>   :group 'org-export-html
> @@ -110,7 +113,7 @@ not be modified."
>  }
> /*]]>*///-->
> </script>"
> -"Basic javascript that is needed by HTML files produced by Org- 
> mode.")
> +"Basic JavaScript that is needed by HTML files produced by Org- 
> mode.")
>
> (defconst org-export-html-style-default
> "<style type=\"text/css\">
> @@ -207,20 +210,20 @@ settings with <style>...</style> tags."
> (put 'org-export-html-style-extra 'safe-local-variable 'stringp)
>
> (defcustom org-export-html-tag-class-prefix ""
> -  "Prefix to clas names for TODO keywords.
> +  "Prefix to class names for TODO keywords.
> Each tag gets a class given by the tag itself, with this prefix.
> The default prefix is empty because it is nice to just use the keyword
> as a class name.  But if you get into conflicts with other, existing
> -CSS classes, then this prefic can be very useful."
> +CSS classes, then this prefix can be very useful."
>   :group 'org-export-html
>   :type 'string)
>
> (defcustom org-export-html-todo-kwd-class-prefix ""
> -  "Prefix to clas names for TODO keywords.
> +  "Prefix to class names for TODO keywords.
> Each TODO keyword gets a class given by the keyword itself, with  
> this prefix.
> The default prefix is empty because it is nice to just use the keyword
> as a class name.  But if you get into conflicts with other, existing
> -CSS classes, then this prefic can be very useful."
> +CSS classes, then this prefix can be very useful."
>   :group 'org-export-html
>   :type 'string)
>
> @@ -235,10 +238,11 @@ CSS classes, then this prefic can be very  
> useful."
>  |
>  <a accesskey=\"H\" href=\"%s\"> HOME </a>
> </div>"
> -  "Snippet used to insert the HOME and UP links.  This is a format,
> -the first %s will receive the UP link, the second the HOME link.
> -If both `org-export-html-link-up' and `org-export-html-link-home' are
> -empty, the entire snippet will be ignored."
> +  "Snippet used to insert the HOME and UP links.
> +This is a format string, the first %s will receive the UP link,
> +the second the HOME link.  If both `org-export-html-link-up' and
> +`org-export-html-link-home' are empty, the entire snippet will be
> +ignored."
>   :group 'org-export-html
>   :type 'string)
>
> @@ -340,7 +344,7 @@ When nil, also column one will use data tags."
>   :type 'boolean)
>
> (defcustom org-export-html-validation-link nil
> -  "Non-nil means add validationlink to postamble of HTML exported  
> files."
> +  "Non-nil means add validation link to postamble of HTML exported  
> files."
>   :group 'org-export-html
>   :type '(choice
> 	  (const :tag "Nothing" nil)
> @@ -349,9 +353,10 @@ When nil, also column one will use data tags."
>
>
> (defcustom org-export-html-with-timestamp nil
> -  "If non-nil, write `org-export-html-html-helper-timestamp'
> -into the exported HTML text.  Otherwise, the buffer will just be  
> saved
> -to a file."
> +  "If non-nil, write timestamp into the exported HTML text.
> +If non-nil Write `org-export-html-html-helper-timestamp' into the
> +exported HTML text.  Otherwise, the buffer will just be saved to
> +a file."
>   :group 'org-export-html
>   :type 'boolean)
>
> @@ -426,7 +431,7 @@ This may also be a function, building and  
> inserting the postamble.")
> ;;; HTML export
>
> (defun org-export-html-preprocess (parameters)
> -  ;; Convert LaTeX fragments to images
> +  "Convert LaTeX fragments to images."
>   (when (and org-current-export-file
> 	     (plist-get parameters :LaTeX-fragments))
>     (org-format-latex
> @@ -460,7 +465,8 @@ headlines.  The default is 3.  Lower levels will  
> become bulleted lists."
>
> ;;;###autoload
> (defun org-export-as-html-batch ()
> -  "Call `org-export-as-html', may be used in batch processing as
> +  "Call the function `org-export-as-html'.
> +This function can be used in batch processing as:
> emacs   --batch
>         --load=$HOME/lib/emacs/org.el
>         --eval \"(setq org-export-headline-levels 2)\"
> @@ -540,12 +546,12 @@ in a window.  A non-interactive call will only  
> return the buffer."
>    nil
>    "Function to convert link URLs to exportable URLs.
> Takes two arguments, TYPE and PATH.
> -Returns exportable url as (TYPE PATH), or `nil' to signal that it
> +Returns exportable url as (TYPE PATH), or nil to signal that it
> didn't handle this case.
> Intended to be locally bound around a call to `org-export-as-html'." )
>
> (defun org-html-cvt-org-as-html (opt-plist type path)
> -   "Convert and org filename to an equivalent html filename.
> +   "Convert an org filename to an equivalent html filename.
> If TYPE is not file, just return `nil'.
> See variable `org-export-html-link-org-files-as-html'"
>
> @@ -565,10 +571,10 @@ See variable `org-export-html-link-org-files- 
> as-html'"
>
> ;;; org-html-should-inline-p
> (defun org-html-should-inline-p (filename descp)
> -   "Return non-nil if link FILENAME should be inlined, according to
> -current settings.
> -DESCP is the boolean of whether there was a link description.
> -See variables `org-export-html-inline-images' and
> +   "Return non-nil if link FILENAME should be inlined.
> +The decision to inline the FILENAME link is based on the current
> +settings.  DESCP is the boolean of whether there was a link
> +description.  See variables `org-export-html-inline-images' and
> `org-export-html-inline-image-extensions'."
>    (declare (special
> 	     org-export-html-inline-images
> @@ -2018,7 +2024,7 @@ that uses these same face definitions."
>   (goto-char (point-min)))
>
> (defun org-html-protect (s)
> -  ;; convert & to &amp;, < to &lt; and > to &gt;
> +  "convert & to &amp;, < to &lt; and > to &gt;"
>   (let ((start 0))
>     (while (string-match "&" s start)
>       (setq s (replace-match "&amp;" t t s)
> @@ -2033,7 +2039,7 @@ that uses these same face definitions."
>   s)
>
> (defun org-html-expand (string)
> -  "Prepare STRING for HTML export.  Applies all active conversions.
> +  "Prepare STRING for HTML export.  Apply all active conversions.
> If there are links in the string, don't modify these."
>   (let* ((re (concat org-bracket-link-regexp "\\|"
> 		     (org-re "[ \t]+\\(:[[:alnum:]_@:]+:\\)[ \t]*$")))
>
> _______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode

- Carsten

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Patchwork: Patch 138 Accepted
  2010-07-15 18:27 ` Patchwork: Patch 138 Accepted Carsten Dominik
@ 2010-07-15 21:28   ` Friedrich Delgado
  2010-07-16 12:03     ` Carsten Dominik
  0 siblings, 1 reply; 13+ messages in thread
From: Friedrich Delgado @ 2010-07-15 21:28 UTC (permalink / raw)
  To: emacs-orgmode

Hi!

I'd appreciate if the subject line and body of those patchwork mails
could be a little more informative, since I can't really be bothered
to click on those urls in order to figure out what's going on.

Carsten Dominik schrieb:
> Patch 138 (http://patchwork.newartisans.com/patch/138/) is now Accepted.
> 
> This relates to the following submission:
> 
> http://mid.gmane.org/%3Cm21vb4bpg2.fsf%40bo.yax.org.uk%3E
> 
> _______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode
> 
---Zitatende---

-- 
        Friedrich Delgado Friedrichs <friedel@nomaden.org>
                             TauPan on Ircnet and Freenode ;)

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Patchwork: Patch 138 Accepted
       [not found] ` <friedel@nomaden.org>
  2009-11-19 23:17   ` Nick Dokos
@ 2010-07-15 22:39   ` Nick Dokos
  2010-07-16  9:45     ` Friedrich Delgado
  1 sibling, 1 reply; 13+ messages in thread
From: Nick Dokos @ 2010-07-15 22:39 UTC (permalink / raw)
  To: emacs-orgmode; +Cc: nicholas.dokos

Friedrich Delgado <friedel@nomaden.org> wrote:

> Hi!
> 
> I'd appreciate if the subject line and body of those patchwork mails
> could be a little more informative, since I can't really be bothered
> to click on those urls in order to figure out what's going on.
> 

I'm really trying to be nice here and not be an officious bastard, but
wouldn't it be more politic to express yourself a bit differently?  It
seems to me that if I'm trying to convince somebody else to do some
amount of work so that I don't have to do a (minuscule) amount of work
on my side, then pointing out that *I* can't be bothered does not invite
much sympathy, let alone a desire to comply with the request.

Regards,
Nick

> Carsten Dominik schrieb:
> > Patch 138 (http://patchwork.newartisans.com/patch/138/) is now Accepted.
> > 
> > This relates to the following submission:
> > 
> > http://mid.gmane.org/%3Cm21vb4bpg2.fsf%40bo.yax.org.uk%3E
> > 
> > _______________________________________________
> > Emacs-orgmode mailing list
> > Please use `Reply All' to send replies to the list.
> > Emacs-orgmode@gnu.org
> > http://lists.gnu.org/mailman/listinfo/emacs-orgmode
> > 
> ---Zitatende---
> 
> -- 
>         Friedrich Delgado Friedrichs <friedel@nomaden.org>
>                              TauPan on Ircnet and Freenode ;)
> 
> _______________________________________________
> Emacs-orgmode mailing list
> Please use `Reply All' to send replies to the list.
> Emacs-orgmode@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-orgmode
> 

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Patchwork: Patch 138 Accepted
  2010-07-15 22:39   ` Patchwork: Patch 138 Accepted Nick Dokos
@ 2010-07-16  9:45     ` Friedrich Delgado
  0 siblings, 0 replies; 13+ messages in thread
From: Friedrich Delgado @ 2010-07-16  9:45 UTC (permalink / raw)
  To: emacs-orgmode

Hi again.

Nick Dokos schrieb:
> Friedrich Delgado <friedel@nomaden.org> wrote:
> > I'd appreciate if the subject line and body of those patchwork mails
> > could be a little more informative, since I can't really be bothered
> > to click on those urls in order to figure out what's going on.
> I'm really trying to be nice here and not be an officious bastard,
> but

That's always a good start...

> wouldn't it be more politic to express yourself a bit differently?  It
> seems to me that if I'm trying to convince somebody else to do some

My intention wasn't to be condescending, just honest.

> amount of work so that I don't have to do a (minuscule) amount of work
> on my side, then pointing out that *I* can't be bothered does not invite
> much sympathy, let alone a desire to comply with the request.
---Zitatende---

And I agree that my choice of words may not be terribly motivating.

However I figured that there are a lot of readers on this mailing list
whose "minuscule" amount of work in order to make some sense out of
these mails will probably add up to a considerably higher effort than
the amount of work needed to make those mails a bit more informative.

I might be wrong about that, of course.


-- 
        Friedrich Delgado Friedrichs <friedel@nomaden.org>
                             TauPan on Ircnet and Freenode ;)

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Patchwork: Patch 138 Accepted
  2010-07-15 21:28   ` Friedrich Delgado
@ 2010-07-16 12:03     ` Carsten Dominik
  2010-07-16 13:44       ` Friedrich Delgado
  2010-07-16 15:22       ` Samuel Wales
  0 siblings, 2 replies; 13+ messages in thread
From: Carsten Dominik @ 2010-07-16 12:03 UTC (permalink / raw)
  To: Friedrich Delgado; +Cc: emacs-orgmode


On Jul 15, 2010, at 11:28 PM, Friedrich Delgado wrote:

> Hi!
>
> I'd appreciate if the subject line and body of those patchwork mails
> could be a little more informative, since I can't really be bothered
> to click on those urls in order to figure out what's going on.
>
> Carsten Dominik schrieb:
>> Patch 138 (http://patchwork.newartisans.com/patch/138/) is now  
>> Accepted.
>>
>> This relates to the following submission:
>>
>> http://mid.gmane.org/%3Cm21vb4bpg2.fsf%40bo.yax.org.uk%3E


If you are using a threaded email reader, this message will appear in  
the
correct thread.  I (and I guess John who made the patchwork process send
a message and in this way automatically inform the mailing list about
the status of this patch) thought that would be good enough.
Because if the issue is one that interests you, I'd think you
are following the thread?

The script sending the message is in the repo: UTILITIES/pw.  It
is written in Python, which I don't know how to write.   I would
accept a patch which provides a better subject line.

Greetings

- Carsten

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Patchwork: Patch 138 Accepted
  2010-07-16 12:03     ` Carsten Dominik
@ 2010-07-16 13:44       ` Friedrich Delgado
  2010-07-16 15:22       ` Samuel Wales
  1 sibling, 0 replies; 13+ messages in thread
From: Friedrich Delgado @ 2010-07-16 13:44 UTC (permalink / raw)
  To: emacs-orgmode

Carsten Dominik schrieb:
> >I'd appreciate if the subject line and body of those patchwork mails
> >could be a little more informative, since I can't really be bothered
> >to click on those urls in order to figure out what's going on.

> If you are using a threaded email reader, this message will appear
> in the correct thread. I (and I guess John who made the patchwork
> process send a message and in this way automatically inform the
> mailing list about the status of this patch) thought that would be
> good enough. Because if the issue is one that interests you, I'd
> think you are following the thread?

Well, I do read a threaded mailreader, and I noticed that the messages
appeared in threads with patches, but frankly I didn't see the
connection until I read something about what that "patchwork" thingy
actually is, so I didn't see the relevance of those mails at all.

This might actually be enough, indeed.

> The script sending the message is in the repo: UTILITIES/pw.  It
> is written in Python, which I don't know how to write.   I would
> accept a patch which provides a better subject line.
---Zitatende---

Well, a little bit of skimming through the code made me come up with
this 

,----
  diff --git a/UTILITIES/pw b/UTILITIES/pw
  index 5a2e3dd..54ed477 100755
  --- a/UTILITIES/pw
  +++ b/UTILITIES/pw
  @@ -320,9 +320,12 @@ def action_update_patch(rpc, patch_id, state = None, commit = None,
   
   This relates to the following submission:
   
  -http://mid.gmane.org/%s''' % \
  -                (patch['id'], patch['id'], state, urllib.quote(patch['msgid']))
  -            shortdesc = 'Patch %s %s' % (patch['id'], state)
  +http://mid.gmane.org/%s
  +
  +(%s)
  +''' % \
  +                (patch['id'], patch['id'], state, urllib.quote(patch['msgid']), patch['name'])
  +            shortdesc = 'Patch %s %s (%s)' % (patch['id'], state, patch['name'])
   
               msg = MIMEText(longdesc)
`----

(Hopefully line breaks won't be garbled...)

But frankly I have no way to test this, so somebody else should
probably look into this.

-- 
        Friedrich Delgado <friedel@nomaden.org>
                             TauPan on Ircnet and Freenode ;)

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Patchwork: Patch 138 Accepted
  2010-07-16 12:03     ` Carsten Dominik
  2010-07-16 13:44       ` Friedrich Delgado
@ 2010-07-16 15:22       ` Samuel Wales
  2010-07-21 12:50         ` Carsten Dominik
  1 sibling, 1 reply; 13+ messages in thread
From: Samuel Wales @ 2010-07-16 15:22 UTC (permalink / raw)
  To: Carsten Dominik; +Cc: emacs-orgmode

gmail "threads" by subject line only.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Patchwork: Patch 138 Accepted
  2010-07-16 15:22       ` Samuel Wales
@ 2010-07-21 12:50         ` Carsten Dominik
  0 siblings, 0 replies; 13+ messages in thread
From: Carsten Dominik @ 2010-07-21 12:50 UTC (permalink / raw)
  To: Samuel Wales; +Cc: emacs-orgmode


On Jul 16, 2010, at 5:22 PM, Samuel Wales wrote:

> gmail "threads" by subject line only.

I did not know this - and I am interested to see if my new messages  
will be threaded correctly by GMail.


- Carsten

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2010-07-21 12:50 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-07-15 17:54 [PATCH] Correct some docstrings [part 2] Philip Rooke
2010-07-15 18:27 ` Patchwork: Patch 138 Accepted Carsten Dominik
2010-07-15 21:28   ` Friedrich Delgado
2010-07-16 12:03     ` Carsten Dominik
2010-07-16 13:44       ` Friedrich Delgado
2010-07-16 15:22       ` Samuel Wales
2010-07-21 12:50         ` Carsten Dominik
2010-07-15 18:28 ` [PATCH] Correct some docstrings [part 2] Carsten Dominik
  -- strict thread matches above, loose matches on Subject: below --
2009-11-19 22:59 Bug: org-agenda-query-and-cmd results in (wrong-type-argument char-or-string-p nil) when org-agenda-query-string is unset [6.33trans (release_6.33c.27.g1bb0d)] Friedrich Delgado Friedrichs
     [not found] ` <friedel@nomaden.org>
2009-11-19 23:17   ` Nick Dokos
2009-11-20 11:00     ` Friedrich Delgado Friedrichs
2010-07-15 22:39   ` Patchwork: Patch 138 Accepted Nick Dokos
2010-07-16  9:45     ` Friedrich Delgado

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).