aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/utils/SkRTConf.cpp
Commit message (Collapse)AuthorAge
* SkRTConf: eliminateGravatar halcanary2016-08-04
| | | | | | | | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2212473002 DOCS_PREVIEW= https://skia.org/?cl=2212473002 CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot [mtklein] TBR=reed@google.com Only removing unused public API. Review-Url: https://codereview.chromium.org/2212473002
* Revert of SkRTConf: reduce functionality to what we use, increase simplicity ↵Gravatar mtklein2016-08-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | | (patchset #8 id:150001 of https://codereview.chromium.org/2212473002/ ) Reason for revert: missed GrVkPipelineStateCache Original issue's description: > SkRTConf: reduce functionality to what we use, increase simplicity > > GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2212473002 > DOCS_PREVIEW= https://skia.org/?cl=2212473002 > CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot > > [mtklein] > TBR=reed@google.com > Only removing unused public API. > > Committed: https://skia.googlesource.com/skia/+/ef59974708dade6fa72fb0218d4f8a9590175c47 TBR=halcanary@google.com # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true Review-Url: https://codereview.chromium.org/2215433003
* SkRTConf: reduce functionality to what we use, increase simplicityGravatar halcanary2016-08-03
| | | | | | | | | | | | GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2212473002 DOCS_PREVIEW= https://skia.org/?cl=2212473002 CQ_INCLUDE_TRYBOTS=master.client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot [mtklein] TBR=reed@google.com Only removing unused public API. Review-Url: https://codereview.chromium.org/2212473002
* Eliminate SkFILE: it always is the same as FILE.Gravatar halcanary2015-11-20
| | | | Review URL: https://codereview.chromium.org/1467533003
* Style Change: NULL->nullptrGravatar halcanary2015-08-27
| | | | | | DOCS_PREVIEW= https://skia.org/?cl=1316233002 Review URL: https://codereview.chromium.org/1316233002
* Style Change: SkNEW->new; SkDELETE->deleteGravatar halcanary2015-08-26
| | | | | | DOCS_PREVIEW= https://skia.org/?cl=1316123003 Review URL: https://codereview.chromium.org/1316123003
* Remove include of stdlib.h from SkTypes.h.Gravatar bungeman2015-08-26
| | | | | | | | | | | | | | | Unfortunately, immintrin.h (which is also included by SkTypes) includes xmmintrin.h which includes mm_malloc.h which includes stdlib.h for malloc even though, from the implementation, it is difficult to see why. Fortunately, arm_neon.h does not seem to be involved in such shenanigans, so building for Android will keep things sane. TBR=reed@google.com Doesn't change Skia API, just moves an include. Review URL: https://codereview.chromium.org/1313203003
* "NULL !=" = NULLGravatar bsalomon2014-09-05
| | | | | | | | R=reed@google.com Author: bsalomon@google.com Review URL: https://codereview.chromium.org/544233002
* Migrate SkRTConfRegistry test to DEF_TEST().Gravatar tfarina2014-08-12
| | | | | | | | | | BUG=None TEST=dm --tests=true --gms=false R=mtklein@google.com, bsalomon@google.com Author: tfarina@chromium.org Review URL: https://codereview.chromium.org/444303002
* fix leaks in SkConfGravatar commit-bot@chromium.org2014-01-31
| | | | | | | | | | | | | | | | | These leaks were small and would only happen with non-default runtime configuration things set, but they were still leaks and they should be squished. Valgrind reported leaks pre-patch, and all the rtconf leaks are gone after patch. BUG=skia:1722 R=reed@google.com Author: humper@google.com Review URL: https://codereview.chromium.org/136963004 git-svn-id: http://skia.googlecode.com/svn/trunk@13269 2bbb7eff-a529-9590-31e7-b0007b416f81
* Reduce verbosity of SkGraphics::Init in default case.Gravatar halcanary@google.com2014-01-10
| | | | | | | | | | | | | | | | | SkGraphics::Init() now checks to see if there are any non-default runtime configuration options before announcing that it is about to print out the non-default runtime configuration options. This makes the executables in tools/ less verbose. Add SkRTConfRegistry::countNonDefault() function. BUG= R=mtklein@google.com, reed@google.com Review URL: https://codereview.chromium.org/133583003 git-svn-id: http://skia.googlecode.com/svn/trunk@13017 2bbb7eff-a529-9590-31e7-b0007b416f81
* New SkRTConf macro SK_CONF_TRY_SET: no complaint on missing configurationGravatar halcanary@google.com2013-11-06
| | | | | | | | | | | | | | | | | | | | | | | | | | SK_CONF_TRY_SET() is like SK_CONF_SET(), but doesn't complain if confname can't be found. This is useful if the SK_CONF_DECLARE is inside a source file whose linkage is dependent on the system. Internally to the SkRTConf system, SkRTConfRegistry::set() was given an additional parameter controling wanrings. A new RuntimeConfig unit test was introduced. It should run silently. In the future, it should be expanded to cover all of the SkRTConf functionality. (For example, the images.jpeg.suppressDecoderWarnings variable is defined and used only in SkImageDecoder_libjpeg.cpp, but on MacOS, we use Core Graphics via SkImageDecoder_CG.cpp - SkImageDecoder_libjpeg is never linked in. The same is true of the Windows Imaging Component on Windows.) BUG= R=reed@google.com Review URL: https://codereview.chromium.org/54503007 git-svn-id: http://skia.googlecode.com/svn/trunk@12155 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix a bug which caused crash in SkRTConfig whith string values in ↵Gravatar halcanary@google.com2013-11-06
| | | | | | | | | | | | | environment variables A minor change to SkRTConfRegistry::parse() fixes this BUG=skia:1717 R=scroggo@google.com Review URL: https://codereview.chromium.org/26373008 git-svn-id: http://skia.googlecode.com/svn/trunk@12152 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix crash when querying a runtime config that is defined in environmentGravatar commit-bot@chromium.org2013-08-14
| | | | | | | | | | | | | | | | | | | | Fix crash when querying a runtime config in case when there's no skia.conf file and the config variable is still defined in the environment. Runs the added SkRTConf::UnitTest test as part of new "UnitTest" test. Previous version of the patch failed Windows build due to setenv usage. On Windows, use _putenv_s instead. BUG=skia:1494 R=bsalomon@google.com, humper@google.com Author: kkinnunen@nvidia.com Review URL: https://chromiumcodereview.appspot.com/23174002 git-svn-id: http://skia.googlecode.com/svn/trunk@10715 2bbb7eff-a529-9590-31e7-b0007b416f81
* Reverted r10654 (Fix crash when querying a runtime config that is defined in ↵Gravatar robertphillips@google.com2013-08-11
| | | | | | environment) due to compilation failure on Chromium win_layout bot git-svn-id: http://skia.googlecode.com/svn/trunk@10669 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fix crash when querying a runtime config that is defined in environmentGravatar commit-bot@chromium.org2013-08-09
| | | | | | | | | | | | | | | | | Fix crash when querying a runtime config in case when there's no skia.conf file and the config variable is still defined in the environment. Runs the added SkRTConf::UnitTest test as part of new "UnitTest" test. BUG=skia:1494 R=bsalomon@google.com, humper@google.com Author: kkinnunen@nvidia.com Review URL: https://chromiumcodereview.appspot.com/22650007 git-svn-id: http://skia.googlecode.com/svn/trunk@10654 2bbb7eff-a529-9590-31e7-b0007b416f81
* search registered rtconf variables backwards and break when we find theGravatar humper@google.com2013-07-24
| | | | | | | | | | | first match. Patch from mozilla BUG= R=mtklein@google.com Review URL: https://codereview.chromium.org/20136003 git-svn-id: http://skia.googlecode.com/svn/trunk@10337 2bbb7eff-a529-9590-31e7-b0007b416f81
* fix boolean check for unknown rtconf variables in files.Gravatar humper@google.com2013-05-20
| | | | | | | | BUG= Review URL: https://codereview.chromium.org/15300019 git-svn-id: http://skia.googlecode.com/svn/trunk@9198 2bbb7eff-a529-9590-31e7-b0007b416f81
* Fixed doParse functionsGravatar sugoi@google.com2013-03-04
| | | | | | | For some reason, "success" wasn't set in a bunch of template specializations, causing it to be invalid when used later on. Review URL: https://codereview.appspot.com/7370052 git-svn-id: http://skia.googlecode.com/svn/trunk@7969 2bbb7eff-a529-9590-31e7-b0007b416f81
* Sanitizing source files in Skia_Periodic_House_KeepingGravatar skia.committer@gmail.com2013-01-15
| | | | git-svn-id: http://skia.googlecode.com/svn/trunk@7173 2bbb7eff-a529-9590-31e7-b0007b416f81
* move template specializations to inline functions in the header to make ↵Gravatar humper@google.com2013-01-14
| | | | | | | | | | windows happy again BUG= Review URL: https://codereview.appspot.com/7099053 git-svn-id: http://skia.googlecode.com/svn/trunk@7164 2bbb7eff-a529-9590-31e7-b0007b416f81
* replace getline with fgets for portabilityGravatar humper@google.com2013-01-14
| | | | | | | | BUG= Review URL: https://codereview.appspot.com/7085058 git-svn-id: http://skia.googlecode.com/svn/trunk@7163 2bbb7eff-a529-9590-31e7-b0007b416f81
* remove dynamic_cast because rtti is disabled on some platforms (but not all??)Gravatar humper@google.com2013-01-14
| | | | | | | | BUG= Review URL: https://codereview.appspot.com/7102056 git-svn-id: http://skia.googlecode.com/svn/trunk@7161 2bbb7eff-a529-9590-31e7-b0007b416f81
* Runtime configuration system for skia. This will allow developers to ↵Gravatar humper@google.com2013-01-14
control settings at launch time without relying on compile-time flags or recompilation. It can be used to turn features on and off, as well as to control numeric quantities to 'tune' algorithms. Once I make sure it's working across all platforms I'll send out a quick tutorial on its use. Review URL: https://codereview.appspot.com/7098051 git-svn-id: http://skia.googlecode.com/svn/trunk@7158 2bbb7eff-a529-9590-31e7-b0007b416f81
generated by cgit v1.2.3 (git 2.39.1) at 2024-06-30 21:24:00 +0000