Bug 61804 - source code reformat does not indent the c++ includes on the left properly
Summary: source code reformat does not indent the c++ includes on the left properly
Status: RESOLVED NOT A BUG
Alias: None
Product: kdevelop
Classification: Applications
Component: Astyle (show other bugs)
Version: git master
Platform: Unlisted Binaries Linux
: NOR normal
Target Milestone: ---
Assignee: KDevelop Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-07-29 15:48 UTC by tnagy
Modified: 2004-11-21 04:34 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description tnagy 2003-07-29 15:48:16 UTC
Version:           gideon cvs kdevelop20030727 (using KDE KDE 3.1.1a)
Installed from:    Unspecified Linux
Compiler:          gcc 3.2 
OS:          Linux

The source code reformat does not align the c++ preprocessor declarations on the left properly.

For example the following declarations :
#include <iostream>
.    #include <vector>

will not be aligned on the left like this after a source reformat :
#include <iostream>
#include <vector>

The user must do it by hand.

(I have seen this with the all the indentation modes : ansi, linux, gnu, java ...)
Comment 1 Jens Dagerbo 2003-08-25 17:54:11 UTC
Is this a bug? From what I can tell 
 
#include <a> 
  #include <b> 
 
is legal, and there may well be a reason the programmer did it like that. 
 
Unconfirming bug and hoping for enlightened input. :) 
Comment 2 tnagy 2003-08-25 17:59:05 UTC
Of course it is legal. It is just ugly.
Comment 3 Jens Dagerbo 2003-08-25 18:15:54 UTC
Hehe. I didn't know the bugsystem stripped leading spaces.. my example wasn't 
very telling.. :) 
 
Well.. if everyone is in agreement about this being legal, and it is obvious that at 
times indented preprocessor directives have their use (nested directives), then 
this is not a bug and shouldn't be reported as such. 
 
I'm closing it. File a wishlist bug if you want, but I doubt the behaviour will be 
changed.