Format On Save with Xcode behaviors.

Open
#57 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
25/100
Issue type
Feature
Clarity
Needs clarification
Activity status
Stale
Tech stack
cpp, shell
Domain
tooling

Research direction

Start by reproducing the Xcode behavior from the issue's Command+S script, including the clang-format -i and touch commands on a .cpp file. Investigate whether the plugin can invoke formatting within Xcode rather than modifying the file externally. Done would mean formatting on save without showing the external-modification dialog and while retaining the newer disk version.

Written by the indexing model from the issue text.

Description

Hi, in order to allow "Format on Save" I avoid using extensions, but rather set new Xcode behavior hat run the following script upon "Command+S" (which replace the original file save):

My script extract the current file name presented by xcode, and perform clang-format on this file (see below), and it works just fine.

However, when the file is modified, I sometimes get the following message from xcode.

the file for the document at “/Users/me/file.cpp” has been modified by another application. There are also unsaved changes in Xcode. Do you want to keep the Xcode version or revert to the version on disk?

This happens because the file changed outside xcode ... and xcode probably cache the currently file being edited, so I'd like to avoid this message box and always take the newer file version from disk (select the "Revert" option instead of the other option of "Keep Xcode Version") . is it possible to avoid this message box ?

Thanks

#!/bin/sh

CDP=$(osascript -e 'tell application "Xcode"
    activate
    set current_document to last source document
    set current_document_path to path of current_document
end tell ' )

if [[ $CDP == *.cpp ]]
then
    /usr/local/bin/clang-format -i ${CDP}
    touch ${CDP}
fi

P.S I'm aware this option doesn't related directly to the plugin, but if the plugin can somehow run clang-format from within the xcode, it may solve my problem and avoid this messagebox...

Dominant language
Objective-C++
Stars
807
Forks
102
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from mapbox/XcodeClangFormat

All issues in mapbox/XcodeClangFormat

Similar issues

More DevTools issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.