Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

listnum と emlistnum で開始行番号を指定できるようにしたい #685

Closed
anchan828 opened this issue Dec 3, 2016 · 7 comments
Milestone

Comments

@anchan828
Copy link

anchan828 commented Dec 3, 2016

技術書などでサンプルコードを掲載する際に、一部を抜き出して掲載することがあります。

その際に listnum か emlistnum で実際のサンプルファイルの行番号を指定したいです。

↓は FbxImporter.cs ファイルの 9行目から19行目を抜き出したものです。現在はキャプションに行番号を記載しています。

//emlistnum[Assets/Editor/FbxImporter.cs line 9-19]{
    // 1.Project内からPrefabのguidを取得
    var guids = AssetDatabase.FindAssets("t:Prefab");
    // ProjectにPrefabがなければ処理を中断
    if (guids.Length == 0) return;

    // 2.FbxReplacementComponentをもつPrefabを取得
    var replacementComponents = guids.Select(guid => AssetDatabase.GUIDToAssetPath(guid))
      .Select(path => AssetDatabase.LoadAssetAtPath<FbxReplacementComponent>(path))
      .Where(component => component != null && PrefabUtility.GetPrefabType(component) == PrefabType.Prefab);

    if (replacementComponents.Count() == 0) return;
//}

ss 2016-12-03 21 49 44

@kmuto
Copy link
Owner

kmuto commented Dec 4, 2016

開始行番号を変えるのはできそうですが、(中略)とか入れたくなりそうなときにどうしようという悩みがあります。
手元の類似のケースでは、プリプロセッサ側にやや複雑めな処理をカスタムで追加していました。

@takahashim
Copy link
Collaborator

(中略)とか入れたくなりそうなときにどうしよう

それは現状でもどうにもならないので、開始行番号指定はあってもよいかもしれません(あまり積極的ではないですが)。
問題は実現方法をどうするかですね…

  • 引数増やす
  • 何かしらのディレクティブ的なものを追加する
  • その他

@kmuto
Copy link
Owner

kmuto commented Dec 5, 2016

引数はオプション記法ができるまでもう増やしたくないので、直後のものに影響する//tsize的なディレクティブがよいかなと思います。

@kdmsnr
Copy link
Collaborator

kdmsnr commented Dec 5, 2016

//listlinenum[9] ですかね。もしかすると終了番号もいけるかも //listlinenum[9,19]

@takahashim
Copy link
Collaborator

#688 で対応してみました。いろいろあって先頭にlistという名前が入ると不幸になりそうだったので、//firstlinenum[...]という名前になっています。終了番号は対応が厳しそうだったので入れてません。

HTML/LaTeX/InDesign(含むhighlight)に対応してみたつもりですが、InDesign対応はこれであってます?>@kmuto

@kmuto
Copy link
Owner

kmuto commented Dec 24, 2016

よさそうに見えます。

@kmuto
Copy link
Owner

kmuto commented Dec 30, 2016

#688 にて対応済みですね。

@kmuto kmuto closed this as completed Dec 30, 2016
@takahashim takahashim added this to the 2.2.0 milestone Dec 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants