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

[Logs] Cleaning code from debug logs #108

Merged

Conversation

pkosko
Copy link

@pkosko pkosko commented Jun 9, 2021

This commit:

  • adds logs level control with 'flutter-tizen run --verbose-system-logs'
  • removes redundant debug logs
  • changes some valuable debug logs into LOGI level
  • changes some logs to LOGW level

This commit does the work needed for #101

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

This commit is a part of flutter-tizen#101
@pkosko pkosko requested a review from a team June 9, 2021 08:26
Copy link

@pwasowski2 pwasowski2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👌

Copy link
Member

@bwikbs bwikbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good! 😄

@swift-kim
Copy link
Member

Don't we need to implement the verbose_logging switch first? Otherwise the INFO logs will be printed out to the console by default.

@pkosko
Copy link
Author

pkosko commented Jun 14, 2021

@swift-kim at first I didn't understand that the implementation is missing. Please check if my latest commit meets your expectations.

@swift-kim
Copy link
Member

@pkosko Yes, I was expecting that kind of thing as a prerequisite. Thank you for understanding. You may just rename prio_min_ to min_priority to follow the general naming rules (no trailing underscore, minimum abbreviation).

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

Related to flutter-tizen#101
shell/platform/tizen/tizen_log.cc Outdated Show resolved Hide resolved
shell/platform/tizen/tizen_log.cc Outdated Show resolved Hide resolved
Copy link

@pwasowski2 pwasowski2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👌

Copy link
Member

@swift-kim swift-kim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise LGTM. Thanks!

shell/platform/tizen/tizen_log.h Outdated Show resolved Hide resolved
shell/platform/tizen/tizen_log.cc Outdated Show resolved Hide resolved
* remove some redundant logs
* change some meaningful logs to LOGI or LOGW

This commit is a part of flutter-tizen#101
@pkosko
Copy link
Author

pkosko commented Jun 15, 2021

Dear reviewers, I've added last commit d3d5ab9 which removes all remaining LOGD usages. This commit makes that whole scope of issue #101 is covered by this PR.

@swift-kim swift-kim merged commit fdd03a6 into flutter-tizen:flutter-2.0.1-tizen Jun 16, 2021
swift-kim pushed a commit that referenced this pull request Jun 16, 2021
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

This commit is a part of #101

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

Related to #101

* Fixes after review

* Review fixes 2

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW

This commit is a part of #101

* Review fixes
swift-kim pushed a commit that referenced this pull request Sep 27, 2021
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

This commit is a part of #101

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

Related to #101

* Fixes after review

* Review fixes 2

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW

This commit is a part of #101

* Review fixes
swift-kim pushed a commit that referenced this pull request Nov 14, 2021
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

This commit is a part of #101

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

Related to #101

* Fixes after review

* Review fixes 2

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW

This commit is a part of #101

* Review fixes
swift-kim pushed a commit that referenced this pull request Dec 9, 2021
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

This commit is a part of #101

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

Related to #101

* Fixes after review

* Review fixes 2

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW

This commit is a part of #101

* Review fixes
swift-kim pushed a commit that referenced this pull request Dec 17, 2021
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

This commit is a part of #101

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

Related to #101

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW

This commit is a part of #101

* Review fixes
swift-kim pushed a commit that referenced this pull request Feb 7, 2022
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

This commit is a part of #101

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

Related to #101

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW

This commit is a part of #101

* Review fixes
swift-kim pushed a commit that referenced this pull request Feb 11, 2022
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

This commit is a part of #101

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

Related to #101

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW

This commit is a part of #101

* Review fixes
swift-kim pushed a commit that referenced this pull request May 12, 2022
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

This commit is a part of #101

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

Related to #101

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW

This commit is a part of #101

* Review fixes
swift-kim pushed a commit that referenced this pull request Aug 5, 2022
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW
swift-kim pushed a commit that referenced this pull request Sep 1, 2022
* [Logs] Cleaning code from debug logs

This commit:
* removes redundant debug logs
* changes some valuable debug logs into LOGI level
* changes some logs to LOGW level

* [Logging] Added verbose-system-logs handling

All logs are filtered via flag usage:

* flutter-tizen run --verbose-system-logs
  all logs are visible in flutter console, including INFO
* flutter-tizen run
  only ERROR logs are visible in console

* [Logs] Clean remaining LOGD from code

* remove some redundant logs
* change some meaningful logs to LOGI or LOGW
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