[Github-comments] [geany/geany] Set indent type when using "replace tabs with spaces" (plus the opposite direction) (#1161)

[Jiří Techet]

When I was making the sources of our ctags use tabs-only indentation, I used Geany's "replace tabs with spaces" and "replace spaces with tabs". However, after using it I realized that every time I also had to set the document's indentation type manually which was a bit annoying and unnecessary. IMO when doing replacements of tabs<->spaces in the whole document, we should set also the document's indentation type to the new type. The first patch does this.

The second patch is just kind of cleanup which makes the implementation of undo() and redo() more similar to each other.
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany/pull/1161

-- Commit Summary --

  * Set document indent type when replacing tabs<->spaces in the whole document
  * Make UNDO_EOL/UNDO_INDENT always precede the attached Scintilla action

-- File Changes --

    M src/document.c (64)
    M src/documentprivate.h (1)
    M src/editor.c (36)

-- Patch Links --

https://github.com/geany/geany/pull/1161.patch
https://github.com/geany/geany/pull/1161.diff

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1161
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160803/805a3fbd/attachment.html>