Differential D17318 Diff 47112 addons/project/kateprojectcodeanalysistool.h

Changeset View

Standalone View

addons/project/kateprojectcodeanalysistool.h

Show First 20 Lines • Show All 49 Lines • ▼ Show 20 Line(s)		43	public:
50	virtual void setProject(KateProject *project);	50		virtual void setProject(KateProject *project);
51		51
52	/**	52		/**
53	* @return tool descriptive name	53		* @return tool descriptive name
54	*/	54		*/
55	virtual QString name() = 0;	55		virtual QString name() = 0;
56		56
57	/**	57		/**
		58		* @returns a string containing the file extensions this
		59		* tool should be run, separated by '\|',
		60		* e.g. "cpp\|cxx"
		61		* NOTE that this is used directly as part of a regular expression.
		62		* If more flexibility is required this method probably will change
			dhaumannUnsubmitted Not Done Why not return a QStringList? The current version just forces parsing in other places. :) ... Later: ok, now I see it's used in the regular expression. Still, I don't like how an implementation detail in a specific location (here: regex later) imposes design decisions here. And also in the UI, a label would rather be comma separated instead of \| separated. dhaumann: Why not return a QStringList? The current version just forces parsing in other places. :) ...
			gregormiAuthorUnsubmitted Not Done I added a comment about this. gregormi: I added a comment about this.
		63		*/
		64		virtual QString fileExtensions() = 0;
		65
		66		/**
58	* filter relevant files	67		* filter relevant files
59	* @param files set of files in project	68		* @param files set of files in project
60	* @return relevant files that can be analyzed	69		* @return relevant files that can be analyzed
61	*/	70		*/
62	virtual QStringList filter(const QStringList &files) = 0;	71		virtual QStringList filter(const QStringList &files) = 0;
63		72
64	/**	73		/**
65	* @return tool path	74		* @return tool path
Show All 29 Lines