GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/21
[Module asc] Cleanup code 1st step
This PR is the first step on the path to good condition code of compiler.
I hope that you will take it and then I can make the new PR.
You can merge this
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/21
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/22
[Module asc] Cleanup code - 1st step
This PR is the first step on the path to good condition code of compiler.
I hope that you will take it and then I can make the new PR.
You can merge this
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/22
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/24
Replace with chained append() calls
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/23
Remove unnecessary intern() on String constant
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/25
Remove unnecessary boxing
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature/remove_unnecessary_boxing
Alternatively
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/27
Remove unnecessary semicolon
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature/remove_unnecessary_semicolon
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/26
Remove unnecessary unboxing
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature/remove_unnecessary_unboxing
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/28
Replace 'String.indexOf(value) > -1' by 'String.contains(value)'
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Sl
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/29
Remove unnecessary enum modifier
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature/remove_unnecessary_enum_modifier
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/30
Remove unnecessary interface modifier
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/31
Remove redundant castings
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature/remove_redundant_casting
Alternatively
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/32
Simplify boolean constructor call
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature/boolean_constructor_call
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/33
Manual array copy replace with System.arraycopy
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk feature/manual_array_copy
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/34
Manual array to collection copy replace with Collections.addAll
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature
Github user SlavaRa commented on the pull request:
https://github.com/apache/flex-sdk/pull/33#issuecomment-149929667
Done
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
GitHub user SlavaRa opened a pull request:
https://github.com/apache/flex-sdk/pull/35
Remove unnecessary String.toString()
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SlavaRa/flex-sdk
feature
From 086a61665cf5814a2561db8ac62ad92f1056bdef Mon Sep 17 00:00:00 2001
From: SlavaRa
Date: Wed, 21 Oct 2015 16:08:47 +0300
Subject: [PATCH] Replace with chained append() calls
---
.../asc/src/java/adobe/abc/GlobalOptimizer.java| 6 +--
.../macromedia/asc/embedding/LintEvaluator.java
Github user SlavaRa commented on the pull request:
https://github.com/apache/flex-sdk/pull/24#issuecomment-150963665
https://issues.apache.org/jira/browse/FLEX-34946
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user SlavaRa commented on the pull request:
https://github.com/apache/flex-sdk/pull/24#issuecomment-152467858
I would like to help in the development of the compiler.
But there is no desire to work with code that looks bad to say the least.
Using string concatenation
Github user SlavaRa commented on the pull request:
https://github.com/apache/flex-sdk/pull/24#issuecomment-152506154
"I think "looks bad" is a personal preference."
Perhaps, but in this situation, combined the two approaches
StringBuffer.append and string conc
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/35
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/33
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/31
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/30
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/34
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/32
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/27
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/29
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/25
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/26
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/28
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/24
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user SlavaRa closed the pull request at:
https://github.com/apache/flex-sdk/pull/23
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
35 matches
Mail list logo
