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

Cdt 9 5 #10

Closed
wants to merge 22 commits into from
Closed

Cdt 9 5 #10

wants to merge 22 commits into from

Conversation

dpc2035
Copy link

@dpc2035 dpc2035 commented Aug 21, 2018

when I format code , eclipse delete my empty lines,
that makes me feel not happy, I wish I can configure that: empty lines should be deleted or not...

jonahgraham and others added 19 commits June 5, 2018 21:08
Change-Id: I7df8191b469bfc0f0d241229563c26a1d85fda4b
Change-Id: Ifeda1a0256cbfa9c87acd9a9bc8bb755b533fed3
AttributeWriter.writeGCCAttributeSpecifier()

* Added test to reproduce initial issue.

Change-Id: I72e60fb2244b7d40492bfcdeaa58a8707c39e277
Signed-off-by: Thomas Corbat <tcorbat@hsr.ch>
(cherry picked from commit 4e4e010)
We weren't dealting with relative include paths returned from
the call to gcc -E properly. Resolves them against the build
directory before checking if their a directory. Results in them
added.

Tested with ESP-IDF's feature/cmake branch.

Change-Id: I1e2583e03cce6380ee4382146b177978c9c81806
This reverts commit e5c7bb6.

Change-Id: I565f9eb2728d0ff0199e61f3f25df4aa0b64f17b
Change-Id: I059636ef27e3bec7e49cc7d6f37f70d8d9c245ec
Change-Id: Icb762d8a4ac56153f7d7637a258fedfcd5b774f0
Change-Id: I3a46159e5cd6f298f85fee148517f90521dfe602
- fix ContainerTargetTypeProvider changeEvent() to start a Job and
  return immediately so it will not cause a DockerConnection
  to be held in multi-threading
- move the DockerConnectionManager addConnectionListener call
  to end of init() method so the fetching of images won't cause
  a notification event to occur
- at end of init(), call CBuildConfigurationManager.recheckConfigs()
  to make sure any disabled configuration due to a missing
  IDockerConnection is now put in the configs master list and
  removed from the noconfigs list
- make similar changes to ContainerGCCToolChainProvider

Change-Id: Idc120d613b99ec365522f5e7bf5da82d1b362425
(cherry picked from commit 2bcd06f)
We were not handling JSON parse errors when loading the scanner info
cache files which causes havoc at index time.

Change-Id: I3a4ee6424134fe756543e4429e9e97b911cda99e
- fix GCCToolChain and ContainerGCCToolChain to not blindly
  take the first token in the command string when processing
  scannerinfo; if it is "ccache", take the second token instead

Change-Id: I4b2b7dfaccae6f3ec968bbe4217c57994ad71963
(cherry picked from commit a5ed8ea)
Change-Id: I0fb0c7ecc2e59cda3e0bb7efb98fd2a72c5aca80
- it seems that fetching the active configuration for a project
  using an ICConfigurationDescription is time-consuming
- the active configuration is sought in ContainerCommandLauncherFactory
  to get the optional build properties and thus we can figure out
  if the active configuration has enabled container build or not
- see if we are given a CConfigurationDescriptionCache and it
  contains a BuildConfigurationData element which allows us to
  get an IConfiguration without creating a new one, serializing
  it etc.. which is adding to the time taken

Change-Id: I8973f1707a602a73fc5ac751f12a1e8dbd549aab
(cherry picked from commit a394557)C
- add a new IToolChain property "cdt.needScannerRefresh" that is
  set if scanner info needs to be refreshed
- for a Container build, turn the property on in ContainerGCCToolChain
  startBuildProcess() if the Container headers for the toolchain's
  have been deleted
- in CBuildConfiguration processLine(), look for the toolchain
  property when looking to see if scannerinfo should be calculated
- in CBuildConfiguration, turn the toolchain property off in
  shutdown()

Change-Id: I67a3537d1e2967dc15b66a1c37abda1ae8f78bff
- apply patch submitted by duh-sa-sekirom@hotmail.com
- in GCCToolChain join the threads that do include path and macro
  searches before waiting for process so as to prevent a
  ConcurrentModificationException

Change-Id: Iefe6f013007b7bbf00117b5295b5e27489a72834
(cherry picked from commit e0c656d)
Change-Id: Ie8180cd33d2a9831e8be10519ef40efd9110b56b
- The commands are launched directly instead of using the shell
(/bin/sh) which is not available on Windows.

Change-Id: I5f8d99a04da1c860bcd37d138103a1bd4e501547
Signed-off-by: François Godin <copelnug@gmail.com>
(cherry picked from commit 0b2053d)
- fix MesonPropertyPage logic concerning a configured meson project
- for possible values, add "-" and "/" as valid characters
- in state flow, don't just wait for blank lines, an option
  header can occur in the middle of a group or a new option group
  name can also occur

Change-Id: I2c6c8f9fb15db0425f03e422b20b58ace5234fd4
(cherry picked from commit ac81db7)
Turns off doclint for Java 1.8 and greater because the javadocs are
incomplete and won't build otherwise.

Some files also had encoding issues with some characters which caused
the javadoc build to fail, these characters have been replaced.

Change-Id: I04690644bec6d7c3219cb0438cb3db6593fefd3a
Signed-off-by: Hansruedi Patzen <hansruedi.patzen@hsr.ch>
@jonahgraham
Copy link
Member

I assume you wanted to submit an Issue, but couldn't find it in the repo here? GitHub does allow us to turn off issues, but not PRs. This repo is simply a mirror of CDT. Please file bugs at https://bugs.eclipse.org/bugs/enter_bug.cgi?product=CDT

@jonahgraham
Copy link
Member

@eclipsewebmaster Please close this PR (I don't have permission to).

Change-Id: I9fdc268819b2298d0b3aeb9db9c29f1a0158d2bb
…fers to a parameter from the index

Change-Id: Ibff680983cc8ac0bd9d33d8f2247a9f0ca2fe2f3
Change-Id: I42e780f90aacf9383b722a8b82e441d4032d14e7
@jonahgraham
Copy link
Member

@eclipsewebmaster Please close this PR - I don't have permission to.

1 similar comment
@jonahgraham
Copy link
Member

@eclipsewebmaster Please close this PR - I don't have permission to.

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

Successfully merging this pull request may close these issues.

6 participants