#76 cscope-indexer: index assembly (.S) files

closed-rejected
nobody
None
5
2012-05-20
2011-08-19
No

This is a tiny patch to have cscope-indexer index assembly (.S) files as well C/C++ files.

Discussion

  • - 2011-08-19
     
  • Hans-Bernhard Broeker

    • status: open --> pending-remind
     
  • Hans-Bernhard Broeker

    I rather much doubt this is a good idea. cscope has no idea what to do with assembly source files, so there's really not much of a point in trying to add them to do the index. People who really think they need to abuse the tool can always make this kind of change locally.

     
  • - 2011-08-19
    • status: pending-remind --> open-remind
     
  • - 2011-08-19

    When working on kernel/embedded code, I've found it useful to have cscope take me to the definitions of assembly functions. The cscope website actually encourages users to index assembly files for the Linux kernel: http://cscope.sourceforge.net/large_projects.html. And, in fact, the Linux kernel's own script/tags.sh (used by 'make cscope') indexes .S files.

    That said, I'm happy keeping this a local change.

     
  • Hans-Bernhard Broeker

    > useful to have cscope take me to the definitions of assembly functions

    That might be a usefule idea --- if cscope had any chance of actually doing that. Which it doesn't.

    Cscope has no way of distinguishing a function definition from any other piece of text in assembly source. It doesn't recognize comments, it doesn't recognize _anything_ in there. You'll just be poisoning your index with hundreds of thousands of what cscope believes are globally defined things called "movl", "sub", ".section" and whatever else. That makes no sense at all.

     
  • Hans-Bernhard Broeker

    • status: open-remind --> closed-rejected
     
  • Hans-Bernhard Broeker

    • assigned_to: darrylo --> nobody
     
  • Hans-Bernhard Broeker

    • labels: 388882 -->
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks