@mridgway could you help to release one patch version for the update?


This looks good to me. I think most (if not all) of these RUNTIME errors should be CODING errors, but that could be a distinct follow up change.


Hola, gracias por la colaboración!...agregue el código, pero ahora me da un error 500. Alguna sugerencia?
Saludos.


Adrian, there currently isn't any extra support for dependencies between
ajax-loaded inputs. When an input is rendered that way, it has no knowledge
of what other inputs are on the page since it is rendered in a separate
request. Though some dependency support may be possible (and will be
working on it more), but likely not between ajax fields loaded in different
requests. When it comes to dependencies, it's best to not use ajax-load for
those. There are also likely to be some inputfields that won't work with
ajax load, especially 3rd party ones... for instance, none of the ProFields
support it yet ... though they will next week. :) All of the core
inputfields should support it now though, that's where most of the time
went this week.

On Fri, Jul 31, 2015 at 5:52 PM, Adrian Jones <notifications@github.com>
wrote:

> Ryan - I am playing around with the new Ajax loading options - really
> awesome addition, but a couple of questions.
>
>    1. I seem to be having an issue with showif dependencies inside Tab
>    fieldsets - they are ignored and all fields are displayed once the ajax has
>    loaded the content. I think this is a bug.
>    2. Any thoughts about extending ajax loading to showIf reveals?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ryancramerdesign/ProcessWire/commit/0d1d91ec7dcb6d859f410d5165495430136c8079#commitcomment-12475778>
> .
>


Unfortunately there is another case, where same fix needed. It's in `Source/cmInstallGenerator.cxx`:

    std::string
    cmInstallGenerator::ConvertToAbsoluteDestination(std::string const& dest) const
    {
    std::string result;
    if(!dest.empty() &&
        !cmSystemTools::FileIsFullPath(dest.c_str()))
        {
        result = "${CMAKE_INSTALL_PREFIX}/";
        }
    result += dest;
    return result;
    }

Here CMake violates mentioned guides… Unfortunately, I've failed to find a proper way (suitable for CMake coding style) to fix that, so can't offer a PR now :(

I had thoughts about to add a (pure?) virtual method `GetInstallPrefix()`, so `ConvertToAbsoluteDestination` can use it to handle special cases. Derived classes (various install handlers) should implement it substituting `this->Makefile->GetSafeDefinition("CMAKE_INSTALL_PREFIX")` or `this->Target->Target->GetMakefile()->GetSafeDefinition("CMAKE_INSTALL_PREFIX")` in case of `cmInstallTargetGenerator` which doesn't have a `cmMakefile*` member.

I've been failed to pass install prefix to `cmInstallGenerator` via constructor when found that `cmInstallTargetGenerator` do not have `cmMakefile*` at construction time, so can't get definition and pass it to base class :( Here I've gave up…

I'm still ready to implement this fix, but need some guidance…


When you stopped assigning the `bValid` member variable in here, I became concerned that an assignment might be missed and let it go stale, so I did a quick audit, and it looks like the only place where `bValid` could not be replaced by a function local variable is `isValid()` (with no parameters).  And `isValid()` is never referenced, nor is it an override of anything.

I propose you get rid of both `isValid()` and `bValid` as members and rewrite `PathRequest::doCreate` to use the parameter that's already being passed in.


@Raesu comment out this line and the app no longer crashes.


Issue https://github.com/jtrumbull/XMLParser/issues/1


De nada.
On Jul 31, 2015 6:34 PM, "Michael[tm] Smith" <notifications@github.com>
wrote:

> [image: :sunrise:] @halindrome <https://github.com/halindrome> much thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/w3c/respec/commit/85e2f950d69028a7300033fa05aee3281a64c0db#commitcomment-12476946>
> .
>


I assume it flickers because the two fields have slightly different vertical offsets.

Would need to correct for that somehow in the video backends.


You are far away from the distribution table. Try from a closer location to eat the food.


Go prefers early return (in this case early Fatalf):

```
ifaces, err := net.Interfaces()
if err != nil {
	t.Fatalf("Cannot get network host's interfaces: %v", err)
}

var ifaceName string
if len(ifaces) >= 2 {
	ifaceName = ifaces[1].Name
}
```


`defer os.RemoveAll(netdir)` after checking for error


Thanks for the explanation - I think a note on the blog post mentioning that ajax loading of fieldsets and fieldsettabs won't work if the containing fields make use of dependencies would be good - I am sure others will make the same assumption that I did - I think it is pretty common practice to use field dependences inside these - hopefully you can get these to work soon.


It still seems to GPF with the patch on (unless I made some mistake with the manual patch). Strange.

Exception: https://ci.appveyor.com/project/vsz/harbour-core/build/3.4.379#L3805

Patch: https://github.com/vszakats/harbour-core/commit/f3a8a7c40c918e6569f8919c37a1ea74292b1360


(BTW, if you decide not to get rid of `isValid()` and `bValid`, there's a call to `isValid (RippleLineCache::ref crCache)` in `doUpdate` on line 538 that doesn't update `bValid`, so that should be fixed for completeness. But it makes more sense to remove the useless code than to add more code to support it.)


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.415.906d1df991304ea46024abe54f48da36e54b7097](http://build.palaso.org/viewLog.html?buildId=60470&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 859, ignored: 54 Build time: 00:19:17


No need for `else` here


@Leemoonsoo I was going through the Zeppelin codebase and found this. Why are we synchronizing twice on the same object next to each other?


Why was the majority of my work reverted? You kept some of my changes, so it's obvious that you did have my changes up to date when working on this.


@UniversalMediaServer/developers @Optimusprimeums do we want to do a release soon? I think the Netty rollback is important to deploy since that fixes some streaming issues...?


My guess is that @coolsquid didn't re-open the mappings when he pulled, this problem also happened on a couple of the others commits overwriting my work.


Changing notification text


이거 제외한 이유가 있나요?
개인적으로는 lombok의  로깅을 선호합니다만..:)


이것도 개인적으론 이게 더 가독성 좋아보이긴하는데.. 어제 확인한 Optional 을 사용해볼 수 있는 부분인듯합니다.


`Fatalf` must be called from test goroutine. See http://golang.org/pkg/testing/#T.Fatalf and http://golang.org/pkg/testing/#T.FailNow


edit 의 경우는 저번에 제가 말씀드린대로 PUT 이 맞지 않을까요?
그리고 URL 뒤에 /edit가 붙는게 좋을까요? (요건 개취일 수 있어서..)


수정인듯한데 GET 은 맞지 않는것 같습니다.


Changing icon image


여기는 제가 좀 더 테스트해보고 다시 코멘트 하겠습니다.


rollback


화면은 차후에 정리하겠습니다.


Added `rc.cwd` which defaults to `'.'` because during tests prebuild kept writing to `./build` folder in the project root. Let me know if you think this is ok or not.


Just added some extra info to the log. It's nice to be able to see which binary that was actually loaded.


91977808_LBNL_NewVRFModel (rongpengzhang) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-61.07%25-red.svg)


freemaker 를 잘 몰라서 화면은 리뷰하기가 어렵네요..^^; 공부해서 리뷰하겠습니다.


I know it defaults to `.get` but it makes it easier to test if `.get` is used, i.e. I can hook the method


I suspect this should be a call to `preflight2 (ctx)`.


comentario teste linha 880


This fixes the problem when the host is invalid (i.e. not even a status code) when we bail out with `cb(err)` which means we have a dangling temp file.


I also noticed that I obviously can't call `tempFile()` and get the same file name twice, since the name is random :) Changed this by hooking into `fs.createWriteStream()` instead.


Arquivos Dropbox/sql/scripts executados na base de test executar novamente no novo dump.sql

Teste comentario em linha


If for some reason a `.tar.gz` does not contain `.node` file we should error


AddCalcCoilSurfTempSource (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (729 of 729 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-7.13%25-green.svg)


@VikeStep It's a problem with the way Enigma works. When the file on disk changes (e.g. after `git pull` gets run) Enigma does not reload it, so the next time Enigma overwrites all the modifications made. The solution is to close and reopen the mappings file after any change is made to them outside of Enigma itself. I am going to see if there's a way Enigma can be made to automatically reload files from the disk.


RadianceRefactor (rpg777) - Win64-Windows-7-VisualStudio-12: OK (2197 of 2215 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.19%25-yellow.svg)


Hey @ericyanLr,

I would think adding the characters in method one would be prefered or cleaner.  Can we leverage something similar to the portal.properties to determine the blacklist of characters/names that cannot be used?

Please see https://github.com/liferay/liferay-portal-ee/blob/master/portal-impl/src/portal.properties#L8938...L8953 for the portal.properties blacklist.


How does that explain why a couple of my changes stayed though?


I suspect that calls to `preflight1()` and `preflight2()` are absent for good reason.  However, as a casual observer, those reasons are not immediately obvious.  It might be nice to include a comment justifying why those checks are not being called.


I would guess @coolsquid putting them back in himself, not sure


http://karma-runner.github.io/0.13/index.html
Here is the JS unit testing link


This is to ensure a custom url based on `pkg.binary.{host,remote_path,package_name}` is properly formatted. I've seen examples where these properties have both leading and trailing slashes and `./foo` as well


Thank you for pointing this one out! I do this all over the place. I'm very curious why the code works as expected when the docs state otherwise :confused:


@edhurtig hmm, that's a good point. I went this route for simplicity's sake. We did not have a cloud-based approach before in this cookbook, by the way.

I'll rework it to have both on Monday.


Broken, sorry, tossing off code while I'm dealing with family packing for vacation is not a good plan.  Fix coming.  In my defense I didn't make a PR yet :-/


this is great.  look forward to playing with it.


You are right, thanks!


How hard would it be to move this algorithm into `mulDiv`?
```
std::uint64_t mulDiv (std::uint64_t value,
    std::vector<std::uint64_t> mul, std::uint64_t div)
{
  [...]
}

[...]
   fee = mulDiv(fee, {baseFee, feeFactor}, denom);
[...]
```


일관성있게 정책정한뒤 가면 될 것 같습니다.
내일 모임 때 이 이야기 꺼내주세요.


[PR](https://github.com/apache/couchdb-couch/pull/82) submitted.  Thanks!


Just confirming that this is correct - the `Change` transaction is special and the checks in `preflight1` and `preflight2` are not applicable to it (in fact, they would incorrect reject the transaction). A comment might be a good idea.


@Subjunk, i'm ready for whenever,  but i'll be submitting a PR (small OSX Pom adjustment) shortly that needs to go in before release


maybe add a comment here to explain why such an if?


unnamed parameters are :(. would this be worth creating a class enum / few constants?


I'm not exactly understanding the data flow here that prevents this from having a logical race. Could the server shutdown flag possibly be set just after this load check and before this function returns? If so, would that be a problem? If it's not a problem (e.g., because the flag only gets set when there is guaranteed to be another event or something like that), then could you document that in the code?


This commit LGTM.


Qt is the one making sure to free this (because you call "connect" on it below)?


Please see my comment on the related commit in this PR regarding how we know that that there's no race on the shutdown_flag, or if it matters.


On Centos 7 do you need a BuildRequires:   python2-devel?   I was getting failures without it.

EBUG: + cd openstack-nova-12.0.0.0b2
DEBUG: + '%{__python2}' setup.py build
DEBUG: /var/tmp/rpm-tmp.aEA7Ay: line 29: fg: no job control
DEBUG: error: Bad exit status from /var/tmp/rpm-tmp.aEA7Ay (%build)

looks like Centos 7 /etc/rpm/macros.python2  is provided by python-devel-2.7.5-18.el7_1.1.x86_64


this needn't check locality?


Done.


I suspect that it's not guaranteed to work but sometimes works -- basically a race.


オーブがそろった瞬間にゲームクリアとする #629


ゲームクリア時に中断セーブデータが消えない #615


My gut is telling me that we should account for `den` here before we throw the `overflow_error`.


It would probably be best just to update the existing `mulDiv` signature to this algorithm.  The resulting algorithm would be simpler than what is under `scaleFeeLoad` since it has only 2 factors in the numerator instead of 3.  I don't think combining the 2 and 3 factor algorithms is worth the allocation cost of the `vector`.

However this is a good reminder that I should go ahead and fix `mulDiv` in this same commit, and not leave that hanging.


`Master` is not optional -- it should always be specified.


`lftNormalFee` is a `static const`. It had better not ever change.


No worries, just got an email off of it when it was pushed in so I glanced at it early.


991757360-DocIssues-4956-4997 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1777 of 1778 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


I only moved this code from another file. If you still feel this is important enough to address, please do it in the relevant PR: https://github.com/ripple/rippled/pull/1210


🌚 can we just put the Unicode string here hahahahah


Looks good.


I'm using the template for `ptp` too which doesn't take `Master` according to https://github.com/appc/cni/blob/master/Documentation/ptp.md.


I found in official, the window is showed before 'close'.


Such long maybe multline?


Should be fine, I vaguely recall adding that for a reason but can't remember right now.


I got an answer on stack overflow...

http://stackoverflow.com/questions/31747029/how-do-i-import-across-submodules

And the intial foray of refactoring works.

Minuses - it's not what I see as a suggested format.  Not sure why, though.
- it doesn't actually explain the root cause, it just provides a new
approach

Pluses - it makes working in test mode easier as there is ONE place for
tests, so if you are working across modules, you can still think about
tests in one place.
- it gives a nice clear place for integration tests - for example, I have
some functions where Events meet Tickets - I want to unit test, but I would
gladly say its "integration" as the point between gbe and tickets
(conversely since the code is functions in gbe, it could live in gbe
tests....)
- it works.  First attempt to refactor my problem test worked.

"It works" is my strongest argument.

-Betty

On Thu, Jul 30, 2015 at 7:01 PM, Jon Kiparsky <notifications@github.com>
wrote:

> Looking at this further. It's surely weird. I can import gbe.models (or
> dependent objects), but I can't import gbe.tests (or any of its dependent
> objects). Same problem holds for scheduler/tests
>
> I've tried some experiments with __init__.py, but no luck so far.
>
> On Thu, Jul 30, 2015 at 1:58 PM, Jon Kiparsky <jon.kiparsky@gmail.com>
> wrote:
>
>
> > Hm. Can confirm that this is an issue, not sure why. Will try to come up
> > with an answer.
> >
> > On Thu, Jul 30, 2015 at 12:40 PM, bethlakshmi <notifications@github.com>
> > wrote:
> >
> >> First test cases...
> >>
> >> Need help…
> >> @jonkiparsky - I am stupidly re-coding UserFactory and ProfileFactory
> >> in my ticketing/tests/factories.py to make this work.
> >>
> >> What I want - is to figure out how to import and reuse your factories
> >> in gbe/tests/factories - but I’m stuck.
> >>
> >> ---
> >> Reply to this email directly or view it on GitHub:
> >>
> >>
> https://github.com/bethlakshmi/GBE2/commit/03c73269a9fa9581c9085b3ff9397794ce238e1f
> >
> >
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bethlakshmi/GBE2/commit/03c73269a9fa9581c9085b3ff9397794ce238e1f#commitcomment-12456611>
> .
>


Yeah, it can be a bit weird to understand if you just glance over it.


93469234-DemandResponse2 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1789 of 1789 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I love spaces yes I do
I love spaces how about two


how does into() differ from through2.obj ?


Les mots et links sont clairement publics mais pour les listes, on peut vraiment voir ça comme un dossier dans un filesystem UNIX.
Il appartient à un utilisateur et un groupe et il y a des droits le lire/écrire pour
* tout le monde
* les membres du groupe
* l'utilisateur
Publique revient à donner le droit de lecteure à tout le monde, le groupe et le user.
On peut encoder les droits avec (0 - aucun accès, 1 - lecture, 2 - lecture écriture).
Le nombre décimal "xyz"  veut dire x pour le user, y pour le groupe, z pour tous
Par exemple, 210 veut dire que le user peut tout faire, le groupe peut juste lire et le public ne peut rien faire.
Le droit 000 n'a pas vraiment de sens car le user doit bien avoir le droit de lecture mais on peut imaginer que 100 a du sens au cas où le user veut s'empêcher de faire des changements par inadvertence (il peut toujours se redonner le droit de lecture si il veut).

Un groupe se créerait assez simplement, un mec crée un groupe, ajoute des gens dedans en donnant au gens soit le droit de rajouter des gens dans le groupe, soit juste le droit d'être dans le groupe.
On peut éventuellement ajouter la possibilité d'ajouter des gens dans un groupe sans leur permettre d'avoir des droits d'écritures grâce au groupe.

Les droits d'exécution des filesystem UNIX n'ont pas tellement de sens ici :)

Après on est pas obligé de faire comme ça.
Pour les droits d'écriture, à la place on peut faire un système de pull request.
Tout le monde peut copier une liste dans ses listes (mais la liste reste au créateur, si le créateur la change, la liste copiée est changée aussi, ce n'est donc pas vraiment une copie mais plus comme les liens symbolique des filesystem UNIX.
On peut alors faire une vrai copie, on a plus les changements de l'autre mais on a le droit d'écriture.
Après si le créateur fait un changement on peut "puller" le changement quand même (comme un fork sur git) et si on fait un changement on peut le proposer au créateur (comme un pull request sur Git).

T'en penses quoi ?


I don't think it should.
You should be able to turn off local db and then remove the existing database anyways.


reminder: remove this var


Ok, presumably that is to catch some errors during initialisation before an import.


Write commit messages clearly to let others know what you're doing.


Should be fine, I vaguely recall adding that for a reason but can't remember right now.

I will do some analysis afterwards to ensure nothing could be broken down the line.


Using `tabID <= 0` because it would be invalid to have a tab with an ID of zero. The condition still fails for null, because `(null <= 0) === true`


With latest master, this has to be re-worked a bit. You need to extend this patch to include these new APIS:
https://github.com/01org/ozone-wayland/blob/master/patches/0008-PlatformWindow-Add-needed-support-in-PlatformWindow.patch

Implement the apis in https://github.com/01org/ozone-wayland/blob/master/platform/ozone_wayland_window.cc,
where you can dispatch the   necessary IPC calls.


We need to be careful with changing routines that perform math, if they are used in transaction processing it could affect validation.


You even cleaned up the whitespace \o/


Sorry, that you have to redo some of this work.


Interesting, I don't know anything about mono but if that makes it work that's great. I will inspect other parts of the code and remove any '\' with the preferred Path.Combine.


Looks fine


copying here in case you didn't see my message in clojurians.
@ulsa:  reactions are tricky if they are not derefed inside a render function, as I've written here https://github.com/reagent-project/reagent/issues/116
using `reagent.ratom/run!` instead of `reaction` for your validation reaction should fix it


Qt frees this because of [Qt::WA_DeleteOnClose](https://github.com/Kazakuri/Shinjiru/blob/master/src/gui/animepanel.cpp#L17).

The connected lambda will run once ap is accepted, before it is deleted (not that it requires ap or anything)


This should specify IgnoreCase possibly


Note: you can probably remove the '@' as well since that was to avoid adding a double slash.


I don't think it's important. I don't know of any server that will be inconsistent in its channel name casing.


The rest of the library only checks case if the input is user provided.


What does this do if a user types e.g. '1,3,none!'?


Alright, nice. I'm asking dumb questions because I don't quite know the "Qt Way", sorry :).


I have had issues with Chatsharp where the item in the collection was differently cased. If the KICK is received with channel name cased differently than you have stored, this might bomb-out


Can you open a Github issue about casing in general?


Don't worry about it. :+1:
The Qt signal/slot system and having another library worrying about parents and children wasn't easy to understand. (if I can even say I understand it :laughing: )


it's not working on caffe-0.999 release....


Ah, I think I approved to myself that that wouldn't be necessary.  It was too much to put into comments, but perhaps this is a good place to outline my "proof."

Consider:
````
x = a * b * c / d;
````
where `a`, `b` and `c` are sorted, largest first.  Obviously we need to compute this without overflow: either in the final result, or intermediate overflow.

If `a * b * c` doesn't overflow, things are easy:  `x = a * b * c / d;`

Otherwise if `b * c` doesn't overflow, we can do either `x = a / d * b * c` or `x = b * c / d * a`.  Which of those we want to do depends on whether `a < b * c` or not (for retaining the most accuracy).  But either will be good.  But if `b * c` *does* overflow, then both of these formulation will overflow, unless `a / d` is 0.  And we've already established an invariant that `a / d != 0` (with the if-statement).  This proof is a large part of the motivation for ordering `a`, `b` and `c`.


Updating local repository.


This function is necessary to prevent a redraw when changing orientation. All that needs to be removed is this line.


fix : Undefined variable: user


fix : Undefined variable: full_remote_file


@dsyme Was removal of these guys intentional? I don't see the handling moved to a different place.  Post-merge I see a test failing due to `InternalUndefinedItemRef` internal error.


fix:Getting unknown property: gftp\FtpComponent::param


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.436](http://teamcity.monogame.net/viewLog.html?buildId=9247&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Fix it!


AddCalcCoilSurfTempSource (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1246 of 1247 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.92%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.97%25-green.svg)


Good to see this, I've removed this in Fedora package for a long time, not sure if other distros are against this. :hamster:


Because there are more than one install script for many games ;)


second commit


This didn't need to be removed, just updated:

case android.R.id.home:
				if (mNavigationDrawerFragment.isDrawerOpen()) {
					mNavigationDrawerFragment.close();
				} else {
					mNavigationDrawerFragment.openDrawer(NavigationDrawerFragment.mFragmentContainerView);
				}
				return true;


Activity MSCS 524


The two lines above could still be used. See line 209


This should be a string resource to support translation.


Still fine after > 12 hrs.


Missing `recvmmsg` still.


Also added rotation logic


Suggest a comment `//note: vlen is not size_t`


We do not need sudo to run this command


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.436](http://teamcity.monogame.net/viewLog.html?buildId=9247&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:23:57


Thanks! :)


    <meta name="google-signin-scope" content="profile email">
    <meta name="google-signin-client_id" content="YOUR_CLIENT_ID.apps.googleusercontent.com">


This hasn't been pushed to `adarkroom.doublespeakgames.com` yet...please do so I can actually play.


![arguing on facebook with a retard_cf066d_3973766](https://cloud.githubusercontent.com/assets/5364479/9020234/b6c9eecc-381d-11e5-9b25-25c86263a83b.gif)


THis is Comment


Why loosen the linter here?


Can you provide me an example about this? I'm trying to make some custom fields, but without success, checked the UserFactory's class to understand the new methods, but it's not working.


how close are we to submission


Thanks for the credit and copyright, nice idea, If I used that app I'd pull this for sure


This doesn't account for the different kinds of link formatting (``[[ . . . ]], google(. . .), wiki(. . .), gl(. . .)``).
The regex you're using here could be more optimal. There's no reason you'd need to capture the entire regex since if you ever need to use the entire captured string, it can be assigned to a variable (or if the second parametre's a function, its first parametre). ``/([`~_*])\1/g`` is more efficient than ``/(\*\*|\_\_|\~\~|\`\`)/g`` in this case. With the latter, if the regex engine encounters the monospace tag in the string, it will still try to compare it with ``**``, ``__``, and ``~~`` first before finally matching it, while the former just checks if the first character is included in the character set or not before failing.
I might consider modifying it to only match valid formatting on the off chance a banned phrase includes characters that this would match. ``.replace(/``([^ \`]+)``/g, '$1').replace(/~~([^ ~]+)~~/g, '$1').replace(/\[\[([^ \]]+)]]/g, '$1').replace(/(?:g(?:l|oogle)|wiki)\((?!\s*\))([^)]+)\)/g, '$1').replace(/__([^ _]+)__/g, '$1').replace(/\*\*([^ *]+)\*\*/g, '$1')`` should match all types of formatting in order of precedence.


@ThomasLee969 话说赛道那个next prev是编号咩……字典真是太机智……界面调用的部分呢？


Ok, I found a formatted version of the phylesystem stats from 1 July - I copied it to phylesystem.json.  This should allow things to run again, though I should do something to make sure a failure like this doesn't happen (e.g., write to a temp file and rename).  If @jar398 has any backups that are more recent than 1 July it might be helpful.  And I'm sorry I didn't notice this formatted file until after 0:00 GMT so the next stats will be for 2-Aug.


I always forget that I disable block breaking for debug purposes when I push changes. Sorry!


Yes, this preserves the state through hot code reload. That's the only goal.


Just to clarify, I am not talking about "dependencies between ajax-loaded inputs", but within an ajax loaded fieldsettab, which currently seems to be broken.


The parameter is still called rate, shouldn't it be the same name?


I plan to test above with attr() to ensure apostrophe doesn't break it still. If it tests well, then will commit this.


Should this be `except (OSError, IOError)`?


Is this a bug?

![image](https://cloud.githubusercontent.com/assets/756669/9021512/3ca93026-3845-11e5-8952-9309a4fefb42.png)

The 4 blue squares instead of 1?


Murmillo Helmet


Yes this number 0 is the columns count in this row, so it should remain 0


OK, what is now the relation between `removeManuscript` and `closeDocument`?


Revert from the past 2 years kRO item updates?
https://github.com/rathena/rathena/commit/5a0f8dc12ae1d0b2c3bf0a9fecc085719a28622b

for reference this is the kRO desc;
타 계정과 거래가 불가능한 아이템 입니다.
사물을 입체적으로 보이게 한다는 신비한 안경. 자신의 성장 정도에 따라 받는 효과도 증가한다고 한다.
검사계열, 상인계열, 궁수계열, 건슬링거, ATK 일정량 상승.
마법사계열, 복사계열, 닌자, 소울링커는 MATK 일정량 상승.
도둑계열은 FLEE 일정량 상승.
노비스, 슈퍼노비스, 태권, 권성은 HP, SP가 일정량 상승.
계열 : 투구 방어 : 1
위치 : 중단 무게 : 10
요구 레벨 : 1
장착 : 전 직업


wurde rausgenommen :D

(hatte ich testweise drin, da der Job nicht richtig lief - lag daran das ich kein PATH drin hatte)


Hi @hartmannaf would you please replace all `unsigned int` to `u32` to make it a little shorter? also `unsigned char` to `u8`. Cause you have those definitions already?


@arunk-s would it be safer to add expect ValueError as e: (for int(transaction_id) and int(target_id) ?


Das mit ```fastd --show-key``` hab ich leider erst hinterher rausgefunden.
Daher auch der Kommentar.

Okay, mach es Rückgängig


Eh.. I recall this from rxTools myThread.c isn't it. However it is due to a timing related patch job. I don't know if this is really need to be done in such a way when all stuffs already loaded totally. Thanks at least you could keep this if not cause too much time.


test


Thank for your fast reply saudet. I already know OpenCVRecognizer and I would be very glad to use it, but I can't because the opencv_face import is broken. I donwnloaded the javacpp-preset 3.0.0-1.0 with maven and the source open_face.class is missing. I also made a check directly from the jar downloaded by the maven rep (http://mvnrepository.com/artifact/org.bytedeco.javacpp-presets/opencv/3.0.0-1.0) but I can't find this source at all. I have seen this class only in the github repository.


Good this is not too hacky as what i expected.. If i do understand in a wrong way, could you please tell me where you are feeling uneasy for this commit?


Yes, I will clean up the  code later today, after that I will implement the other functions needed for the patch types and rewrite the menu generation. Don't know how much time I will need for it.


essai essai essai


Just do it step by step. You could clean it file after file.
This u32, unsigned int, or other type names are just naming problem. Not severe.


`| UltraEdit |  | IntelliJ-Based Editors (PHPStorm, WebStorm, etc.) | `


Hi Rene,

the ordering of muons and electrons for the EM pairs was defined in this unintutive way because of the definitions of the pair selection algorithm (https://twiki.cern.ch/twiki/bin/viewauth/CMS/HiggsToTauTauWorking2015#Pair_Selection_Algorithm). If I understand it right, candidate 1 is the muon and candidate 2 the electron in the EM channel. The more intuitive ordering (candidate 1 = electron, candidate 2 = muon) was preserved in the product.m_flavourOrderedLeptons. I think, this commit needs to be reverted.

Cheers,
Thomas


Turns out there was a byte-order problem with 'Packet' most objects. This fixes that problem.


Can you give an explanation why it is outdated? What did change? Is it still working?


This is a proof of concept for the design where `listenTo` is passed event names as strings.  I only implemented it for a single `event` string argument, but the generalization to more arguments is clear.

Before we get there though...

## There are some problems with this design.

Let's consider the example usage:

```js
@listenTo('mouseDown')
class MyComponent extends React.Component {
    onMouseDown() {
        console.log('down')
    }

    // render component
    render() {
        return (
            <div styles={[
                styles.container,
                this.props.mouse_down && styles.mouse_down,
                !this.props.mouse_down && styles.mouse_up,
            ]}>
                <p>My Component!</p>
            </div>
        )
    }
}
```

### Problems for us implementing this decorator
(my commit fails to address these)

- Based solely on the string `'mouseDown'`, how will our implementation know that it must not only attach a listener for `'mouseDown'` events, but must also listen for `'mouseUp'` events so as to reset the `mouse_down` flag to `false` when the mouse comes back up?  Sure, we can hardcode `listenTo` to know that `'mouseUp'` and `'mouseDown'` must go together in this way, but that's clearly not extendable to all other react events.  In fact...
- Some react events don't even fall into this design pattern of flicking a boolean when the event happens (for instance, the `'onSubmit'` event).  So what events are we trying to cover with this decorator?  If all, then this design might not work.  If just the ones where flicking a boolean makes sense, then it might work.  In that case we need to hand pick out the list of such events (which is ok).

So, in cases like `'onSubmit'`, where you really just want to attach a handler for that event, it seems to me that most of the code is done in this commit.  However, if we also want to address the concept of adding some state to the component based on react events like `mouseUp` then we have more work to do, and a good design is not entirely clear to me at the moment.

### Some other side notes to keep in mind for later
- A `'mouseUp'` event might not even be triggered on this component if the user clicks down on the component, moves the mouse off the component, and then releases the mouse click.


Earlier would need to be earlier than something else, compared to something. I think early is correct here.


@mikelove I'm using this one currently, old but it's working: https://testtoolshed.g2.bx.psu.edu/view/bgruening/deseq2
Now that we have the newest R version, I can work again on the new version.


This is based on the changes that we've been making with the code review process. It's expected that each commit be reviewed going forward. Perhaps the language could be a bit less "forceful" but the expectation is that 100% of the commits is reviewed, at whatever time schedule that may be.


NOT YET FINAL ...  need more work ...

some note:
* only prototype,  just a proof of concept  ( I am still learning golang )
* this patch add 5 osm_* meta key-values to the other tags:
 * osm_changeset
 * osm_version
 * osm_user
 * osm_uid
 *  osm_timestamp
*  work with  "import" and "diff" mode
* no paramater
* no optimal and  not optimized code
* no test code

my simple config for testing

```bash
cd /gopath/src
git clone  https://github.com/ImreSamu/imposm3.git -b osm_metadata --single-branch /gopath/src/github.com/omniscale/imposm3
cd /gopath/src/github.com/omniscale/imposm3
go build  .

OSM0=...your_osm_file.osm.pbf
./imposm3 import  -mapping ./test/single_table_mapping.json  -read $OSM0  -write -overwritecache -deployproduction  -connection postgis://osm:osm@xx.xx.xx..xx/imposm4
./imposm3 query-cache -rel=1596736
{
  "relations": {
    "1596736": {
      "tags": {
        "amenity": "place_of_worship",
        "building": "yes",
        "denomination": "roman_catholic",
        "osm_changeset": "29323712",
        "osm_timestamp": "2015-03-08T07:22:55Z",
        "osm_uid": "2352833",
        "osm_user": "Keresztvölgyi",
        "osm_version": "3",
        "religion": "christian",
        "type": "multipolygon"
      },
      "members": [
        {
          "id": 114422717,
          "type": 1,
          "role": "outer"
        },
        {
          "id": 114422745,
          "type": 1,
          "role": "inner"
        }
      ]
    }
  }
}
```


`.idea` directory should not be included in the commit, due to this, I have some conflict while trying to forward my master branch to yours. You can also consider adding it to `.gitignore` to ignore `.idea` directory.


Nice one adnan.


Before moving them in slow tests, I'd prefer checking why it is slow. I don't think it ought to be :)


I think it's ok since it defines a style that is maintained in the following paragraphs but that style could be more defined instead of removing the bullet point. Perhaps having a subsection that lists the things that an auditor and/or an author can do separated by roles would be better.


That's why I said earlier -In de dev process or smt like that- Instead of saying just early. I didn't mean to just change a word, sorry I didn't explain myself correctly.


yeah, but not using that port.


This is just a copy/paste of the code above (and it's missing the part that handles passing `options` as a computed property). Instead of having a separate type for this, you'd be better off having multiselect as an option in the descriptor.


We could work more in the language, since it's not reflecting what the process is. The objective is to have all reviewed, of course, but there are steps to follow in order for that to happen.


Ok, that seems alright. But I'd put that in another commit, this is the current language we're using, Simon didn't change it so it's not really associated to this pull request


Agree on this.


Cool.


It is the util in the application itself probably better to have it in the GOPATH I am guessing. Should be something to add to your own env like a pak. Ewwww.


I meant to move it outside the critical section. Thanks for pointing this out.


Sweet looks good. Yeah I should switch to using my own branch as well. Was thinking of working on the server side of things tomorrow. Testing and new code. I feel we should try to keep on top of commenting and tests, form the start.


Each Ember view already gets a unique id and already implements the `$` method.


It's outdated commit, current version work ok


pls revert to 2014 ![horrible](https://cdn.joduska.me/forum/public/style_emoticons/default/roto2.gif)


Use a computed property here: `Ember.computed.not('isDisabled')`


Could you please rename this to AttackBase? `AB` is a rather ugly variable name.


This can also be achieved using the `classNames` property (and then setting the tagName to empty string isn't needed).


:D


Where you got the info nana?

dari mana infonya na? setau gw kRO ga ada slot deh, kecuali hidden slot


this is not the right way to obtain such a score in a responsive design and it's even not flexible in relation to different word lenghts in different languages.

it is possible that "Context" and "Files" in Urdu have a reversed lenght.


Basically I agree, but back when I did this I felt it might be the lesser evil compared to ambiguity (AttackBase of this particular actor vs. the trait in general).


Let me explain the reasons why I did that.

> Don't you think that we are trying to optimize/outsmart V8 here without having seen any performance hit?

No I don't think so. In javascript, there is no way to express that a given function will be [pure](https://en.wikipedia.org/wiki/Pure_function). Because of that, there is no way for an external tool to have a statistical hint about the behaviour of the function. None of the javascript engines (V8, Spidermonkey, Chakra and so on) can have a real clue about the behaviour of the function. You can go through a function a millions times with the same parameters, and still at the next call you can have a different output.

But what is [memoization](https://en.wikipedia.org/wiki/Memoization)? This is a way to cache the results of a pure function. It trades computation for memory. Afterwards, the memoized function will cache the result to avoid computing the results again.

And here, we are in the perfect case. Template compilation is a pure function. We  know that we will have few templates (in the tens) that we ask for compilation (a hard task) a lot of time. Just take the results component, a single template each time we do the rendering, at each key stroke. Let's say we have a user that types 5 characters and we have 20 result component, it gives 100 compilations. Therefore we have a 100 factor optimization in time for just one memory taken. Also without the memoization, the memory would have been filled with generated compilation that would have to be garbage collected afterwards.

BUT :) After reading Hogan compiler code, we can see that it [has a caching system](https://github.com/twitter/hogan.js/blob/master/lib/compiler.js#L407-L422). So it seems ok to remove the memoization. So it's ok to remove the memoization process.


test


I think the title of the activity can be improved. `FirstTimeUserIsShowingActivity` gives me an impression that it is about TourGuide being shown to the user only **once**, then you will hide it forever until the user clears data or reinstall the app.

However it looks like what you've included is `mTutorialHandler.isShowing()` method. This only return whether TourGuide is currently showing. I think `FirstTimeUser` should be removed, what do you think?

Besides that, I'm wondering what is the use case of `isShowing()` method, can you give an example? Like a practical use case in an app user flow. No need to write an example (it's best if you can of course :+1: ) but describing is fine too.


Passing mtu then using harcoded value 2000? Does not seem correct to me


imho, Nana you should follow kRO's not iRO's description


This is inconsistent with the other tests, which are in the `OpenRA.Test` namespace.


We should avoid state here. Indeed because of the internal cache of Hogan, we can simply call for the compilation directly and we will have a pretty consistent low time for compilation + rendering of the template.

```javascript
var content = hogan.compile(this.props.template).render(this.props.data);
```


In rxtools for the timing it is mostly the big area that is searched. Maybe it could be improved, but that could be done later


Copy-and-paste programming is generally frowned upon. It would be better to either add an option to MenuView, or subclass MenuView.


This comment is incorrect, the mouse events are attached to the MenuView element (and only after the MenuView has been inserted in the DOM).


This comment is also incorrect, here the mouse event handlers are removed (as cleanup) right before the menu view element will be destroyed.


Interesting suggestion, but I think it would be unnecessary to add a `expect` as the handler is triggered only when the urls looking like this occurs: https://github.com/arunk-s/owtf/commit/dbd64aa1afe2aa99fdccbbeeb777854f02583abf#diff-ac967e581fa8c10320906369486d9f9aR22 , i.e `/api/targets/([0-9]+)/transactions/([0-9]+)/transform/?$` There you can see that both target and transaction id are expected to be integers only.


Does not work for me. Gives me the following error:
```
Uncaught Exception:
ReferenceError: episode is not defined
    at Request._callback (/home/max/development/web/nedflix-api/index.js:25:13)
    at Request.self.callback (/home/max/development/web/nedflix-api/node_modules/request/request.js:198:22)
    at emitTwo (events.js:87:13)
    at Request.emit (events.js:172:7)
    at Request.<anonymous> (/home/max/development/web/nedflix-api/node_modules/request/request.js:1057:14)
    at emitOne (events.js:82:20)
    at Request.emit (events.js:169:7)
    at IncomingMessage.<anonymous> (/home/max/development/web/nedflix-api/node_modules/request/request.js:1003:12)
    at emitNone (events.js:72:20)
    at IncomingMessage.emit (events.js:166:7)
```

Have you written some more in nedflix-api but not yet updated the gh repo?


Aaah perfect. I'll update.


Jup, you can pull it now.


Thanks for the information.

Look again to see the changes.

I don't like the fact that Dictionary raise an exception if key does not exist on reading. To get the discount from discounts (Dictionary), I had to check if key exists. I think this also takes O(n) time. If you have a better idea please let me know. Thanks.

> On 1/08/2015, at 10:12 am, Yue Wang <notifications@github.com> wrote:
>
> Hi @ducktyper ,
> This code looks a practice for C# syntax, so performance isn't the point.
> Anyway, just a reminder that the statement in line 53 performs a linear search, taking O(n) time to run, where n equals to products.count(). [1]
>
> //line53:
> return products.Find(x => x.barcode == barcode);
> To improve the performance here, IMHO the better option should be Dictionary<Key, Value>, which works similar to Ruby's Hash and has a faster lookup. [2]
>
> [1] List.Find Method from MSDN (note the Remark section)
> [2] A benchmarking on List and Dictionary
>
> —
> Reply to this email directly or view it on GitHub.
>


Implementing is not finished, and the test function loads the patches :)


질답 내용은 Lob에 담는게 나을까요?
파일로 떨구는게 나을까요?


may use `rtlJson[i].poster`. Not sure..


요 커밋 내일 설명 좀 해주세요.


南方电网项目，自然环境、复杂背景下绝缘子目标检测


Good catch. Maybe write a test for this? You should be able to easily test this method directly by using mocks :)


Is this going to become a point release?


Tnx


After some thoughts, it is actually a good idea. For some reasons :
 - no more hard dependency on a specific version of the helper (eg in the proptypes)
 - more precise requirements, which lead to a smaller black box effect
 - this is indeed more easy to test :)

I already encountered the first point while reusing the SearchBox component. The only meh good point is that it requires to create a new function to encapsulate parts of the Helper. This should probably be cached at some point, or we could use bindall after we instanciate the Helper (we'll see that later)


On Fri, 31 Jul 2015, scoder wrote:
> Assuming it's not a thread-local operation, it's also dangerous for multi-threaded code as a thread switch between the two statements can easily leak the FPU state into other threads or even leave the FPU in an invalid
> state if another thread gets interleaved which calls fpu_restore(). That would suggest to add protection by the GIL.
>
> In any case, it seems like this should be a purely internal operation.

Thank you; yes, I can add some protection. But actually, the module is intended for local single-user computing.

Best regards,

--
Thomas Baruchel


Temporary measure until #98 gets resolved to keep CI builds passing


How about removing `if (rc.path) process.chdir(rc.path)` and just pass around `rc.path` to where we need it. It could default to '.'


hahaha oops, but yeah change what you think is right :smiley:


This is not to remove a blank space, but to remove the quotes.
"key" : "value"
If the value doesn`t have the quotes, then the json is wrong ( considering that this is expecting a string )


Ah ok so this part bellow doesn't quite make sense then `we've made the count = '' above because`


This is not a valid change, because it allows pg-promise do the query formatting for prepared statements, which goes against the very nature of prepared statements that are to be [formatted by postgres itself](http://www.postgresql.org/docs/9.2/static/sql-prepare.html).

I have just released a new version that supports prepared statements correctly.


test


declare as ``final`` and move to the "Internal methods" section of the class.


declare statement as ``final``, its just our style


DB에 한꺼번에 담는 @Lob 가 관리면에서 좋다고 생각합니다. 파일로 떨어뜨릴 이유는 없다고 보내요.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 359](http://192.168.30.225:8111/viewLog.html?buildId=1823&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


그럼 검색엔진에서 인덱싱할때는 배치가 DB->ES 로 쏴주는 구조가 되겠군요


Okay I understand now. Yes it should be possible to support dependencies
within an entire fieldset or tab, so long as the fields operating on the
dependency are loaded together. If it's not working, most likely it's just
a matter of me needing to have the JS that handles dependencies attach JS
events to the fields in question.

On Fri, Jul 31, 2015 at 9:59 PM, Adrian Jones <notifications@github.com>
wrote:

> Just to clarify, I am not talking about "dependencies between ajax-loaded
> inputs", but within an ajax loaded fieldsettab, which currently seems to be
> broken.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ryancramerdesign/ProcessWire/commit/0d1d91ec7dcb6d859f410d5165495430136c8079#commitcomment-12477724>
> .
>


have nothing to do with me but Hi =))


I don't think this is the right solution. It introduces multiple levels of locking where only one lock is required. We should rather fix the counting_sempahore. It needs to be split into two separate types, one holding the mutex and one implementing the functionality (we have done a similar splitting for condition_variable). For the shared mutex we would need the inner counting_semaphore implementing the functionality only.


You have push rights to salkinium/hwut.
Just sayin'.


toplel


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 359](http://192.168.30.225:8111/viewLog.html?buildId=1823&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:53


Isn't this functionality already available through [boost::shared_lock](http://www.boost.org/doc/libs/1_58_0/doc/html/thread/synchronization.html#thread.synchronization.locks.shared_lock) or starting with C++14 through [std::shared_lock](http://en.cppreference.com/w/cpp/thread/shared_lock)?


Fontconfig manque ds la collection graphic. name=oxygen...->_name=oxygen.... Et pas besoin de spécifier la version. Elle est spécifié dans le port plasma.


@jdegoes I'm completely stuck here. The  `whatX` and `whatY` functions that are supposed to be handling the right part in `queryParent` seem to be unimplementable :disappointed:

I tried flipping the `WidgetF` and `StateF` coproducts to see if that helped with the types, but it didn't help at all (perhaps unsurprisingly), just rearranged the problem.

Maybe there is a way of doing this still, but I've spent quite a long time on this and can't see a way out in this form. I think one option might be to change things around so instead of having `Subscribe` feeding back `f` inputs into the query processor it returns a `Free (Coproduct (StateF s) g)` instead, like the result of the query processor (almost - no `WidgetF` in the coproduct).


It'd be great to make this into a daemon that uploads backlog data from aprs.fi to tracker.habhub.org in real time. I'll make a few issues to do this


Hi Thomas,

thanks a lot for checking and commenting. Indeed you are right about the assignment of candidate 1 and 2 in EM. I will revert this commit and push another fix for the problem which was intended to be addressed with this one.

Cheers,
Rene


Well, it's the eclipse project generator by cmake, eclipse it self does not support cmake at all :P


Exercício 2.11.1 by Rodrigo : Adicionada a classe contato


Почему то мне кажеться что тут должен быть другой энтити


И тут


Please do not add include statements in random places. Move this to the top.


return 0;


Are these 4 includes really needed or do they belong to gradient.cpp ?


Since this is internal, please rename it to "PanTilt"
I fear the '&' can cause troubles in XML


Can you use HTTPS here?


> And therefore we can have a pure component, YEAHA!!

Ok I did the change


Fix Markdown anchor for SysVinit section.


@coreytrampe ah, I misread the documentation. Thanks for the tip about initialising too!

@mgold thanks for the advice on seeding too :)


Why not just `source ~/.bashrc`?


Header missing ?


Maybe /usr/local/miniconda is a better location?


Make api_key a (class) method instead of a variable. It never changes anyway.


Right. But the name is confusing.


well this is some bs right there, I'd prefer it if git wouldn't pollute the xmls with those conflict tags, then again i did not really understand what caused this conflict and i only managed to resolve it haphazardly :/


The file contents should be in a different code block than the commands to execute.


This is a QtCreator thing that I hate.
Please move this .h in the longer HEADER section, before grandmaster,h


Same as above


@schoeke does this fix your build problems?


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


External Cocos Helper Android Frameworks/Classes/JNIResults.h

You should include  `cocos2d.h` before `#if`
And you need actually only `platform/CCPlatformConfig.h`


For Enterprises version


This revision is clean.


Roxas told me that the region-free he implemented is timing related.
I did write a ugly memmem implementation for it but own build does not run properly.
Yeh you could always test out new implementations - users can not easily find it out.


I think this should be `var html = ...` as I don't see `html` declared anywhere else.


what we want to check here?


ooh. I see. You mean, I can just use a unique_lock or a shared lock depending on my access needs. That's great. I can throw this lot away. Thanks


If you mean `findAndPatchCode_test` i don't find much problems.
And for the layout.. Oh it could be improved to make loading patches easier.
Just load it sequently, you don't really need to use the structure while loading (not while writing of course).
*Eh if you do really like OO a lot, you could ignore such things..*


Removed all references to ActorProxyFuture, Fixes #156


set_content_cbの外でrputsすると、ここにNULLで来ますので、判定入れています。


So this class will extend the common refactoring class, the refactoring class will extend to the org.eclipse.refactoring class. I move everything to the common refactoring class, except the test
Cases. But I creat the resources  folder at test class  not in the common refactoring class. . Am I right.


@axelerod I use local variables in these cases because FindBy annotation doesn't work with variables (postTitle, translateTo) in XPath.


set_content_cb　の外でrputsを呼び出すと、write_large_bufがNULLで呼ばれるので、判定しています。


This is part of the work that @troydai was looking at to do with `dnu list` and build time dependencies.


Well, but including these themes only make sense if they are used, right? So shouldn't we run `conf.py` to decide which theme to include?


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1776 of 1780 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.78%25-yellow.svg)


Add private constructors to `binding` package's classes


We should not be using a LinkedList internally. We are implementing a Deque from scratch.


I believe this is a typo


No need for locks, since we are implementing a lock-free deque


NICE!


What RescueGiant is? )


We should usually use the stored setting and not UI values.


- Fixed some smaller Bugs with the External check creation and modification
- Added support for External  check periods
- Added some examples for external checks


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


The method signature should not be changed. popBottom() means that we remove and return the LockeThread at the bottom of the deque, so we should not have to pass in a LockeThread.


Sorry this is a refuse used only for testing and I forgot to delete it. stdio is not used anymore.


Thanks @psiroky !


Why the E item parameter? The element we are trying to add to the queue is a LockeThread, not an E. Again, no need to change the method signatures.


Same as above: the method signature should not be changed. popTop() means that we remove and return the LockeThread at the top of the deque, so we should not have to pass in a LockeThread.


Did you really mean to remove this?


 upstream backend {
        dynamic_resolve fallback=stale fail_timeout=30s;

        server a.com;
        server b.com;
    }

    server {

        location / {
            proxy_pass http://backend;
        }
    }

指定在某个upstream中启用动态域名解析功能。

fallback参数指定了当域名无法解析时采取的动作：

    stale, 使用tengine启动的时候获取的旧地址
    next, 选择upstream中的下一个server
    shutdown, 结束当前请求

fail_timeout参数指定了将DNS服务当做无法使用的时间，也就是当某次DNS请求失败后，假定后续多长的时间内DNS服务依然不可用，以减少对无效DNS的查询。

解析:
其中有一个结构体类型:

ngx_http_upstream_dynamic_srv_t  {
	ngx_int_t	enabled;
	ngx_int_t	fallback;
	time_t		fail_timeout;
	time_t		fail_check;

	ngx_http_upstream_init_pt  original_init_get_peer;
        ngx_http_upstream_init_peer_t  original_init_peer;
}

其中有一个fail_check变量,每次解析dns解析之前都会进行判断和上次解析失败的间隔时间是否超过了fail_timeout,若超过fail_timeout则直接进行调用原有的get_peer.

问题整理:
   1.当使用了ngx_http_upstream_dynamic的时候,每次在选择后端peer的时候都会先调用ngx_http_upstream_get_dynamic_peer(该函数可能会访问两次).该函数会去判断pc->resolved的值:

    其中resolved的值为:NGX_HTTP_UPSTREAM_DR_OK/NGX_HTTP_UPSTREAM_DR_FAILED 当其为第一个值的时候就直接返回(解析成功),当为第二种值的时候分为两种情况:
         第一种:当第一次dns无法解析的时候,就按照配置文件配置的fail_back进行处理(1.使用旧的ip,2.连接下一个server,3.结束当前请求)
         第二种:当进行第二次访问的时候要看dscf->fail_check,判断上次访问失效与本次访问的时间间隔,若其间隔时间小于配置文件所设置的参数fail_timeout(为了防止再次访问dns失败)就根据配置文件参数dscf->fallback进行相应的处理(get_peer或结束当前请求)

最后进行:ngx_reslove_start(clcf->reslover,&stemp)启动dns查询,设置一些ctx值.
接下来调用ngx_resolve_name(ctx)进行名字查询.
上面函数调用ngx_resolve_name_locked来根据hash=ngx_crc32_short(ctx->name.data,ctx->name.len)调用ngx_resolver_lookup_name(hash).

当在server中配置一个域名的时候----round_robin的时候:
     其该域名解析出来若有多个ip,则和普通的server一样进行初始化后端peer,如www.baidu.com有两个server.

注意点:
  1.当一个域名下对应有多个ip的时候:
      但是在调用ngx_http_upstream_get_dynamic_peer的时候调用round_robin_get_peer选择了一个server,但是其判断该peer的host->name是一个域名  此时nginx会做域名解析,最后只会随机的进行选择一个server进行连接.

  2.配置了dynamic_resolve指令可能不会生效:
      当在使用ngx_http_upstream_dynamic(upstream动态域名解析模块)模块的时候切记点:只有配置了resolver的时候才会去动态DNS查询(此处的配置resolver有两处:第一处是在nginx的配置文件中location server http配置块都可以  第二处是nginx会在该路径下读取(/etc/resolve.conf下寻找)当#define NGX_RESOLVER_FILE  "/etc/resolv.conf"
))

  3.当其upstream模块动态dns生效了如何进行dns解析:
      首先判断该host->name是一个ip还是一个域名,若是一个ip直接return ok,否则会在本地catche进行查询域名对应的ip,对其域名进行crc哈希得到一个hash值,然后在name_rbtree中进行查找得到resolver node节点(rn),进行判断rn ?= NULL ,当其不为空则表明在catche中找到了,然后判断其是否过期,没有过期的话就直接进行选择,执行回调函数ctx->handlr(ctx)设置pc->sockaddr进行后端的连接处理.
     当其所访问到的catche过期了则把该节点从队列中删除,然后重新发起dns请求,把得到的域名对应的ip copy到 name_rbtree rn中(rn->u.addrs=addr).(注意在前面只是删除了队列中数据  并没有删除name_rbtree中的节点)
  4.当查询一个catch中不存在的域名的时候:
      这个和第3点就多一步工作分配一个rn节点,并且插入name_rbtree中,得到该值后在插入队列.

  5.缓存中的rn节点失效时间是如何管理的:
      其中在resolver指令有一个参数vailed值(r->vailed = vailed),其中在进行节点失效与否时有这样一句: rn->vailed >= ngx_time();
      其中rn->vailed = ngx_time() + (r->vailed ? r->vailed : rn->ttl):
      其中rn->ttl = 0xff ff ff ff default  rn->ttl = min(rn->ttl,ttl)  ttl 是每次dns返回来的ttl.
      所以其rn节点的有效时间用户可以自己配置,若没有配置则使用dns返回来的ttl作为有效时间.

   6.当本地catch的某一个rn节点长期没有访问时如何处理:
      每次在get_dynamic_peer查询结束的时候（调用ngx_resolve_addr_done），都会检查有没有缓存过期，如果有，就会进行释放。
      其原理就是调用:ngx_resolver_expire  其中:
           q = ngx_queue_last(queue);
           rn = ngx_queue_data(q,ngx_resolver_node_t,queue);
           now=ngx_now  if now > rn->expire  则把该rn节点分别从queue以及name_rbtree中删除.
           其中每次缓存命中的时候都会 update expire: rn->expire = r->expire + ngx_now(); r->expire=30

   7.区分resolver指令中的vailed与dynamic_resolve中的fail_timeout的区别:
      其中resolver指令中的vailed是catche中rn节点的有效期时间,dynamic_resolve中的fail_timeout表示上次dns解析失败与本次是否可以使用dns解析最大时间间隔(即DNS服务无法使用的时间).

    8.当其resolver后配置了多个server的时候(即有多个dns的时候)
       nginx 会采用Round Robin的方式轮流查询各个dns server。在方法ngx_resolver_send_query中通过在每次调用时改变last_connection来轮流使用不同的dns server进行查询.

    9.dns本地缓存数据结构以及架构是如何设计的:
       函数ngx_resolver_create()对其进行创建:
       对查询结果的缓存，采用Red Black Tree的数据结构，以要查询名字的Hash作为Key,其中有可能不同域名的hash的值相同,此时则比较域名的字符串大小插入rbtree中,当然最后查询的时候也是按照同样的规则进行查找,节点信息存放在 struct ngx_resolver_node_t中。
       因为resolver是全局的，与任何一个connection都无关，所有需要放在一个随时都可以取到的地方，如 ngx_mail_core_srv_conf_t结构体上，在使用时从当前session找到ngx_mail_core_srv_conf_t，然后找到resolver。

       为不阻塞当前线程，Nginx采用了异步的方式进行域名查询。整个查询过程主要分为三个步骤，这点在各种异步处理时都是一样的：

         (1).准备函数调用需要的信息，并设置回调方法
         (2).调用函数
         (3).处理结束后回调方法被调用


I suppose that in your project `__cplusplus` is alwayes defined


This is nice to have, but it's unnecessary for our purposes. If the deque is empty, we should return null to the client.


Again, this is superfluous


Yes they belong to Gradient.
I start to implement Gradient directly on EfxFixture. When I move the code on Gradient I forgot to remove.


External Cocos Helper Android Frameworks/Frameworks/AdColony/AdColonyAds.java

problem with tabs


No `null` check


too long line


In this world java methods starts with a lower char


Since TourGuide consist of 3 components, Overlay, ToolTip and a Pointer, we should also check for the visibility of the other 2 components.


Yes.


We should not be creating a new process in this method. We are simply picking an already existing Process instance and returning it. All Process instances should be created in the constructor using numProcesses.


External Cocos Helper Android Frameworks/src/sonar/systems/framework/SonarFrameworkFunctions.java

Code repetition. Possible to create a function for this and previous cases


Note that `instance` is undefined here for declarative modules, so can't be assigned to in this case. So the fix may need to be checked here.


er.. I think this should be fixed in docker, not provisioner.

https://github.com/docker/docker/issues/5382


in_addr is not a list, in fact it is just one unsigned int:
```C
typedef uint32_t in_addr_t;
struct in_addr
  {
    in_addr_t s_addr;
  };
```
the cast to `(struct in_addr **)he->h_addr_list` seems weird to me as well, because `he` is in fact this:
```C
/* Description of data base entry for a single host.  */
struct hostent
{
  char *h_name;			/* Official name of host.  */
  char **h_aliases;		/* Alias list.  */
  int h_addrtype;		/* Host address type.  */
  int h_length;			/* Length of address.  */
  char **h_addr_list;		/* List of addresses from name server.  */
#ifdef __USE_MISC
# define	h_addr	h_addr_list[0] /* Address, for backward compatibility.*/
#endif
};
```
so addr_list should be a `char**`


:-) That is OK. It is temp fix for the docker machine with ECS driver. And the Docker machine will provide the new plugin model, so it is OK to deliver the temp solution in our branch at this time.


Add new line


Nice catch, there's a typo, should be `{ index, element -> element.index }`


@XhmikosR: Do you think I should revert this? We stopped supporting ICL quite a long time ago and ICL 15 has been out for more than 6 months now.


External Cocos Helper Android Frameworks/src/sonar/systems/framework/SonarFrameworkActivity.java


This have been added in a separated commit


@mosra wrote:
> Huh. First time I see target_include_directories() being used other than in "best CMake practices" blog posts :) It's weird that Eclipse can't handle the former, it's been there for ages.

@Squareys wrote:
> Well, it's the eclipse project generator by cmake, eclipse it self does not support cmake at all :P


it should be `autoload :AlgoliaJob, 'algoliasearch/algolia_job'` because if module called `AlgoliaSearch` autoload will look for `algolia_search/algolia_job.rb`


In our case we have deal with consequent flow of actions: for example, there is no sense to download translation without creating it. Seems that for cases like that (but I use priority for convenience) Test Dependency functionality was added to TestNg. Correct me if I'm wrong.


ok


We always copied the data into the response struct before encoding.
So there should be no performance hit. But the datasource is simpler.

For zero-copy performance, we need a datasource_release function and we need to delay the release until after the encoding. Maybe if we collect post-encoding UA_Jobs during the service...


융합인재교육 강의계획서 생성


and `mapIndexed`  -> `map`


This comment is useless, the next line is obvious


Updated.


I know I've done the same in some of my classes, but that doesn't make it any better, I should go and if those :)


Hi David, I want to also add description and geoPointTypeEnumId to OFBiz. I just did not find any geoPointTypeEnumId demo data. What do you plan to use for as geoPointTypes? What is it used for?

Thanks!


and `params.indices.toList()` looks better for me


should probably be `for(i=0; i < he->h_length; i++) {`


Done


probably need to free some stuff before exiting


just like that!


Please remove this comment. It's OK to copy from Project, but don't also copy things you don't need.

You don't need to override `updateGeometry` at all.


You also don't need this one.


Did you create this image, or did you get it from somewhere?


issue #3


done


This refactoring has introduced and `optparse` conflict.

	optparse.OptionConflictError: option --database: conflicting option string(s): --database

I've bisected from `v1.5.2` to `master` (0b15c16) and this is the first occurrence.


@three-amigos/owners FYI apparently project.pbxproj is part of the build system, bit like pom.xml, so it needs to be under git


"Modify installation" or "Change installation" may be more clear in the UI.


Thanks alot! Huge performance boost.


I agree I am going to change this, thanks @weepy


Забыл как-то )


tengine  动态域名解析模块的分析


@mkreisl

1) have you recently tried building kodi addons ?
all do go clone from git.

2) normally root rights are not needed. either you setup schroot as $xbiangit expects it (normal user who ha defined root rights inside build root) - or - you don't care security and run * with root rights.

and yes, (if) root is needed then only to create the symlink.
I wanted to inject another git via PATH=, but this stuff is put into Makefiles/CMakefiles during configuration and it wasn't fully reliable - mayba I crack it later.

if you have better idea, go for it. really. this problem with git (and for instance gettext, also java) is not random issue with one package. it is systematic qemu problem for which we don't have other solution for now.


Eiei, it's about `this string strg`, NOT about `this byte[] bytes`....


dom-if => `dom-if`


Discordo* na 1 e na 3


just use kill() is fine


start + stop = reload


finish #15


This was a bug caused by nrc/strings.rs#3.


Hi Erik, sorry for the delay in this response.

It's strange that the files have different SHA-1s, since I don't recall
changing them between those commits. The files are (should be) the same
since I have installed them.

Here's what I did to be precise:
- Forked and Cloned your repository locally
- Updated the files with their latest versions
- Committed the changes
- Added the NDK 3.9 files locally
- Committed the changes
- Updated the posix files locally
- Committed the changes
- Added Roadshow locally
- Committed the changes

after you mentioned that the lib files were excluded, I checked and found
that *.lib files were in my .git "exclude list", so I removed that and
re-committed the changes. That's where you have the latest commit from, no
changes in the files themselves occurred as I recall.

In any case, the latest "version" of them is what we should keep.

On Wed, Jul 29, 2015 at 10:47 AM Erik Faye-Lund <notifications@github.com>
wrote:

> No, they are not the same version, they have different SHA-1s. Where did
> the different binaries come from?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/midwan/amiga-dev/commit/1e4e683520b71961d8a72a642f8f194e8951d045#commitcomment-12415083>
> .
>


Line no:18

Exception Type:	IntegrityError
Exception Value:	wagtailcore_page.depth may not be NULL


These files are used and needed by tests (See https://travis-ci.org/ekylibre/ekylibre/jobs/73677489 for example). I'll have to revert this commit.


Yes. Installs again without a hitch.


hmm, I see what you mean.

Yes, roadshow is an opt-in feature itself, that's why it's also included in
a different folder. There's no chance that someone would use it by mistake,
since you'd have to explicitly add it in the INCLUDE paths (and you would
only need it if your project references it anyway).

I'm OK either way with this one, we could leave it out and have people only
download and use it from the official source.

On Wed, Jul 29, 2015 at 4:31 PM Erik Faye-Lund <notifications@github.com>
wrote:

> (Note that my worry is mitigated a bit by the fact that Roadshow is an
> opt-in feature)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/midwan/amiga-dev/commit/ee583d28e679144ea04e675001a7673256a5ed8a#commitcomment-12421770>
> .
>


@joneskoo Thanks for the comment. Do you have any references indicating when the `-i` option was added?


Do not commit directly to master, create pull requests.


Even for this small stuff?


@NgocLong tui đã update xong cái template rồi, Long có thể sync về máy rồi thực hiện những thứ Long làm rồi chiều, làm cho những câu query đó hiện ra trên trang catalogue.


This is the "old form" of handling this. We now use "ENABLED" rather than "defined".
Also, "HAS_*" properties will need to always be defined.

Let @thinkyhead show us the proper way to handle it.


Related commits:
https://github.com/OpenViX/enigma2/commit/61efc03ad188d78c0493002bd4ebd8a38b18af7e
https://github.com/OpenViX/vix-core/commit/29a80264d82b3c81c83219dfa9c7feb5f2e63323


@mk01
> 1) have you recently tried building kodi addons ?
> all do go clone from git.

Sure, no problem

> with one package. it is systematic qemu problem for which we don't have other solution for now.

Ok, that is an explanation why I never had issues building packages, because I don't use qemu, I build packages on a native armv7 system

Btw, I  built and installed xbmc-package-xbmc ( rpi-isengard ) sucessfully


@alexmarsev: I was talking about this some days ago. The idea was to know what we should do with those:
1. Keep the global declarations, remove the locally duplicated constants.
2. Move everything inside the functions.
3. Something else, like using function wrappers?


No need to create separate pull request for this small change, you can include this commit in you feature branch and then create pull request for you feature together with this small change.

Generally nothing should go to master directly except pull request merges. Even small change have to be reviewed, small changes brakes things too.


https://github.com/Mei152/address-bloc/commit/287575232b74dfe2da8e6093c86a59421f3473ad

https://github.com/Mei152/address-bloc/commit/9cc419f27629c860b979fbae8ba442fbee664f82

Both of these commits have been merged into master.


``static __m128`` doesn't make any sense to me, I always thought ``__m128`` implies register usage.
Does MSVC produce a warning here?


finish #15 this time.


@ivandrofly
`text.StartsWith("{\\an1}")` ==> `text.StartsWith("{\\an1}", StringComparison.Ordinal)` ?


Yeah that's why I find this weird too but I don't see any warnings and as far as I can tell it works...


Also try to keep you pull requests small, do not include too many changes in your pull request. Always try to break your feature into smaller features that work.


I got it, these makes our life one millon time easier. Good job


natively. ...
then update common.functions and the check for git please (for native building, so no wrapper). currently this is already case of CROSS compiling.


@mpc-hc/developers: https://github.com/mpc-hc/mpc-hc/compare/develop...Underground78:7d77e08 this is everything I have found in the new VS15 warnings. Some should probably be committed before the VS15 branch is merged. The last commit could be ignored completely or some parts of it.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (353 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


science!


Isn't compiler clever about intrinsics? It is just higher level of writing things, compiler is likely to figure this thing out and those, `static __m128` are inlined?


Outch, that's insecure :-) Check the comments on the StackOverflow answer.


Signed-off-by: dougiefresh douglas.orend@comcast.net


Excelente.


Before duplicating this values everywhere, we should probably check the generated binary if it will make any difference.


I've already updated common.functions before I build rpi-isendgard


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (353 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


We should start checking if `Value` is null to avoid the Server from crashing in unexcepted ways


Did you forget to add "qcom,wakeup-on-idle;" to the dtsi file, or was that on purpose?


Yes, though it's not const of  in theory the value of `zero` could be changed for example.


Might want to fully qualify the `mono` to `HOMEBREW_PREFIX/opt/mono/bin/mono`


FCVAR_PLUGIN has been deprecated since SourceMod 1.8
https://github.com/alliedmodders/sourcemod/pull/344


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (353 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


then push ;)


AddCalcCoilSurfTempSource (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1772 of 1772 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (353 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Yeah, I'm too used to working with Immutable or Lodash where a result is always returned.


For example, If I sent a reconnect command (To simulate a client disconnection) while transferring files, the server will hang


SOKP :/


Shamu :/


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (353 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


That's on purpose... Need to test that.
Currently, just synced parts with CAF.


Woops I thought I had changed all the defines for ENABLED.

I'll change the and re push.

Enviado desde mi iPhone

> El 1/8/2015, a las 15:21, Richard Wackerbarth <notifications@github.com> escribió:
>
> This is the "old form" of handling this. We now use "ENABLED" rather than "defined".
> Also, "HAS_*" properties will need to always be defined.
>
> Let @thinkyhead show us the proper way to handle it.
>
> —
> Reply to this email directly or view it on GitHub.
>


Alright, thx for your fast reply (=


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (353 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


https://github.com/Mei152/address-bloc/commit/62e682e2052df5d8f65701dc29e6958ee7cdfb7e

Done.


The statement below was copied from 7ae1396d31c7f9e157b6508d3dcae1429da09618 at line 69. Probably it's the change mentioned in reply. To be honest, I have no idea if any better way exists as well. But  it's better than the old code in at least two ways :

 * Better performance. The method `ContainsKey` takes around O(1) rather than O(n). See : [ContainsKey Method from MSDN, (also note the Remark section)](https://msdn.microsoft.com/en-us/library/kw5aaea4.aspx). As a result, the new code takes only O(1) + O(1) => 2 x O(1) => O(1) time to run.

 * Better exception safety. Note that the old code may raise the same exception as well. It happens when the `List.Find` method finds nothing and thus returns a `null`. I appended a simplified code trying to present such case.
```c#
//
//From line 69:
//
return discounts.ContainsKey(barcode) ? discounts[barcode] : null;
```

```c#
//
// Simplified version, which crashes at run time and raise a NullReferenceException.
//
using System;
using System.Collections.Generic;

namespace TestCsharp
{
    public class Product
    {
        public Product()
        {
            this.Barcode = "000";
            this.Name = "someName";
            this.Price = 42;
        }

        public string Barcode;
        public string Name;
        public int Price;
    }

    class Program
    {
        static void Main(string[] args)
        {
            var data = new List<Product> { };
            var result = data.Find(p => p.Barcode == "000");
            System.Console.WriteLine(result.Price);
        }
    }
}
```

Perhaps more elegant way could be found later on. If so, just let me know.
This implementation looks quite expressive. Great read. :smile:


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (353 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Haha it is, I just noticed a DS_Store file in my folder and I thought it had come from getting fit, but it hadn't. 😆


great stuff! Just a little comment... Maybe it's an idea to always check for the special sync endpoint and try to use it ? If it doesn't exist, switch to the fullsync ? So, no user choice, just by default ? If the user has the serversid eplugin installed they benefit from it ?

Or is this a really bad idea ? :-)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (353 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I've nothing to push, I'm using your xbian version and can build packages w/o any issues.


up


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (352 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.72%25-yellow.svg)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


I see things like `movdqa      xmm2,xmmword ptr [one (01402519B0h)]` so as far as I can tell it's loading the constant from memory.


Potentially yeah, I don't really see the benefit, but others might, for now just trying to get it working.

Shaun
________________________________
From: Marcel van der Veldt<mailto:notifications@github.com>
Sent: 1/08/2015 11:47 PM
To: MediaBrowser/Emby.Kodi<mailto:Emby.Kodi@noreply.github.com>
Cc: Shaun<mailto:shaun@bluebit.com.au>
Subject: Re: [Emby.Kodi] first cut at using the new server plugin to do start up sync (9903b8d)

great stuff! Just a little comment... Maybe it's an idea to always check for the special sync endpoint and try to use it ? If it doesn't exist, switch to the fullsync ? So, no user choice, just by default ? If the user has the serversid eplugin installed they benefit from it ?

Or is this a really bad idea ? :-)

---
Reply to this email directly or view it on GitHub:
https://github.com/MediaBrowser/Emby.Kodi/commit/9903b8d53db505a5e5f95bd36d5782ecc52eb035#commitcomment-12479478


@d3agle Thanks, fixed it.


Hey Felix,
I created a class to handle the creation of account transactions and the subsequent updating of the account balance so that the AcctTransaction model would not have to worry about updating the accounts balance. Let me know how this looks and if anything needs changing.

Regards,
Blayne


If this was a node module I'd probably split things differently, but for a website or a project like this I usually include a lot more stuff in dependencies.

My thought is for a site/project like this if you have a copy of the repo you're going to be doing either building the production assets, or you're going to be doing active development. So I split things up based on those two tasks. All the loaders are needed for building assets. The only things I put into dev are test/lint tools and development helpers like HMR.

Like I said, if this was a npm package, you'd do things very differently. But it's not, it's a website and it won't be distributed via npm, or required via node.


This won't work unfortunately, I'm not really sure why.  It will evaluate both conditions, and checking for FrmFm if its null will still crash the server


Why only in a loop?


develop (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (352 of 353 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.72%25-yellow.svg)


Its AppZomoss world ... world of possibilities ... towards light =====>>>>


Just a start......Buddies :)


Shouldn't this be `[]BrowseNode` instead of `*BrowseNode`?


We would have to evaluate in its own condition if `Value` is null instead


Looks like the naming convention is prefixing the response structs with `Amazon`.


you are right, let's try also on the nested tags. Initially I thought that this was handy only in the loops, but it could work also for nested custom tags. But then we need a test for it


These essentially do raw text replacement in your JS. So during build it will actually search the files for all instances for process.env and __DEV__ and replace them with the object value.

So for __DEV__ it'll actuall swap instances of if (__DEV__) with if (false) which Uglifier can easily know that it impossible to execute, so it's considered dead code and is removed.


It would preserve 1 and 3 because that is the OR of 1, 3, and none!


@d3agle according to [this MSDN page](https://msdn.microsoft.com/en-us/library/6373h346.aspx): `If the first operand evaluates to true, the second operand isn't evaluated.` So it should work.


Oh woops thank you!  I should have known this by now LOL


@d3agle No problem, we all learn everyday. :)


haha, okay :-)
Thanks for looking into this. It sounds very promising.
Let me know when you're ready to test


It is ready now.

Shaun
________________________________
From: Marcel van der Veldt<mailto:notifications@github.com>
Sent: 2/08/2015 12:01 AM
To: MediaBrowser/Emby.Kodi<mailto:Emby.Kodi@noreply.github.com>
Cc: Shaun<mailto:shaun@bluebit.com.au>
Subject: Re: [Emby.Kodi] first cut at using the new server plugin to do start up sync (9903b8d)

haha, okay :-)
Thanks for looking into this. It sounds very promising.
Let me know when you're ready to test

---
Reply to this email directly or view it on GitHub:
https://github.com/MediaBrowser/Emby.Kodi/commit/9903b8d53db505a5e5f95bd36d5782ecc52eb035#commitcomment-12479530


Hey, you didn't update the minified CSS, rdash.min.css!


Wow.  I should've looked at this submission closer.  Didn't notice the old constructor


This currently (Aug1,2015) does not work, as Namecheap requires HTTPS. See:
https://www.namecheap.com/support/knowledgebase/article.aspx/29


@mosra Done.


if Rails.env.development? || Rails.env.production?


Did you forget to update the branch? The new test has exactly the same diff as the old one.


Cairo.jl was. That's what I thought at first too but then I realized that
that's not what you want. This represents an io device abstraction, do it
should return the number of bytes written to the abstraction. The
underlying stream is irrelevant. A good comparison is a TLS stream. You
don't care how many bytes are sent over the network, what you care about is
how many bytes were encrypted.
On Aug 1, 2015 12:49 AM, "Jameson Nash" <notifications@github.com> wrote:

> who was checking? also, shouldn't this be reflecting the notion that write
> may be putting more / less bytes into the underlying stream than would be
> implied by taking the sizeof the input?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaLang/julia/commit/69acc2d65158e9838d3f79af8cac5028a817b75e#commitcomment-12478183>
> .
>


Yeah, will do. Will also check for `-a`. And sorry, been away for a week


Ehhh, the names don't bug me. It's this actual certs that would be the issue.


My opinion:
 - zero should be local to every function
 - the rest are compound intrinsics, so aligned memory load shouldn't be that bad,
if ``static`` works - why not (shrugs), but maybe add ``const``


ah, now it makes sense, one is to reduce the size of React and the other to be able to introduce development code only which gets removed on production build. Nifty!


Now I'll switch my Minecraft name to "lobby" :D
Nah, just kidding~

Thanks for this commit, I was hoping this would be done someday ^^


@twizmwazin ^


Sounds reasonable to me.


Indeed, I forgot.


I think 9ae246d6 fixes this issue (in a general way)


You're totally right. Probably why it's gagging on heroku :)


Nope, not anymore.  At first I wasn't using it in the exception handler in FrontLineWorkerImportController.java.   However I was getting an exception.  I thought that was because I was returning a string instead of an object, so I created this class.  However the exception was because of a mismatch between the exception class in the annotation and the method argument.

tl;dr I'll remove this class.


Отлично!


相比在后台打印文件变化，使用grunt-contrib-watch + grunt-contrib-connect + grunt-livereload 实现前端无刷新更实用，是否可以实现一下？


Just a notice:

The annotation results look like this:

```
--- missing/usr/sbin/anaconda
+++ covered/usr/sbin/anaconda
+ #!/usr/bin/python
+ #
+ # anaconda: The Red Hat Linux Installation program
+ #
+ # Copyright (C) 1999-2014
+ # Red Hat, Inc.  All rights reserved.
+ #
+ # This program is free software; you can redistribute it and/or modify
+ # it under the terms of the GNU General Public License as published by
+ # the Free Software Foundation; either version 2 of the License, or
+ # (at your option) any later version.
+ #
+ # This program is distributed in the hope that it will be useful,
+ # but WITHOUT ANY WARRANTY; without even the implied warranty of
+ # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ # GNU General Public License for more details.
+ #
+ # You should have received a copy of the GNU General Public License
+ # along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ #
+ # Author(s): Brent Fox <bfox@redhat.com>
+ #            Mike Fulbright <msf@redhat.com>
+ #            Jakub Jelinek <jakub@redhat.com>
+ #            Jeremy Katz <katzj@redhat.com>
+ #            Chris Lumens <clumens@redhat.com>
+ #            Paul Nasrat <pnasrat@redhat.com>
+ #            Erik Troan <ewt@rpath.com>
+ #            Matt Wilson <msw@rpath.com>
+ #
+
+ # This toplevel file is a little messy at the moment...
+
- coverage = None
+
- proc_cmdline = open("/proc/cmdline", "r").read()
- proc_cmdline = proc_cmdline.split()
- if ("debug=1" in proc_cmdline) or ("debug" in proc_cmdline):
-     import coverage
-     cov = coverage.coverage(data_file="/mnt/sysimage/root/anaconda.coverage",
-                             branch=True,
-                             source=["/usr/sbin/anaconda", "pyanaconda"]
-                             )
-     cov.start()
+
+
- import atexit, sys, os, time, subprocess
+ # keep up with process ID of the window manager if we start it
- wm_pid = None
+
- def exitHandler(rebootData, storage, exitCode=None):
+     # stop and save coverage here b/c later the file system may be unavailable
-     if coverage is not None:
-         cov.stop()
+         if os.path.isdir('/mnt/sysimage/root'):
-             cov.save()
+
-     if exitCode:
-         anaconda.intf.shutdown()
+
-     if "nokill" in flags.cmdline:
-         iutil.vtActivate(1)
-         print "anaconda halting due to nokill flag."
-         print "The system will be rebooted when you press Ctrl-Alt-Delete."
-         while True:
-             time.sleep(10000)
+
```

A diff between two annotated sources looks like this. In this particular case a graphical vs. text mode installation of Fedora:

```
--- /usr/sbin/anaconda
+++ /usr/sbin/anaconda
@@ -106,21 +106,21 @@
 -         except BaseException as e:
 +             # catch all possible exceptions
 -             log.error("Problems running the window manager: %s", str(e))
-+             os._exit(1)
-+
-+         log.info("The window manager has terminated.")
+-             os._exit(1)
++
+-         log.info("The window manager has terminated.")
 -         os._exit(0)
 -     return childpid
 +
 - def startAuditDaemon():
 -     childpid = os.fork()
 -     if not childpid:
-+         cmd = '/sbin/auditd'
+-         cmd = '/sbin/auditd'
 -         try:
 -             os.execl(cmd, cmd)
-+         except OSError as e:
-+             log.error("Error running the audit daemon: %s", str(e))
-+         os._exit(0)
+-         except OSError as e:
+-             log.error("Error running the audit daemon: %s", str(e))
+-         os._exit(0)
 +     # auditd will turn into a daemon so catch the immediate child pid now:
 -     os.waitpid(childpid, 0)
 +

***** SKIP *****

@@ -654,8 +654,8 @@
 +
 -     if anaconda.displayMode == 't' and graphical_failed and \
 -          flags.vncquestion and not anaconda.ksdata.vnc.enabled:
-+         app = App("VNC Question")
-+         spoke = AskVNCSpoke(app, anaconda.ksdata)
+-         app = App("VNC Question")
+-         spoke = AskVNCSpoke(app, anaconda.ksdata)
 -         app.schedule_screen(spoke)
 +         app.run()
 +
```


It will throw out an error, but /system will be writable.


What is the purpose of these `full` statements? Are there cases where the output is not a dense matrix?


Fair enough. I had them split up according to where they are going to be used. I guess, ultimately it doesn't really matter as long you have all the tools to work with / you rely on.

However, I liked [that example here](http://survivejs.com/webpack_react/building_kanban/#-process-env-node_env-):
```js
entry: {
      app: path.resolve(ROOT_PATH, 'app/main'),
      vendor: Object.keys(pkg.dependencies)
    },
```

Where you treat everything outside of devDependencies as `vendor` code. Obviously this would be a more lazy way of defining the vendor plugins. It could potentially be better to be explicit with the vendors instead of a that blanket statement.


Yeah, or LLE audio w/ gfx in default RSP 1.7 plugin also works.


I'm not really sure about it because I just started looking at the code again. But: `openDocument()` seems to be used exclusively in `viewers.AbstractViewPanel.createWidget()`, that is upon creation of the widget when there is a "current document" in the document chooser.

I'm wondering if we can't remove or modify that method altogether and change it to something more consistent.


It doesn't try to listen on [::]:8443 here, which was the default with the old packageing.


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (746 of 746 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-7.08%25-green.svg)


I did not find Java jar file insde


@poying 你確定這裡原本不是寫錯的？
@czchen 你怎麼看？


No, this is just a way to know that RepositoryScanInterval has not yet been read and ticker is not set yet. It will be read for the first time around line 161.


No, I used it to covert from `Hermitian` or `symmetric` to `Matrix`


是說有時候遇到問題可以去看 pcman-firefox 那邊的比對一下，因為這裡的是從 pcman-chrome 多過了一手來的。


It's a cute idea, though I don't see the value in sending vendor files down separately. That doesn't gain the end-user very much. the user doesn't care what is vendor code and what isn't, only the developer does. Instead, I like to use the common chunks plugin and send down packages and files that are common across your different entry points.

Really, dev vs normal is purely for npm packages, on a website like this, they could all just live in dependencies because it doesn't affect anything or anyone really. The reason why I go with this pattern though is when you add server-side rendering, then you get to keep your test utils off your server.


Embarrassingly, it was a misguided solution. As you noticed in your fork, this rule caught a couple of "/*global" config comments. I had an expectation that it would be ignored with a space. I realize now this was baggage from an older, stricter linter. Thanks for prompting me to review this change. The rule has been reinstated (as `spaced-comment`, of course).


GengoClient.setBaseUrl() method and the baseurl variable may be revive and GengoClient.getBaseUrl() method may be upgraded.


Webpack-dev-server could go in dev dependencies though. 😦


AL-1 updated name of project.


LGTM


One issue with this modification is that the method name is never reset when exiting the scope of the class. For example, if I configure the plugin to remove `demo.unicredit.Foo2.hello` and I have the following source:
```scala
package demo.unicredit
class Foo2() {
  def hello() = "hi"
}
class Foo3() {
  def hello() = "hi there"
}
object Demo2 extends App {
  val foo = new Foo3()
  println(foo.hello()+" planet")
}
```
then the plugin currently also removes `demo.unicredit.Foo3.hello`. That's because at the point when it traverses class `Foo3` the name has been set to `hello`, and so it will match `hello` in `Foo3` as well.

To solve this issue, I'd suggest to have the plugin components run after the `namer` phase, and to find matching methods using method symbols. Otherwise, implementing correct matching would likely amount to re-implementing Scala's scoping rules.


I considered someting along the lines of
```
    _curPerfHudMode++;
    if(_curPerfHudMode > 2) {
        _curPerfHudMode = PerformanceHudMode::Off;
    }
```
but decided against for 1. readability (it's and example after all) and 2. staying independent of the implementation of that enum.


@arunk-s , is this check needed?


I think we should, yeah.


Ouch :/


"Silence"


覺得有問題的都先發 issue，等到要補 unit test 時再看看行為對不對。


貌似删了之前做的一堆东西= =还有些引用还没删来着...


ugh. I got worried when I saw this message in my github inbox. But eh, potato patato :)


Unsatisfied requirement doesn't mean uninstallable, right?


@martincostello I've reverted my PR with https://github.com/mono/mono/pull/1957 and verified your project builds correctly again (plus the xunit 2.0 tests work as well :smile:)


No need to create a template string.
Just use `'` for the string.


cherry-picked


@arunk-s , you can avoid duplicated code here.
## meaningful comments about GET vs POST
skeleton_code += gen_proxy_code(details_dics) + generate_request_code(method, url, headers, searchString)
if meth == "POST":
       skeleton_code += generate_body_code(details_dict['data'])


Ah oui, bien vu.


ES6 never use var, use let.


I haven't really tried but I think it might have worked anyway.


Ok, will do.


Excellent, thanks @akoeplinger!


Good catch!


Thanks Ryan - looks like https://github.com/ryancramerdesign/ProcessWire/commit/8318b7dee75aec759f24bd65628ac795485bf293 has fixed dependencies.


External Cocos Helper Android Frameworks/Frameworks/ChartBoost/ChartBoostAds.java

Can you read this ? )


```cpp
_curPerfHudMode = (_curPerfHudMode + 1) % 2;
```

Rather not, it's perfectly okay and readable the way it is now :)


@danandrei this fixes the `basePath` of dynamically loaded `ace` scripts. Maybe you could create a "default" client config (ex. `composition.client.config.basePath: "..."`) in the `engine-editor` package to improve maintainability.


Will add that.


I think I do this typo every time. ^^'


최상위 디렉터리에서 cmake로 최종 실행 바이너리 apus를 빌드하는 것으로 이해하고 있었습니다. 이 부분을 @enghqii 팀원이 해줄 것으로 생각해서 삭제해두었는데요. 다시 복원시켜서 새로 커밋을 추가하겠습니다


조언 감사합니다 SignedExtension함수는 지우겠습니다.


Oh, right. I should test for unstatisfiable requirements then, like `MaximumMacOSRequirement`.


To avoid the `: _*` you could also use `(try ... catch { ... }).to[mutable.Set]`, like this:
```scala
(try new String(readAllBytes(get("./method_eraser.config"))).split("\n").toSeq
 catch {
   case err: Throwable =>
     println("Method eraser configuration file is missing")
     Seq()
 }).to[mutable.Set]
```


Like


When I discovered this repository, I almost had a heart attack when I saw the r-word in 40px Helvetica on my monitor. Then my PTSD kicked in. Having served in the Iraq war, I immediately began to recall horrific images in my mind, seeing fellow soldiers' limbs being blown off and intestines spilling out their guts. Fortunately, I was visiting my dear old mother, who saw me rolling naked around in my own bodily waste like I was having a seizure and promptly ensured I was taken to a hospital. After I recovered, I was told I have two weeks left to live.

I will likely be dead in a week or so, but I hope my story can serve as an inspiration to others. The r-word is extremely harmful—it is putting an end to my life, and will continue to endanger and takes the lives of millions of others. GitHub staff is doing the right by attempting to quell the ableist, misogynist & racist behaviour that runs rampant on this site, and I sincerely hope they one day succeed.

Yours truly,
Frank


I think we should delete these files if we're no using them


合成の巻物が手に入ってしまう #636
HP増加の指輪の説明文が間違っている #632


Good catch, thanks.


have you thought about using a lazy method for this instead of using `before_action`?
you can use something like:
```ruby
def curriculum
  @curriculum ||= Curriculum.find(params[:id])
end
helper_method :curriculum # <--- this makes the bare 'curriculum' available in the view
```


Yep, thanks i got your point. I'll use exceptions only in exceptional
situations.

2015-07-29 0:40 GMT+03:00 Maxim Kot <notifications@github.com>:

> strange name of error code proves my words ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/libproject/libproject/commit/f62b59a3bfbe13ba4065de99e1401c1b67caecdc#commitcomment-12407315>
> .
>


So this fixes it?


You probably overlooked line 16-20 of parser.y:

```
/*
Removed until we know more about the min versions of Bison and Yacc required for this
to work, see Byacc man page: http://invisible-island.net/byacc/manpage/yacc.html
%expect 6
*/
```


Honestly the formatting looks entirely broken here so I don't know what is up with that but I would nak this change on that alone.


Also use ",ok" to check it us a []byte to avoid a panic  here.


sorry i dont see the link here


done


U told me that!!!!bu kao pu!!!!


sdsd


Should be "White_Number_File"


@DDoS

May as well as do a Transform sub(Transform other).


@arunk-s , why not scheleton_code += stuff
before the whole checking (to avoid duplicated code)


you missed a whitespace between ',' and "item"


polyphase merge, replacement selection


As just commented at #19:

@property in the docblock is meant to make "magic" properties visible in the docs, not to hide existing properties in a class.
http://manual.phpdoc.org/HTMLSmartyConverter/HandS/phpDocumentor/tutorial_tags.property.pkg.html


넵 부족한 부분에 있어서 조언 감사드립니다.
3. BinaryReader 부분은 제 생각에 하나의 feature라고 생각이 되어서 일단, 1번 2번으로 커밋을 쪼개서 PR을 새로 날려서 머지가 된 후에 다시 브런치를 새로 따서 만드는 편이 나은거 같습니다 (현재 로컬에서도 그렇게 진행하고 있긴 합니다)
수정할 때 말씀해주신 부분 참고해서 제대로 작성해보겠습니다

늘 도와주셔서 감사합니다