Just happened to browse through this but it looks like the two cases that 'id' is used in the if-clause should be broken since it's undefined.


Yes (the rooms also use the tile). Concerning the tile, it makes sense IMHO because GameEntity is, by design, supposed to be on several tiles. Moreover, with the old code, when a creature has partial vision on a room, it would engage even if seeing only 1 destroyed tile. That is not the case anymore.

Concernning the seat, it is obviously needed for traps but it may be also be of use for later spells...


Maybe better to make the appropriate constructor?


what is the difference between old version and updated version? means I have used it before july 2014 after you made some changes in sdk.
At that time I have the jar - 1.1.2 but now it is jar 1.15. These are the changes I am able to find at the same time, You have made changes for gradle and android studio support.

Please let me know about updates in android side also, as in iphone there are some changes with UI provided by you ( means validating the card and like that... )

Thanks
Vinod Sakala


funny?


Updated jacobians and mass matrix
@traversaro @DanielePucci


台中您好


Are you sure the current ocaml-version of the trunk of camlp4 is 4.02.1? It seems that it follows ocaml/trunk more closely (74ecf930b1fe6c1bc6f146d463b6c0c3b4c7af37).

I was trying to test 4.03.0+pr132 and the program to test recursively depend on camlp4.

```
# opam pin --switch=4.03.0+pr132 add camlp4 git@github.com:ocaml/camlp4.git

[camlp4] git@github.com:ocaml/camlp4.git updated
Installing new package description for camlp4 from git@github.com:ocaml/camlp4.git/opam

camlp4 needs to be installed.
[ERROR] camlp4.4.03.0 is not available because your system doesn't comply with !preinstalled & ocaml-version = "4.02.1".
[NOTE] Pinning command successful, but your installed packages may be out of sync.
```


Thanks for the feedback, could you provide some more detail?
Is there an app that triggers this? How soon are we talking about?

Also would you be so kind as to open an issue over at http://jira.cyanogenmod.org so that we can track this?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/9822/testReport/) for this commit**


The same is needed for `2.1-stable`. The latest copying is commented out but with a wrong comment & should be removed.


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


ouch because
1. password was stored public in a repository
2. mysql connections on old server was allowed for all users

also there was no "firewall" rule which limited this to a single ip. i only remember that i was heavily blamed when i disallowed connections to the mysql server from remote hosts.


What should I change to, CFG_CRYPTO_SHA256_ARM32_CE?


Right, of course :see_no_evil:


Ah indeed, I missed this, thanks. I'll just remove the constraint in the local opam file.


Found BROKEN by full.


Maybe ```<=```?


Thanks.  Fixed.


hello, i try to use the naxsi to my project, can you give some examples?  thanks a lot.


delete


use to Django and the others projects.  configure is different?


YOLO


...can change __from__ release to release...


Heh. So the solution we talked about of doing the pulls ahead of time.. We actually would've masked this issue instead.


does it work?
see
https://github.com/Singular/Sources/issues/684


Less should be moved to next to sass


Is this console.log in here deliberately? I assume if so it's to make sure the stack trace ends up in the log file.


less nearer to sass


Spacing issue here


Let's use `getRandomValues` to make nonces unpredictable here.


This stream needs to be returned by the function


Oh right, I can't count apparently ^^
I was mainly replacing Skip with Get, would the patch be ok with the proper number of Get?


This shouldn't be inside gulpif


These should be aligned with the ones above


Hi Ben

Greetings from Paris!

>> https://github.com/va3c/viewer/tree/BH_r9_Minimal/va3c-viewer-html5/r9

Nice one!

I have had a quick look at the work using HTML Preview (which anybody can
use):

http://htmlpreview.github.io/

I was not able to open a file with the preview, but at least I got an idea
about its operation.

The interface seems clear and easy to use.

Also I see that you have headed in the direction of the Mr.doob Coding
Style ( an abundance of white space ). Yay, I think.It's not what I grew up
with, but keeping with his style makes it much easier to move from his code
to yours.

Some thoughts:

If you want us peeps really to play with the code, you might consider
publishing it to the gh-pages branch

You might also consider starting a new repo or creating a new folder for
the work. The work is now in the 'va3c-viewer-html5' folder. This name was
created so that other folders such as 'va3-viewer-dat.gui' could be created
and we could have multiple viewer types developing side by side. Please let
me know if I can help you set up something like this.

In any case, great start. Please do continue.

Theo

On Mon, Jan 5, 2015 at 5:02 AM, Ben Howes <notifications@github.com> wrote:

> first commit on minimal branch
>
> Pulled in the VA3C codebase we have been developing in the CORE studio
> for internal projects.  Represents a major refactoring of the original
> VA3C code base and some cherry picking from @theo-armour 's latest work.
> Loads a model created in grasshopper.  Uses dat.gui for UI.  More to
> come...  Selection and attributes, open files by url / locally, and
> revit exporter verification next.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/va3c/viewer/commit/80db598ace5c7c442097c4fb6a5f2d4670ef7aa0
>


Getting same error  not at home to have a look for what did it

frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/phone/StatusBarHeaderView.java:572: error: cannot find symbol
        mQSPanel.vibrateTile(20);
                ^
  symbol:   method vibrateTile(int)
  location: variable mQSPanel of type QSPanel


This could be replaced with the [built-in URL parser](https://developer.mozilla.org/en-US/docs/Web/API/URL), hopefully


Alignment


It is a good workaround, but we'll get again the issue when / if I use CompressorName in the future.


This error check should go under line 117, after running sass


This should't be in a gulpif


Unless there's a matching example.html template/file I think this could be misleading


Nit: You may want to use [Template Strings](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/template_strings) for improved readability.


Please merge core-mixins.less and site-mixins.less and alter all references in all files to import just one.


Consider rename to main.less ? which matches the sass equivalent.


Hello Martin,

I have made the suggested changes in this commit, removed the xml code part, As code of this commit fix the original issue for showing the owner_id on the created quants from picking.

Thanks.


Mysql was configured to only accept login from localhost and specific windows host IP for spring back when i was setting it up.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1796](http://teamcity.monogame.net/viewLog.html?buildId=4213&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 293, ignored: 6 Build time: 00:18:24


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


@jgillis Do you the reason for the failure?


Not so sure about this. There are two issues:
1. Having OpenMW change the grid size without input from the content file might be problematic. For example Redemption would break if you do that, because several scripts depend on the 3x3 grid. I guess we can go along with it for now, but before we finalise this feature (in 1.1 or whatever) we have to go over it once more.
2. Storing the size as a settings looks problematic to me. Having even values wouldn't work so well. Of course when we add a widget for it, we could limit it to odd values. But that is unnecessary work and still makes the cfg file more fragile than it needs to be. Instead I propose to store (size-1)/2 and call it something like "exterior cell range" (yes I know I suck at naming things).


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3046
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3040
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3041
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3032
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3032
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3035
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3028
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3023
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2700
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2698
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1536
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1528


fast-forwarding master to auto = a3a16e96


@e-frank It was incorrect usage from start. You should use `$model->load($data, '')` instead. Because `$formName` described as string. Booleans does not make sence there at all.


I've copied this test in my QGIS plugin (to the test the compatibility with GDAL).
but this test doesn't produce the error, error only occurs with <SrcSql dialect="sqlite">SELECT * FROM test</SrcSql>


I was confused as to why my gulp watch was still running without errors but no routes file was being created. This was the reason :)

A shame to see the feature go but I guess it was necessary.


Yes, internal problems with windows virtual machine communication; restarted.
This is one type of false positive that i won't mind missing after migration to travis+appveyor is complete..


Is this intentional?


To pass ```int``` as constant reference is overkill.


saw approval from acrichto
at https://github.com/nikomatsakis/rust/commit/e93e613d0657ed95601a8ef8fbb59529315c4562


merging nikomatsakis/rust/carl-ice = e93e613d into auto


status: {"merge_sha": "385b380210b0388fd80cb478951a787c5a48d18a"}


nikomatsakis/rust/carl-ice = e93e613d merged ok, testing candidate = 385b3802


Aren't these deletions a bit too eager?


Add git commands used for each step.
Add missing steps  (commint modifications, push the new branch)


status: {"merge_sha": "dfba5025041594d840802423c9dbd22b65a435a7"}


nikomatsakis/rust/carl-ice = e93e613d merged ok, testing candidate = dfba5025


No sure that this is correct.
I think that the copyright should contain the date frame to which it refers. Otherwise it is like eternal.
Keep in mind that  the copyright/owner of a project can change from time to time


ditch


Consider adding a SASS version of all the less files added, so that switching between the two is easy


thx. my fault.


yup fixed - sorry, for some reason I'd only part committed my code (very irritating)


@vincentchabannes @prudhomm I have the same crash in my application (doftablempi.hpp:1339).


have sorted editor - hopefully no more alignment issues


Will move.


Add image optimisation task, separate to watch


@kaikreuzer  auto formatter seems to be useful :)
Is that something that can be installed in eclipse or something external to run?


yup - fixed.


it wasn't returned in the previous code - the sass task should return and doesn't, but the js gulp task - should that return?


Try to cut down the actual real variables provided as they're likely to get scrubbed. Instead, document with comments the ideal approach to variable namespacing.


It's incorrect. If we have already received a message with ```id = 0```, we will get it again (as ```lastKnownId  = 0```).


Remove


Remove debugger entry point `binding.pry`.


TeamCity Hyrise :: Unit Tests [Build 56](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=64&buildTypeId=Hyrise_UnitTests) is now running


What's the point of explictly adding the `private` modifier? Isn't that the default anyway?


idd_modifications (rbuffat) - Win64-Windows-7-VisualStudio-12: OK (1582 of 1582 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)


fixed in d282e40abfba7b85b4dfb984e20b4dbd5c1ed45b


Fixed in 31e0bf19d6b4fe531aa5c7288168b2b575204a37


@mhamrle pointed me to this, and as it's quite heavy task i'm going to test out if they can be done in Dockerhub automated build, it should help speed up the docker run/build locally


Fixed in 1af9c78d7c166a14a39f3563d6a278c11e9c9e29


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1797](http://teamcity.monogame.net/viewLog.html?buildId=4215&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 295, ignored: 6 Build time: 00:16:40


TeamCity Hyrise :: Unit Tests [Build 56](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=64&buildTypeId=Hyrise_UnitTests) outcome was **SUCCESS**
Summary: Running Build time: 00:03:45


Why would you have "tipo: combi" and "type: combi" (the same thing twice)?


Hi Michel,

I'm wondering why you removed this record. Actually, it fails to compile for me with quattor-14.6.0

Pierre


What was wrong with "Motorista"?


That would be awesome ive looked and will continue to. If i find a solution i will share it with you to. Thanksfor the response.


Same: Fixed in 1af9c78d7c166a14a39f3563d6a278c11e9c9e29


What was wrong with "Trabalho" ?


Oops, sorry about that.


Thundurus and Thundurus-T are in both lines


Fixed in 15fb61f5789038ae5fb93c0d26527bf9f2c4a7bd


You can't get the field enum by casting the depth value. It's just a coincidence that it happens to work (as it's logical for the enum values to be zero based and defined in hierarchy order), but there is no guarantee that the enum values will continue to match like this in the future. That's why there is a lookup table for the mapping: `LookupKey::kHierarchy[depth]`.

While changing that, I'd also suggest that you restructure the code a bit to better fit the style of the rest of this file:

```C++
    AddressField field = LookupKey::kHierarchy[lookup_key.GetDepth() + 1];
    if (IsFieldUsed(field, address.region_code)) {
      // If children are used and present, check them too.
      for (std::vector<std::string>::const_iterator child_it =
               rule->GetSubKeys().begin();
           child_it != rule->GetSubKeys().end(); ++child_it) {
        LookupKey child_key;
        child_key.FromLookupKey(lookup_key, *child_it);
        CheckChildrenForPostCodeMatches(address, child_key, node, hierarchy);
      }
    }
```


Shouldn't this comment say `s+v[range(i1,n)]` or `s+v[i1:i1+n-1]`?


They should both return the stream. If it wasn't there, would you mind adding it please? :)


Sure, this is expected! It can work only with 14.10. You need to upgrade your machines to 14.10 first (should work with your current grid templates) and then to upgrade your grid templates to 14.10. Let me know if it does not work for you.


It should go under the sass gulp plugin. It was line 117 in the other commit and is now 115


There is a CCL version for 1.8


No probs - will do


Don't we use a wider type for, e.g., `UInt8`?


boni told me that he's overhauling the mod and won't do a basic port, when I asked him if he wanted me to port it.


Hostname, as returned by gethostname(),  or the string "localhost" if that
fails or it is unset/empty. The letter 'q'  is the only ascii alphabetic
character that is unused by strftime().

 On

Wed, 7 Jan 2015 20:36 Tim Ansell <notifications@github.com> wrote:

What is %q ?


Sorry, I thought it was allowed since I saw you doing on some places, like Visitors::addDevice


This line did not exist when I edited that file just now.


Ah, wait, if you've gotten the element type of `c` correct, this will be fine here. But it looks like we don't get it right, at least not now (current master):
```
julia> a = rand(Uint8, 10^4);

julia> c = cumsum(a);

julia> eltype(c)
UInt8

julia> c[end]
0x75

julia> sum(a)
0x0000000000136c75
```


@dannyqiu I should have thought of that when I paginated, shouldn't I have? :+1:


Yes, because `v7_create_value` uses `v7->object_prototype`. The root prototype's prototype is NULL. This is somewhat cryptically commented in the line below :-)

In alternative I could write:

```
v7->object_prototype = v7_value_to_object(v7_create_value(v7, V7_TYPE_GENERIC_OBJECT));
v7->object_prototype->prototype = NULL;
```

But I prefer to ensure that `v7_create_value` never gets garbage pointers, in case we will later rely on some pointer tagging in the prototype field to encode some behaviour within `v7_create_value`.


Then, the method should be named `setState`. It'd save `__state` and use `setBits` inside


еще_один_файл.txt

угу


Default Einstellung alle Offinline Artikel in XML anzeigen ist bedingt gut, da viele die Offline Artikel sicher nicht nur zum verstecken in der Navi nutzen.
Vorschlag: alle Offline verstecken in Sitemap solang nicht ein Werte yrewrite_index gesetzt ist


Tiny point, but would it be better (faster) to use `n2 = n>>1`? Probably won't matter a bit in practice, of course.


Trabalho => Work
Motorista => Driver

2015-01-07 9:41 GMT-02:00 Jakob Tischler <notifications@github.com>:

> What was wrong with "Trabalho" ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Admilson/courseplay/commit/d21a91bc420ae8e667174d30df750f81680a73e1#commitcomment-9183921>
> .
>

--
*Admilson Afonso*


I am getting the error:

Starting redis-cache-cpe
Traceback (most recent call last):
  File "db_cpe_browser.py", line 30, in <module>
    r.sadd("prefix:"+prefix, cpetype)
NameError: name 'prefix' is not defined

I checked, and the reason is new formats, which contain more or less then 4 :'s


Reviewed !


Reviewed. Looks good.
Will instead of just rha-summary as id rha-case-summary will make more sense ?


Wow, I didn't know about default values. Excellent stuff!


and what about the commit?
And add the CLI commands for each step (like this: "git checkout -b initial")


If there are still problems with this, let me know. I don't have commit privileges on the Zotero translator repo, so it's in your hands.


Dear traveller. Testing editor with auto paragraphing off is totally irresponsible, because significant part of the tested algorithm is disabled.


In such cases it can be useful to pass parameter as ```int``` and check its value (and fix it if values are invalid - then subsequent code will be simpler and clearer).
Or at least carry out checks on the boundary values at the beginning of the function, for example:
```c++
if (lastKnownId >= msgCounter)
    // we already received all messages
    return QVector<Log::Msg>();

int firstId = msgCounter - m_messages.size();
if (lastKnownId < firstId)
    // we need all messages
    return m_messages;

return m_messages.mid((lastKnownId + 1) - firstId);
```


Done


The code will be shorter and better readable it of would use Unicode characters directly (I mean `external/localization.js`):
    - `"Schließen"` instead of `"Schlie\u00dfen"`
    - `<Zurück` instead of `<Zur\u00fcck"`
    - `"Vor>"` instead of `"Vor>"`

Best regards,
Oleg


```explicit``` is useful for constructor with single parameter only.


section6までの訳が終了しました。
明日以降はsection7にとりかかります。


@mc007 Please consider creating a pull request at https://github.com/dojo/util/ for that


Про это и хотел тебя спросить в скайпе. Но тут другой момент, по идее в дальнейшем для каждой строчки должны добавиться действия? Ну к примеру при прощелкивании строки STN подсвечивается вершина, к которой относится эта строка, при прощелкивании SO - вероятно подсвечивается открытая вершина и ее потомки. Но я тоже за меньшее количество строчек.


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


Impressive!


sorry to be pedantic, but strictly speaking we don't know who call this method, so we don't know it's a blocking operation. Maybe pass a message and have "Expected current thread [" + t + "] to be a transport thread (reason ["+ reason + "])


This is not a test class. And it was completely comfortable inside the ListWidgetModule, which is where it belongs. You can rename it if you wish to reduce confusion.


Why `typeof` rather than `=== undefined` ?


IMO, word ```Log``` in these function names is unnecessary.


Fixes #19


same here. also on hangouts... or if you delete an email. delete and undo words are in black... i couldnt find where to fix.


I've noticed that k8s guys are naming the references as objectRef to distinguish it's that kind of object (see [here](https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/api/types.go#L556)) , so I'd propose `ImageRepositoryRef` here.


Due massive `@inheritdoc@` usage this could be happening for a long time now.


Please convert the tab to spaces as per our code convention. Thanks.


`strcmp("vfpv3", "vfp")` will not pass now. Any reason why it's a replacement? What about `else if (!strcmp(tok, "vfpv3") || !strcmp(tok, "vfp"))` instead?


Call it “GenericListWidgetDelegate” or “PrototypeListWidgetDelegate”.


Ok, great! If you find a sustainable solution a pull request would be fantastic.


This file shouldn't be here.


Yes, looks good to me


Fixed later.


Возможно это действия сведутся к вызову единого метода для всех случаев.
Так или иначе, если вдруг потом надо будет разделить часть кейсов - разделишь, а пока, чем компактнее, тем лучше.


Lack of context is the larger problem. People get into the chat not knowing what to do with it, so we have a lot of test messages. There isn't much active conversation in the channel between events, so to everyone who comes in it looks like the place is mostly dead.

We're going to continue using Scrollback on the event websites and will find new places to use it, but with some thought to context so people know what to expect.

The app helps of course, but at this time doesn't support email login, so I can't use the same account I use on the website. Don't see a way to migrate the existing web login to Gmail or Facebook either.


Can't test it because you need to pay £100 to try it out on a device since you need to be a certified developer. Mine expired a while back :(


Actually it might be better to rename the SetDepthRange znear and zfar like we do elsewhere instead of de-fucking stupid compilers. Just from the "avoiding cruft" angle. Your call though, because its ifdef'd.


yeah, thought about, you're sure a re-compiled gcc would be accepted ?


Well spotted, will get that fixed.


Видимо туплю, но почему нельзя объявить идекс
```java
int index
```
и потом написать вот так:
```java
for (index; index<10; index++)
```
?

Мне выдавал ошибку, либо инициализировать переменную в `for`, или инициализировать до него, а в `for` изменять его каким-либо образом, например index--. Я сделал последним способом, благо мне действительно нужно уменьшать на единицу, чтобы он 2 раза не проверял одну и ту же строчку в начале.


The new evaluate_pawn_endgames doesn't seem to take K position into account. Is this intended?

Symmetrical pawn layouts are not dead draws if the kings are far enough away.


Dev tool bar requires this?


As discussed, do not use `My*` for naming


Where did you find the 'strcmp("vfpv3", "vfp")'?
Maybe your version is safer, however I have yet to see an ARM that have vfpv3 but not vfp.


This needs to be prepended with the bundle namespace


Not able to verify this, but as far as I know, you can directly use *rvalue* in **foreach**:
```c++
foreach (const Log::Peer& peer, logger->getPeers())
    //...
```


The DefaultTokenServices have reuseRefreshToken by default set to true.
I don't see what this method adds ?
Shouldn't it accept a boolean and set reuseRefreshToken ?

public AuthorizationServerEndpointsConfigurer reuseRefreshTokens(boolean reuse) {
                       this.reuseRefreshToken = reuse;


If the form class is now removed from the Frontend repo, please remove associative information. This service should be in the relevant repo with the class (eg. Domain Model)


Do you mean `if val is not None` ?


@kalasjocke It's coming up soonish!


@mc007 I don't know, maybe there's another way... best option is to create a ticket in the relevant place and we can discuss there.


Fixed #19


Who is 501? On all linux systems I have worked with, UIDs for users start at 1000. Hard coding this seems dangerous


This should be in the same repo as the Form.


it may be done in one single line like this: return unless current_project.try(:end_at).present?


You're missing an end-parenthesis on line line 537. You have
```c
calloc(1, sizeof(coincidenceTimes);
```

A fix is
```c
calloc(1, sizeof(coincidenceTimes));
```


It's `components/tests/python`


Btw I didnt touched anything for the sake of not breaking anything. But is this 2-ways-to-get-time code still relevant? Why theres 2 ways to get time?


Thanks


Hi ,bạn ơi movihd hiện nay không xem được rồi là do bạn lấy link của phim nhanh nếu bạn sửa lại thành link chất lượng cao thì sẽ ổn định hơn
#EXTINF:-1,[COLOR yellow] PHIM LE [/COLOR] HÃY QUYẾN RŨ EM
http://media2.movihd.net/phimle/20142210/QuyenRuEm.mp4
#EXTINF:-1,TRÒ ĐÙA TÌNH ÁI
http://media2.movihd.net/phimle/20140901/TroDuaTinhAi.mp4
#EXTINF:-1,CÔ GÁI MẬP
http://media2.movihd.net/phimle/20140901/CoGaiMap.mp4
#EXTINF:-1,NGÀY MƯA
http://media2.movihd.net/phimle/20140901/MyRainyDay-2010.mp4
#EXTINF:-1,LỄ TÌNH NHÂN BUỒN
http://media2.movihd.net/phimle/20140901/Le-TinhNhanBuon.mp4
#EXTINF:-1,THOÁT Y
http://media2.movihd.net/phimle/20140901/Thoat-Y-2012.mp4
#EXTINF:-1,CẢM GIÁC TỘI LỖI
http://media2.movihd.net/phimle/20140901/Marriage.With.A.Liar.2010-okok.mp4


some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3041
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3047
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3042
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3033
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3033
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3036
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3029
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3024
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2701
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2699
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1537


I simply threw https://github.com/hvr/hackage-matrix-builder at `fclabels` which pointed that out to me... :-)


Looks good.  The version is taken from the data objects when exported to XML so we'll need the corresponding update to `SPProgram.VERSION`.  One might think it would make more sense to use the same `spModel.core.Version` everywhere instead but, well, sorry.


It is deliberate, as you say it's so the error message ends up in the terminal in development or in a log file in production.


Should be ammended to:
if ((args.y && args.y.percentage !== 0 && args.y.percentage !== 1) || (args.x && args.x.percentage !== 0 && args.x.percentage !== 1)) {


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


I think the `function () { resolve() }` is redundant, you can just say `resolve`?


I think you mean `retain = retain == 'True'`. The quote marks are important - you want to set a boolean value to `True` if the string value is `'True'`.


Why not just return `promise`?


right, ok then, i will try, but actually have no time for this things, sorry.


1: M111 unrecognized component name: authorids


+1, better than _NEON I think, but to nit-pick I'd suggest CFG_CRYPTO_SHA256_A32_CE because the official ARM doc refers to "*The [ARMv8] Cryptography Extension [that] adds new A64, A32, and T32 instructions to Advanced SIMD*" (see [here](http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.ddi0500e/CJHDEBAF.html)).


корректный синтаксис был бы таким:
```java
int index;
for (index = 0; index < 10; index++)
```
или
```java
int index = 0;
for(; index < 10; index++)
```


This isn't right, those array should to be not translatable.
I ment to change array entries from

<item>Regular</item>
<item>Center-mirrored</item>
to
<item>@string/battery_bar_style_regular</item>
<item>@string/battery_bar_style_center_mirrored</item>

or something like that.


I think some more letters may be unhappy in other files. Fix them as well. -_-


probably will give it a try.


This line creates an extra 16px (scrollbar height) space between the last row visible and the scrollbar


Why?


because promise is not always a promise.


saw approval from brson
at https://github.com/alexcrichton/rust/commit/01ce6efd859ed6ce3d20d331ae6e035ebd0a7678


merging alexcrichton/rust/stabilize-libc = 01ce6efd into auto


status: {"merge_sha": "dfd557bd73a27966b242635b04f2a9fca6eadce8"}


alexcrichton/rust/stabilize-libc = 01ce6efd merged ok, testing candidate = dfd557bd


Ugh great function prototype without a real function Great code reviewing people :(


What about having only this line in #ifdef LTC_CLEAN_STACK rather than duplicating the whole function?


You can put them on the same line without brackets if it doesn't exceed 80 chars, but as soon as you break the line, the brackets are required.


Ah thanks for your explanation.
I discussed with @Valodim and we came to the conclusion that this would overcomplicate the scheme. When adding key signatures, the URI would change every time. Thus, if calculating a hash over the whole key, even the key owner  always needs to have the newest key.
There is no easy way to preserve the integrity of the whole key using a URI. If you fear such attack scenarios, you should possibly exchange the key via NFC or Safeslinger (which has been build into OpenKeychain) instead of sharing by QR Code.

However, after understanding the "4" in "openpgp4fpr", I think we will stay at "openpgp4fpr" instead of "pgp-fingerprint".


Understood, thanks for explain


@eldh  :+1: :+1:


i love :coffee:


this character:  ×


i live :+1:


Sure, the content files could have a say about the grid size in 1.1.

I considered the even/odd number issue, but I would value clarity over correctness here. "grid size" is easier to interpret than "grid radius" or "grid range". Actually, even numbers in the cfg file wouldn't break anything, behaviour-wise it would would simply round up to the next odd number.


Whenever possible, please also test with Long UNC paths.


HELLO TEACHER !! is this commit??


I wonder why this class actually has a main method, testing code should go into a unit test that is separate from the class to be tested!


Add an issue. :)


I'm not even sure..

Btw, SDL2 seems to provide interface to high resolution timer:
https://wiki.libsdl.org/SDL_GetPerformanceCounter

2015-01-07 15:19 GMT+03:00 Alexey <notifications@github.com>:

> Btw I didnt touched anything for the sake of not breaking anything. But is
> this 2-ways-to-get-time code still relevant? Why theres 2 ways to get time?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Warsow/qfusion/commit/b3dd89841473a3fee9b97033e4fbbe9b728ca147#commitcomment-9184325>
> .
>

--
Best regards,
 Victor Luchitz


done in this commit: https://github.com/ibm-js/deliteful/commit/9e44252e154e68d651c09949878d909df0e523c3


wrong coding style? one param per line?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/9829/testReport/) for this commit**


Ah OK.


what about windows? do we need to get it from another place?


@fundon gitignore? or take it to a submodule?


yeah I know, but why i current windows backend there are two implementations. Why not to remove timeGetTime() version?


I had such thoughts but decided not to break things


PS: Is promise is either a Promise or undefined?   If so maybe you should `return promise || Promise.resolve(true)`?


use && product == NULL?


But taking into account that there were some win95 hacks from original quake2 codebase...


Yes.  My apologies.

Too much time spent with the pip vs git versions yesterday.

Mod should read:

if val is not None:


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1798](http://teamcity.monogame.net/viewLog.html?buildId=4217&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 302, ignored: 6 Build time: 00:16:13


Formerly it was sufficient to be battle host to be able to force, so Springie new-era QM was implemented in Springie.
This change requires the forcejoining users to **also** have moderator privileges, which breaks QM without warning.


Once https://github.com/ibm-js/deliteful/issues/434 is fixed, I'll use an icon instead, won't need to set font-size, line-height and the content property in this class. Can we wait for https://github.com/ibm-js/deliteful/issues/434 to be fixed?


There's any reason why this function is called toQuerty? Shouldn't it be called toQueryString()?


Corrigé, merci !


Remove this line


Those two strings are duplicated.


Let it looks good.


The fingerprint is exactly enough to identify a keyring, ie, a master key. Certifications for this master key are correct regardless of other state like revocation certificates or anything, since they bear no semantics on the state of anything which is part of the keyring, other than "this keyring is the one referred to by the person who gave me the fingerprint (or qr code)"

If you fear a compromised keyserver might strip revocation or other certificates, that is a more fundamental problem, because it might just as well happen at any later time. Even if you assure that at the point of exchange the keyring is intact, that assertion holds no longer than that instance of exchange. If you write a mail to this person a year later, how can you be sure you didn't miss a revoked subkey? Checking at the moment of fingerprint exchange does not change anything about that problem.


Oui, plusieurs constructions dans ce fichier sont à revoir. Je laisse ça sur le feu et je ferai une édition la semaine prochaine, puisque de toute façon je compte illustrer ces sections.


I wonder if instead the context should just keep the list of root hubs and then each device (hub) keep a list of the devices it contains?


I'd prefer to calm hasSetGstDelete mixin Map


Загрузка музыки из VK/MainWindow.xaml.cs

Строки 345 и 378 - передача MediaElement источника по протоколу HTTP вместо HTTPS

См. баг: https://connect.microsoft.com/VisualStudio/feedback/details/934355/in-a-wpf-standalone-application-exe-when-the-source-of-a-mediaelement-is-set-to-a-https-uri-it-throws-a-nullreferenceexception


@abstractFlo +1


Better be consistent with parallel in using Iterable for parameter. Ditto for next method.


what is this..?


reference #118


kljslkdjflkjasf


This will not work if real_T is not double (ok if we assume that real_T == double).


Cool stuff!


getName() is better suited. Every Feature has a NAME but doesn't need a Label.


If there's a necessary change, add it.


I considered `Iterable` but wanted to force callers to give a collection with a consistent iteration order. What is the point of calling `sequential` with a `Set`?


Rather than simply remove this, could you just comment is as "obsolete - available for reuse"? And do the same for FileSourceTypeId?


das war davids einziger Beitrag


Don't have. It's fine. :smile:


It appears that QPC has its own share of issues, especially on multicore machines and perhaps this is why it's turned off by default.


This just a static reference. Can't we just require this at the top? Especially because it's in the same package.

I would only inject if you anticipate another StoreIt reference being passed in.


Congrats!


You have inconsistency here, since the `ImageRepository`'s `Name` is just `ruby-20-centos`(see [here](https://github.com/bparees/origin/blob/add_dependency_id/examples/sample-app/application-template-dockerbuild.json#L57)), but you're using here `dockerImageRepository` field value.


Not by default, it doesn't.


It is called "protoduction": http://blog.codinghorror.com/new-programming-jargon/ (see #22) :-) I absolutely agree this has to be removed. But in _this_ particular PR.


> @Redsandro what exactly is complicated about the iconUrl?

@TomOne I was doing things by heart. I did the trivial first push to humor the guidelines, and I checked on GitHub for a button that would provide a CDN link to the icon file in that specific push, which I could not find. As I understand now, there is no link anywhere to be found, right?

Personally, I do not care much for things that require reading in stead of intuition. Especially if there is no value in my opinion. If I did, I would excel more in speaking French and less in math. Also, @ferventcoder seems to care less about old icons (https://github.com/chocolatey/chocolatey-coreteampackages/issues/20). However, some people do care about this, so I thought I'd at least push it in so that it's there.

I neglected to click around to that part of the wiki because I had no time, yet people are using the package so it would benefit all to get it to update as soon as possible. :)


I've released 0.9.5 which includes your fix. Thanks!


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/9830/testReport/) for this commit**


Why empty? Is it because ARM TF already performed the init?


tab this line over one to cleanup the formatting a bit


Hey there, sorry for taking so long to reply. I didn't remember my earlier comments and seem to have changed my mind about this. So, yes, this reverts 51c9731.

Vim defaults to using 0 for the current line number in relative mode, but introduced ["hybrid" line numbers](http://jeffkreeftmeijer.com/2013/vims-new-hybrid-line-number-mode/) in 7.4. Right now, it seems like toggling between "hybrid" mode to absolute mode is more useful than switching between relative and absolute mode.


Unavailable


move this over to the left one tab to cleanup the formatting.


That's great. Thanks @z3t1 !


I think it's basically best-effort at this stage


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/9831/testReport/) for this commit**


인코딩 설정이 문제인걸까요..?


I thought only the import of GurobiML` in init.m would be necessary.


We're already mixing styles, but if you want we can fix it up.


With this regEx the method returns (<id>,None) or (None, <id>), but not (<host>,<id>).
'//(?:embed.)?(.+?)/(?:video/|embed.php\?[\w=&]*v=)(\w+)' would match.


I don't think libusb give you any way to get "children", other than to enumerate all devices and then look at the parent.


Future idea: scrobble song which has been played at least on 40%. We need to count amount of seconds played (in `player._update_position_callback`) and in `_on_bus_eos` this value is at least 40% of the total duration


Why does normal word need to access CP10/CP11? Is it required for the subsequent SHA acceleration patches, or is it a more general improvement?


Yes, that would be better, thanks.


This `_setAfterTransitionHandlers ` and `_afterTransitionHandle ` will work but perhaps it's more natural to do this:

```js
_setAfterTransitionHandlers: function (node) {
	return new Promise(function (resolve) {
		function after () {
			node.removeEventListener("webkitTransitionEnd", after);
			node.removeEventListener("transitionEnd", after);
			resolve();
		}
		node.addEventListener("webkitTransitionEnd", after);
		node.addEventListener("transitionend", after); // IE10 + FF
	}).then(this._afterTransitionHandle.bind(this);
},

_afterTransitionHandle: function () {
	$(this).removeClass(prefix("under"));
	if (!this._visible) {
		setVisibility(this, false);
	}
},
```

I think it cleans up the code by not having to putting the `addEventListener` and `removeEventListener` in the same function, so that `_afterTransitionHandle` is just actual DOM manipulation stuff.


동사형으로 메소드이름 부탁드립니다.
toHexadecimalNumber
라고 하는것도 괜찮아 보이네요.
추가적으로 인자값이 intNum이라고 되어있는데, 이미 int형이기 때문에 intNum이라는 이름보단 num, number 등이 좀더 나을것 같습니다.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/9832/testReport/) for this commit**


Is this method needed somewhere else otherwise I don't think it is really needed and os.architecture should be used directly.


Thanks @ximenesuk, I guess this is another point for excluding the broken anchor from the Docs job so that we start paying attention to the broken links reported in https://ci.openmicroscopy.org/job/OMERO-5.1-merge-docs/351 /cc @hflynn


각 연산자마다 공백을 추가해주면 좀더 가독성이 올라갑니다.
int i=0; 보단 int i = 0;   이런식으로요


세미콜론(;)으로 라인을 종결한 이후엔 다음행으로 이동해주세요.


I suspect the original motivation was to have a high contrast friendly visual distinction between checked/unchecked states.
- In normal mode: this information is conveyed through the box-shadow so the border is to be hidden (hence the transparency).
- in high contrast mode: no box-shadows, no transparency, you get the border you see on the picture.

![image](https://cloud.githubusercontent.com/assets/2982512/5645786/3d80d8f0-9674-11e4-867e-91a688407c10.png)


문법은 아니지만, if나 for문 등을 사용할때 항상 대괄호 { } 를 사용하여
범위를 표현해주시길 바랍니다.
위 코드처럼 한줄이라도요.
그렇게해야 이 코드를 볼때 햇갈리지 않게됩니다.


> errors should contain the flag names the programmer requested, not the internal / recommended names.

The error doesn't contain a flag name at all: `Unable to read auth-path <specified  path to auth file> for <nickname of the auth info config> due to <error that happened during the read>`.  I suppose that "nickname of the auth info config" being empty (as in the case where all value are passed via cmd line) would look weird, but someone specifying all the flags will recognize the "path to auth file" they wrote.  I think having the nickname in the message is particularly useful if you get the message after merging happens, so you can see exactly which user stanza is being used.


I honestly do not know which directives can break Tracy or Latte (otherwise I would to fix it). Do not rely on `default_charset` is exactly the same as do not rely on `iconv.internal_encoding` or `mbstring.internal_encoding`.

In addition, we are talking about serious security issue. Wrong `default_charset` can lead to massive XSS.

PHP 5.6 is really giant step back :-(


This is called on the server.
I believe the problem is that `BlockLeaves beginLeavesDecay` removes the tile entities on the client. I'm not sure how. It took a crazy amount of debugging to narrow the problem down.

If you can help with this it would be much appreciated.


Maybe, all this code is still work in progress so i'm open to any proposals


Also forgot the fact that rendering works now.


fixed


Since `computeAttributes()` always returns the default value, this whole method should be replaced with `h = parent->attributesPlug()->defaultValue()->hash()`.


Pretty cool link, thanks :) Since I've just had an error in Markdown 2.5.2 from the Link extension, I'll try again and see if this was accidentally the fix for #365


Since `computeGlobals()` just returns the default value, we should return the hash of the default value here.


Checking it out!


Comment for this thing


space in the end?


@selu91 cum pot sa fac revert aici ?

@robciucioan la text domain ai folosit medica-lite, la temele live ar trebui sa fi mai atent.


The bottle hashes fell out of sync with the actual updated bottle packages after this commit. Maybe a missing BrewTestBot hash commit?


Thanks for sorting.

Please can you not have generic comments like `fixing stuff`, `DevTool bar security added back in` would be better. If mine are not understandable please also nudge me.


Mmmm


Good idea @ledvinap .

I've implemented your idea here (and done some more tweaks to it):
https://github.com/nebbian/cleanflight/compare/cleanflight:master...baseflight-pid-horizon-mode-tuneup_a


code review test


Perhaps it's worth updating the indentation here?


А чем функция array_values не устраивает?


great


@mcm-ham could you please tell me why we have data.date here?
Where exactly we are using it?


i just switched to the latest version of this gem and everything works with utf8mb4. I haven't tried with just utf8 because in mysql-land that would not really be what we wanted. either way, thanks for responding.


I can't seem to think of a case where nil versus !nil would be a problem.
Why would you require this kind of method to return a boolean?


This meta tag already exists on line `6`


Missed that, thanks. Pushed in ea8898dde8fb90df781977d9e1af2d0a74ec7702.


Please match the indentation.


Good job


you can use a const block here:

```
const (
        foo = 1
        bar = 2
)
```


Thank you.


test


헉...
아예 다시짜셨나


@jrfnl Thank you for your feedback. I have undone the separation part.

I didn't know splitting the classes in separated files causing significant performance issues.

I'd discussed with @omarreiss for separating the files for development purposes and combining them with a grunt task. But we didn't find out a proper solution for this right now.


Here it looks like one has to add it to every model individually.
https://qt-project.org/wiki/Model_Test
Is that something else?
Or did you also do that part?

I think we could link it to Debug
And enable it for travis on the command line (building with Debug took too long and triggered test timeouts once in a while, that's why it's disabled)
It could then be removed from the GUI options


What for is good this commit? I can't build with it bricked_hotplug due to conflicts in here and ../mach-msm/msm_rq_stats.c.


This line has the problem.


aaaa


그런데 흠님은 약간 함수 선언과 사용을 남발하시는듯... 1103~1113줄은 한줄이라서 그냥 집어넣어도 미관성 나쁘지 않을 텐데... (함수 참조는 의외로 성능에 많이 영향을 줍니다)


Unnecessary interface; remove and rename BasicQueryResult to QueryResult.

I know this pattern is elsewhere in the codebase, it's a bad one :)


Nifty, this should help with the JIT spec apparatus.


The ref says “*The base URI for a URI reference appearing in any other attribute value, including default attribute values, is the base URI of the element bearing the attribute.*” so the value of `xml:node` has to be extracted before processing other attributes.  Will do that.


@DerRomtester  Here /sys/class/kgsl/kgsl-3d0/gpuclk


Don't push broken code!
On 7 Jan 2015 13:40, "Oliver Faircliff" <notifications@github.com> wrote:

> I think it was unfinished
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itsapi/net_game/commit/bf3ab991cc07608b5aef69e322339d68c0eace8b#commitcomment-9185274>
> .
>


这个地方对吗？ 我以前使用的是replace-regexp-in-string


Bad naming. This is more of a result-carrier class than a state.


As discussed:
- Some people would expect this to return a bool at all times
- `0`would evaluate to false when you don't want it to


Removing static?  "static final" is the correct way to do constants in Java.  Removing static means an extra copy of this field in every instance of the object - wasteful of memory, and slower too.


Also, `atLeastOneIndexUsed` is not used outside the SQLTranslator class, so shouldn't be here really?


I keep forgetting this... thanks Pedro!


Very much inspired by @jackfranklin!

> On Jan 6, 2015, at 10:03 PM, Evan Hahn <notifications@github.com> wrote:
>
> steal @adamyonk's brilliant script
>
> https://github.com/adamyonk/dotfiles/blob/master/script/install_brews
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/EvanHahn/dotfiles/commit/791cc79efb4b5a39bb7f48b94562adf26075c26c


I suggest that you make the key store implementation types private and create a constructor
function that returns the interface, e.g.

```
func PassphraseStore(path string) KeyStore {
         ...
}
```

For an example in the standard library, see [the hash packages](http://golang.org/pkg/crypto/md5/#example_New). All hash implementations
return a Hash interface value. The benefit is that you only need to document the interface and the constructors.
godoc will put all the constructors under the interface definition.

package p2p [does this for NAT](http://godoc.org/github.com/ethereum/go-ethereum/p2p#NAT).


当禁用分词时，不应该执行shell-command-to-string


Je l'aurais plutôt appelé Node.js


Valid point, not the case though.


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=73bb8f98a54746be906ef714cd6cac96
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [FAILURE]
plone-5.0-python-2.7 [SUCCESS]


Makes sense. Will change.


will change it in my next commit.


typo


Just to confirm:
Dashboard is using "oscar/js/oscar/ui.js". But this script is fixed to be Bootstrap3 compatible (danger messages from error). Dashboard will be ported to Bootstrap3 in 1.1? Otherwise it will be problems!


Is that 'b' intentional?


Will integrate straight into the existing run_powershell_cmd() method.


`query.keySet().toArray()[0].equals(AND)` would be clearer, I think.

Creating an iterator implies you're going to loop.


yo


> 当禁用分词时，不应该执行shell-command-to-string

看这里，禁用分词时，就不会再分词了（不执行 `yasdcv--chinese-word-prediction`），


That was a mis-understanding. Thanks for pointing it out. I just corrected it in 6984d65fb53319cf0ffd8fd8a9685ceedb3f634c .


profile_soundcloud_test.php is a page where an event creator can choose a track or one of his own playlists to add to the event (
profile_soundcloud_test2.php is a page where the soundcloud track(s) can be played (test_event - id: 999999).


My IDE removed this since it was not being used


@Magnus-G Can this partial be removed? It’s all commented out.


yeah i guess w/ changing it to a ref, this should be changed.


In the cases in which the win client is not used in the test I agree that it is useless to initiate the win_client too. But if every test in the test class depends on win_client I think it's as good a place as any. Or?


Please try to avoid naked returns, especially in long functions. They make it hard see what's being returned.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/9836/testReport/) for this commit**


You've had success building the solution with multiple PC's and I'm having the same issues with both PC's I've tried. At least we're both consistent! lol

The errors I am getting are stemming from the project ServiceStack.OrmLite.SqlServer and two different files.

File: ServiceStack.OrmLite.SqlServer\SqlServerExpression.cs
Error 1 : 'System.Type' does not contain a definition for 'GetDefaultValue' and no extension method 'GetDefaultValue' accepting a first argument of type 'System.Type' could be found (are you missing a using directive or an assembly reference?)

File: \ServiceStack.OrmLite.SqlServer\SqlServerOrmLiteDialectProvider.cs
Error 1 (three occurrences): 'string' does not contain a definition for 'Fmt' and no extension method 'Fmt' accepting a first argument of type 'string' could be found (are you missing a using directive or an assembly reference?)

Error 2 : 'string' does not contain a definition for 'SplitOnFirst' and no extension method 'SplitOnFirst' accepting a first argument of type 'string' could be found (are you missing a using directive or an assembly reference?)

Note: SplitOnFirst and SplitOnLast do not have definitions.

Error 3 : 'string' does not contain a definition for 'StartsWithIgnoreCase' and no extension method 'StartsWithIgnoreCase' accepting a first argument of type 'string' could be found (are you missing a using directive or an assembly reference?)


@tumashu 我预设了中文分词的结果用空格隔开（结巴分词的结果是这样的，但是我不知道 scws 的分词结果的格式），所以这里 scws 不知道行不行。


Typo?


See issue #351


If following WP code standards, all of these should use braces.


You can use struct literals, e.g. `&KeyStorePassphrase{DefaultDataDir()}`.


It's ok. Take a look for example here - https://github.com/temasek/android_packages_apps_Settings/blob/cm-12.0/res/values/cr_arrays.xml#L32
there are defined array items as translatable strings.


Depending on Netmarketshare.com, more than 30% of users still using IE < 10; So this will display an alert for 30% of users? I don't think that is a good idea.


This is, I think, due to [setShowsDialog](http://developer.android.com/reference/android/app/DialogFragment.html#setShowsDialog(boolean)), and the fact that I start the Fragment with a TransactionManager rather than with DialogFragment.show(). We really want to use show() here since the functionality should be essentially that of a popup dialog, though that may again add trickiness relating to alignment.


So I know what I said yesterday, but maybe the loader pipe methods should be getPipe.  All of the other managers follow the getXXX pattern


> Travis CI's bug , it will override your CC' variable beforemake'

但我还是认为得在 `./configure` 的时候加 `--target`。（然后再加 `CC`）


You can use `t.Error(err)`.


Without that commit it does not work for me because i have disabled msm_rq_stats and bricked_hotplug only needs get_rq_info so i have added this include thing.

You have to add that from ak: get_rq_info inside msm_rq_stats:
https://github.com/anarkia1976/AK-OnePone/commit/b40914b72ebc63699346fece8193f9536037ea46
Than compiliation should run fine without my commit.


HVACSizingSim_CoincidentPlant (EnergyArchmage) - 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)


Maybe I should test directly against `'user.signed_in'` rather than `controller.resource_name`?


poor mike


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3048
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3042
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3043
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3034
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3034
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3037
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3030
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3025
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2702
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2700
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1538
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1530


fast-forwarding master to auto = dfd557bd


Very minor detail, but is there a case where
```
_==true ? true : false;
```
is different than
```
_==true
```


Another reason I don't want to change the name to `flycheck-clojure` is that it hides the significance, as I see it.  Emacs is now the only Clojure IDE that performs the sort of error highlighting that developers in most mainstream languages have come to expect, and this happens because of composition-friendliness at all levels of the development stack.  If you build a plug-in for IntelliJ, it's just that, a discrete piece in a big OO framework that probably implements some `IntelliJPluginI` interface and so by convention has IntelliJ in its name.   Squiggly Clojure is just a trivial composition of Cider+Flycheck+Kibit+Eastwood+Core.Typed that isn't obviously a part of just one of them.


Very minor: this var is unused.


Just a small note: Could you try to `git cherry-pick` commits like this one from our repo? (see my mail from yesterday for a how-to) It will make synchronizing our branches easier in the long run.


**FAILURE** - TeamCity Infinispan :: Pull requests monitor [Build 8226](http://ci.infinispan.org/viewLog.html?buildId=22617&buildTypeId=bt9&guest=1) for ecd616d0c5eb69b6e5749c436486b57691d00fe9
Tests: 11217, 0 failed (0 new), 23 ignored. Build time: 02:08:30

Error message is logged

Build duration (secs) is too large: 8027


Ok, thank you


All done now.

Enoch Appathurai

> On 7 Jan 2015, at 19:39, tristen <notifications@github.com> wrote:
>
> Please match the indentation.
>
> \
> Reply to this email directly or view it on GitHub.
>


please collapse into the original commit before merging


while I don't see an issue, was this tested?


It seems that this don't work for me. After boot I see, that max GPU frequency is set to 657 MHz.


You're missing a lot of tests from https://github.com/cloudant/CloudantQueryObjc/blob/master/Example/Tests/CDTQQuerySqlTranslatorTests.m, even just for supporting the current functionality (e.g., where clause generation).


great


I will double check today


saw approval from nikomatsakis
at https://github.com/huonw/rust/commit/0ea534f10700fe934f90bcae67484c71433c338c


merging huonw/rust/no-drop-and-copy = 0ea534f1 into auto


status: {"merge_sha": "bf08c527513b95910b5058bb7e187b3d2b3889b6"}


huonw/rust/no-drop-and-copy = 0ea534f1 merged ok, testing candidate = bf08c527


PLease also push the AbilityyUtil.java :)


@tysongach fixed!


you can write
```go
id := keyId.String()
return ioutil.ReadFile(path.Join(keysDirPath, id, id))
```


yeah, will change the json and the code to do it this way, I didn't think about changing this when i switched the type to Ref.  thanks for the catch.


Thank you a lot for explenation. I played with it for some time, now it's clear to me :).
BTW. can you show me, where did you disable msm_rq_stats? I can't see it in defconfig.
Thanks!


Re: setting `kwargs` to `None`. It is poor practice to default to an empty dict in a function declaration. This code should check to see if `kwargs` is `None`, and if so, set it to an empty dict inside the function itself. A good explanation can be found [here](http://pythonconquerstheuniverse.wordpress.com/2012/02/15/mutable-default-arguments/).

Re: line ~492. This looks like a pull request that I would be happy to accept, were that `log.info()` to be changed to a `log.debug()`.


status: {"merge_sha": "bf6847aaf98fb7c06a2fc64a0dd94e5b0dc2e3f9"}


FUUUUUUUUUUCK


huonw/rust/no-drop-and-copy = 0ea534f1 merged ok, testing candidate = bf6847aa


1: M101 signature is missing
2: M102 unrecognized bullet/signature
3: M102 unrecognized bullet/signature
4: M102 unrecognized bullet/signature
6: M102 unrecognized bullet/signature
7: M102 unrecognized bullet/signature
8: M102 unrecognized bullet/signature
10: M102 unrecognized bullet/signature
11: M102 unrecognized bullet/signature
12: M102 unrecognized bullet/signature
13: M102 unrecognized bullet/signature
15: M102 unrecognized bullet/signature
1: M110 missing component name
1: M190 line is too long (73 > 50)
2: M190 line is too long (78 > 72)
3: M190 line is too long (77 > 72)
6: M190 line is too long (77 > 72)
7: M190 line is too long (80 > 72)


Haha, glad to be of use :balloon:


> I honestly do not know which directives can break Tracy or Latte

Couple random examples for Tracy.

~~~
zend.multibyte = 1
zend.script_encoding = "UTF-16"

disable_functions = "register_shutdown_function"

memory_limit = 128B
~~~


This is intended. I made a lot of experiments locally for how to take into account king position and this seems to improve endgame classification of about 5%. What I exactly did was to take ca. 100  2 pawns fen from tablebases and look in which case shallow evaluations rate draw higher as wins. I tried different implementations of king distance to the next pawns and this was improving shallow evaluations marginally, but bench was slowed down. So I've gone back to this original version which passed STC and made a bit faster using the hash table. I hope that the speed improvement compensate the evaluation.


These checks completely alter (break) the tests for HHVM when looking at undefined variable. Here's the original:

![screenshot 2015-01-07 14 35 25](https://cloud.githubusercontent.com/assets/88371/5647089/a6be99aa-967a-11e4-84a2-d21a7e9a0d7e.png)

Here's the result with your code:

![screenshot 2015-01-07 14 36 48](https://cloud.githubusercontent.com/assets/88371/5647109/b0354b32-967a-11e4-87cf-84c8a4b6aebf.png)

By forcing it to false, when it's not set, you're basically not testing the case when it's undefined.


I believe that might have been a cat on the keyboard, or something of the like. I didn't mean to change it, if I did.


:congratulations: β


Hmm weird it works for me. But gpu oc does not work for me dont know why. I have to find out where the problem is.


> However, all our graphs were created using our log files but we wanted to graph things than we log.

you are missing a word here. `to graph MORE things`


should the hrefs not do hello/play as well?


Got it. DONE in 8b379f9ff1fafc3ec0f2f5ea0c79461425473d26


nit


Oh really? Please tell me more! @bufke


Somebody else did the individual enabling for some models, I only did the one for LayerTree and ported the pre-existing code to use the new macro.

Travis part is in testing here: https://github.com/qgis/QGIS/pull/1756

I hadn't noticed the new macro was in the GUI options, and still don't see it. You mean cmake GUI ?


Oops. I wasn't thinking, obviously.
Thanks, Rene.


Oops! Good catch.


Line is too long. [113/100]


Line is too long. [130/100]


Line is too long. [114/100]


Line is too long. [109/100]


Line is too long. [126/100]


Inconsistent indentation detected.


Use 2 (not 1) spaces for indentation.


So I waited for 2 years because you did not even try to build it on OS X? Toss :) Bad Attila, bad.... :)

Both methods work, Amber updated while Quartz was not found at transifex?

Thanks.

> On 6. 1. 2015, at 23:11, alanwww1 (Attila) <notifications@github.com> wrote:
>
> I just tried to compile my tool on OSX. Well as a surprise it did compile without any errors or problems whatsoever. It works perfectly.
> Just do a git clone:
>
> git clone https://github.com/alanwww1/xbmc-langdload.git
> cd xbmc-langdload
> after that simply run make
>
> make
> and a binary called xbmc-langdload is created. You can use it now for language file download.
>
> ./xbmc-langdload xbmc-skins/skin.amber /your/path/to/skin
> I also uploaded my binary in case compile does not work:
> http://s1.toldacuccot.hu/dl.php?sid=1a4cd94e09e621232cc7350de899dcf3&file=xbmc-langdload <http://s1.toldacuccot.hu/dl.php?sid=1a4cd94e09e621232cc7350de899dcf3&file=xbmc-langdload>
> Could you please try both ways, so I can recommend these way to OSX users ?
> Thanks!
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/pecinko/skin.amber/commit/2491ebdf521e4b67d8e0db3dcd366c43b727dfba#commitcomment-9176449>.
>


> why simple type here?

because what's the `Inside` member of this then? This is improper use of "simple type", of course, but it's just easy.

> You can perfectly bound it to be yourself at each case manually

why doing it in a lot of places instead of using just one trait


@ironsidevsquincy is this what you meant?


Great


Got it! 1st lesson of translations, lmao :)


This closes #2


Why must extracting the transport message be done in the behavior? I agree with @SzymonPobiega that making it more explicit by having a first class citizen usually makes the code easier to understand. See how I handle it here:

https://github.com/danielmarbach/ServiceBus/blob/master/source/MMS.ServiceBus/Dequeuing/MessageReceiverReceiver.cs

The message receiver translates the low level message into the higher level transport message


Sure I'll do that.
I'm having another error when creating a VM:
```
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/salt/cloud/clouds/msazure.py", line 555, in wait_for_hostname
    data = show_instance(vm_['name'], call='action')
  File "/usr/lib/python2.7/dist-packages/salt/cloud/clouds/msazure.py", line 406, in show_instance
    salt.utils.cloud.cache_node(nodes[name], __active_provider_name__, __opts__)
KeyError: 'vm_name'
```


I'm using FKU for checking and setting values.
For me work set it via RAMdisk - https://github.com/cmke/android_device_oppo_msm8974-common/commit/42e37b398c756188b6d4f0c2cc011979ec48a767


fixed


This branch is now stale, @SzymonPobiega will push an updated version later tonight that is very close to your code @danielmarbach


Logic has been removed.


Since you are touching whitespace nearby, would you mind removing the tab character after the closing a element here please - it is my mistake.


Hmm maybe synapse is the problem. I will check somewhere else.


Great! Thanks a lot. This is it!
I will do my job ASAP :D


sneakquie, не слышал об этой функции, спасибо опробую.


Well....I learn something....rofl :)


This shouldn't have an `id` of `iphone5` (as the header above it does). We briefly talked about `id` but haven't really gone over them, but there should only be one `id` with any given name in the document (ie. at most 1 `id="bacon"`, `id=iphone5`, etc.).

Also I haven't talked about "copy and pasting" yet, but for the sake of these lessons, don't *ever* copy and paste – always type it out (repetition burns the knowledge into your brain and you'll remember how to do it later on if you type it every time, not the case if you copy and paste).


I used enums and an array of strings for easy reuse.


@kumpera This breaks the Windows build:

```
  CC       networking.lo
In file included from networking.c:11:0:
../../mono/utils/networking.h:14:23: fatal error: arpa/inet.h: No such file or directory
compilation terminated.
Makefile:772: recipe for target 'networking.lo' failed
make[4]: *** [networking.lo] Error 1
make[4]: Leaving directory '/cygdrive/c/projects/mono-817/mono/utils'
```


I'd have checked before merging but yeah, I'll go add it to the ignore list


Same as above.


I care less about it, but some folks believe it is important.


Line is too long. [135/100]


Line is too long. [129/100]


Sorry for my eventual consistency ;)


空格阿。。。


wtf ?


Could you please explain why this is needed?
I looked at the tests for this in the main repo, doesn't really have a good intent.

Also, have you tried running the tests, there seems to be a test for the code that is deleted https://github.com/trademob/restify-oauth2/blob/master/test/cc-unit.coffee#L349,
should be failing.


I don't know


Because it's not the same thing; I'll code this later today


I have a question: Wouldn't be it better to have one just condition? So instead of `or` add another `else if`:

```
else if (rawAsset is URLRequest)
    enqueueWithName(URLRequest(rawAsset).url);
```

And, where is the test? :smiley:


----是干啥的？


1. 提交信息不专业。。。。


Great to hear !

You want also quartz to be uploaded ? Just give me a link to the right English po file.
Thanks, Cheers, Attila


but, why put node_info if is kept empty here, and is not used later ?


自己有看过么？


return y[i] == null;


+1 thanks


@ZachOhara
You are aware that I had already done this in the commit before this one, right?


This looks so simple... Hats off!


So even in routed mode you are only using one connection to one random server.


Without this being buffered you could be getting a slow consumer.


Hi,

the if bracket content woule be almost the same so i guess we don't need another else if here.
By the way Daniel just merged it so you can give it a try ;)


Zwei Mal?


We should talk about code indentation as well. Remember when I drew that tree graph of the HTML element relationships (parent, child, sibling)? All siblings should have the same indentation (so everything inside body should be aligned) and child elements should be indented one level more than the parent (this is obvious, but sublime text does this automatically I believe).

The reason for this is functional more than aesthetic – its so the reader (myself, someone on your team, a stranger, or maybe even yourself in a year) can go back and quickly know where in that relationship graph a given HTML element is (is it a child or a sibling of element X). At first glance it looked like your `<h3>` tags were nested under your `<h2>` tag (and `<p>` tags within `<h3>` tags). Really though, they're not nested but all siblings of each other.

Your instinct to group them was good however, and as you already know we typically do this with `<div>` tags and their semantic derivatives (ie. `<article>`, `<section>`, `<header>`, `<footer>`, `<nav>`, etc.). We'll get more into grouping elements on Friday.


Nice touch!


How about naming this `shouldRebuildMissingCountsStoreAfterRecovery`?


I suggest to check ...options.route if is defined like on line 15 in previous version.
This is because old way works for me (I use PHPCR for entities) but new version generates error:
Key "route" for array with keys "label, code, sortable, placeholder, link_parameters, _sort_order" does not exist in SonataAdminBundle:CRUD:base_list_field.html.twig at line 13


I feel with you ... it's a pain. Maybe this one is of any help: http://nbviewer.ipython.org/github/bgruening/notebooks/blob/master/R/extract_all_dependencies_from_an_r_package.ipynb


@Walkman100 Thanks bro ;p


Ja, my bad, ich hätte die Zeile rauslöschen sollen, die hatte alex so drin. ich lösch sie mal :)


thx


sure


@traxair, I'm not seeing that variable in this commit's branch, or in the recent branches. Would you mind opening a new issue, including the output of `salt-cloud --versions-report`?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/9840/testReport/) for this commit**


Notizen aus Treffen:

- Mehr Bilder (wo sinnvoll). Im Moment ziemlich viele Textfolien.
  - z. B. SCADA-Screenshots (aus TKaC) für SCADA-Folie.
  - Mal schauen, ob’s ein gutes Bild der Zentrifugenhalle gibt (Überblick). Besonders beim zweiten Angriff („Lagerhalle von Waschmaschinen, die aus dem Schleudergang kommen“).
  - Diagramme aus Symantec- oder Langner-Dokumenten?
  - Exploittabelle nach Stuxnet 0.5-Dokument?
- CVE-2012-Folie ist Wall of Text. Runterbrechen, vielleicht auf Stichwörter.
  - Brauchen wir den CWE? Ist ja schön dass das ne Nummer hat, aber nötig? Eigentlich nicht. Vielleicht nur mündlich.
  - CVE und erstes SSA-Zitat sagen auch ziemlich ähnliches.
  - Vielleicht weg von wörtlichen Zitaten.
- Evtl. nach Tool suchen, das CVE-2010 gefunden hätte (Verifier). Aber nicht zu viel Mühe verschwenden.
- Zu CVE-2012: Wird Step7 überhaupt als sicherheitskritisch gesehen? Wird es unter diesem Gesichtspunkt entwickelt?
- Treffen nächste Woche optional (Rückmeldung per Mail).
- Hab vermutlich was vergessen.


메소드명은 영문소문자로 시작합니다.
또한, Adder라 하면 더하는사람? 덧셈자? 뭐 이런거같은데.. 메소드명은 동사로 합니다.
Calculator(계산기)가 add(더하다), subtract(빼다) 라는 행위를 하는게 논리적으로 맞겠지요
Adder는 클래스명에 어울리는 이름입니다.


나눗셈은 int형반환보단.. 실수형으로 반환해야겠죠?


Hi, how do I translate into my language, Portuguese of Brazil?
Thank U.


good


콤마(,)로 구분하지말고 행으로 나눠주세요


if main_object and main_object._name == 'ir.ui.view':


I was unable to sync when I made the commit, because I wasn't currently in
the group. I didn't see your change until I had already pushed mine


Is it possible to push unmatched value here?


여기부터 아래는..무슨 의미를 잘 모르겠네요


Woops - my bad!


Which variable? It wasn't clear but vm_name is actually the name I entered for my vm. However, I'm investigating the issue, I'll post a pull request when I'm sure.


DB table `quest_objectives` in Type = 8

More info:
https://github.com/TrinityCore/TrinityCore/blob/a782515246d5583a4c0e5cc8834133d425920b56/src/server/game/Quests/QuestDef.h#L187


:clap:


It looks like there is: https://github.com/magfest/ubersystem/blob/master/uber/templates/registration/register.html#L96

Admittedly, this is just in the code and I'm not in a position to easily check how the page actually renders, but I think it is there.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/9841/testReport/) for this commit**


PHP files should not end in `?>` and should have a blank like at the end


GCC reports:
```
lib/suppressions.cpp:268:117: warning: unused parameter ‘unusedFunctionChecking’ [-Wunused-parameter]
 std::list<Suppressions::SuppressionEntry> Suppressions::getUnmatchedLocalSuppressions(const std::string &file, bool unusedFunctionChecking) const
                                                                                ^
```


Note that line 526 "#ifdef RB_HAVE_WAIT_FOR_SINGLE_FD" of ext/em.cpp should read:

#ifdef HAVE_RB_WAIT_FOR_SINGLE_FD


The quests not remove reputation, the objective is valid by min reputation and max reputation, but never is removed only is rewarded.

The reputation objectives now are in `quest_objectives` table by Type: 6 (min reputation) and 7 (max reputation).

More info:
https://github.com/TrinityCore/TrinityCore/blob/a782515246d5583a4c0e5cc8834133d425920b56/src/server/game/Quests/QuestDef.h#L187


@Allactaga This change is causing a debatable unexpected behavior. Dateutil parse method treats '15 minute' as '00:15' and returns a datetime with current date and 00 hour 15 minute time. While freshness parser for the same string returns a 15 minutes old date. The tests so far have been catering for the freshness parser behavior. Please suggest shall we modify tests in favor of dateutil  for such string or rather think of a workaround to force freshness parser behavior.


Let's try together then... ;)


Ack on this one.


This has to be ```!world.Map.Contains(world.Map.CellContaining(position))```


PLEASE!


Test?


I did this to avoid sleeping while waiting for selenium to boot: https://github.com/redCOMPONENT-COM/redCORE/blob/develop/.travis.yml#L31 see the script at: https://github.com/redCOMPONENT-COM/redCORE/blob/develop/.travis/wait4selenium.sh


Does this handle non-MRI platforms? I recall they had to be specified explicitly.


`vm_name` was the variable in question; thanks for clarifying.

If you end up opening an issue, a (sanitized) copy of your profile and provider config, as well as the command that you were using, would be also helpful.


مساء الخير


This is our new world


Yeah. Will remove this. I already did a grep for "GetBy" to identify more of this "obtain the original for a copy" stuff but had no time so far to analyze/fix. All these masterpieces of software engineering can now be removed, but if they stay in (for now) they do not harm, because functionally they all should by noops now.


이렇게 멤버변수를 생성한 이유가 있나요? 의미없는 변수로 보여집니다.


변수이름을 고민하고 지어주시길 바랍니다.
a, b 라는건 어떤뜻을 가진 변수인가요?


These filechoosers were latter removed. Alternative new output files for Gcoder and Collapse options are now based on the input files (in case its a conversion) or on the main output file (in case its a concatenation) with the addition of a specific suffix in each case (e.g. "_collapsed" for Collapse option)


some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3044
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3049
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3043
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3035
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3035
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3038
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3031
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3026
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2703
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2701
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1539
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1531


Jajajaja de pegados.


HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


탭간격 맞춰주세요. 가독성이 떨어집니다.
컨트롤+시프트+F.. 기억 나실까요


Heh, and now I just noticed "the the". :)


This should be `title: title` (`title` var declared is unused here).


I agree, this is too much of a change. Do we need Persian support at the moment? If not then let's create an issue and move on for now.


이런식으로 내부변수 i++과 같은게 있다면.. while문보단 for문을 쓰는게 좀더 깔끔합니다.


This should be `x-csrftoken`.


```ruby
def available?
  has_no_memberships? ||
    has_non_billable_membership? ||
    has_memberships_or_projects_with_end_date?
end


의미없는 TODO주석이네요


Actually, it was right based on http://en.wikipedia.org/wiki/Cross-site_request_forgery


This fixes a problem with a PDF having the following value for CIDSystemInfo: {u'Ordering': 'Identity\x00\x00', u'Supplement': 0, u'Registry': 'PDFXC30'}

This was the full spec, fwiw:
```
{u'BaseFont': /BVJJVL+ArialMT, u'Subtype': /Type0, u'DescendantFonts': [{u'FontDescriptor': <PDFObjRef:10>, u'BaseFont': /BVJJVL+ArialMT, u'CIDSystemInfo': {u'Ordering': 'Identity\x00\x00', u'Supplement': 0, u'Registry': 'PDFXC30'}, u'Subtype': /CIDFontType2, u'W': [404, [604]], u'DW': 1000, u'Type': /Font}], u'Type': /Font, u'Encoding': /Identity-H}
```


This is not the right way to use `$_LP_SHELL_zsh`.

Do this instead:

    if $_LP_SHELL_zsh; then


function보단.. 연산자라는 뜻을 가진 단어가 좀더 어울릴것같습니다.


Oops, wrong commit message, it's not enabled for "make deps app".


opa ! =D  !

 é isso mesmo !

 topa ?


This looks wrong. This case is for userland programs executing the brk instruction.


maybe smth like this?
```ruby
def run!
  @user.update_attributes(available: available?, available_since: @available_since)
end


세미콜론(;)이 있다면, 다음행으로 넘겨주시길 바랍니다.
위와같은 코드는 가독성이 떨어집니다.


This change results in a test failure, can you take a look at it?


saw approval from steveklabnik
at https://github.com/vrana/rust/commit/7846eb6aaae9094694e4aa3effec214ad3cd601b


merging 8 batched pull requests into batch


status: {"merge_sha": "2a8cb678e61e91c160d80794b5fdd723d0d4211c", "rollup_pulls": [[20669, "7846eb6aaae9094694e4aa3effec214ad3cd601b"], [20670, "94c170e366607871174119ccd41c7b2ee5ba7ea6"], [20672, "83d01cc5ae6029276788ddc94af4488671a9fcd3"], [20674, "e3b7fedc20fcfe54e0ddb285fc6e6c6e862b3279"], [20675, "938a705ff17abaa502f2617b4a733f7352007097"], [20679, "3ece65700413a74cb3caa0a045a959c843ee6b48"], [20682, "47c9cc44dc05a3e67947f827786c367960a0b61e"], [20689, "6c7291ece47d0b263a3d3dadbda2ff7d034f705f"]]}


```ruby
dates = [available_by_membership, available_by_project].reject(&:blank?)


Testing rollup candidate = 2a8cb67

**Successful merges:**

- #20669 Use a better word in the guide (vrana/patch-1 = 7846eb6aaae9094694e4aa3effec214ad3cd601b)
- #20670 Fix type annotation in guide (vrana/patch-2 = 94c170e366607871174119ccd41c7b2ee5ba7ea6)
- #20672 Fix a typo in guide (vrana/patch-3 = 83d01cc5ae6029276788ddc94af4488671a9fcd3)
- #20674 Fix misspelled comments. (jbcrail/fix-misspelled-comments = e3b7fedc20fcfe54e0ddb285fc6e6c6e862b3279)
- #20675 Fix misspelled comments in tests. (jbcrail/fix-test-comments = 938a705ff17abaa502f2617b4a733f7352007097)
- #20679 unused variable 'i' (geekcraik/master = 3ece65700413a74cb3caa0a045a959c843ee6b48)
- #20682 Fix JS error (sfackler/fix-impls = 47c9cc44dc05a3e67947f827786c367960a0b61e)
- #20689 Update editor syntax files for isize/usize. (huonw/editor-_size = 6c7291ece47d0b263a3d3dadbda2ff7d034f705f)


if, for, while 등을 사용할시엔 항상 대괄호 {    } 를 통해 범위를 지정해주세요.
비록 범위가 한줄이라도 말이죠.
그렇게 해야 코드를 볼때 햇갈리지않습니다.


I wondered about that but didn't have the full context. Perhaps both of these functions ought to have a comment preceding them explaining their use.


메소드명 동사형으로 지어주세요


thought about it, but `@available_since` is set always, but I want to pass it to user only if `available?` is true


Loise


위의 2줄은 1줄로 줄일수있겠죠?


r=cmr


@PEMapModder There is a problem with that?


かんぺきです！


The problem with the uberjar exporting com.google.protobuf is split-package occurring if another bundle exporting the same package is installed in the OSGi container.


여기서 변수는 '점수'가 되겠죠.
score가 적합하지 않을까요


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/9842/testReport/) for this commit**


Thanks for spotting this.


You saved my Day :-)


No need to revert this. I could fix all the issue and I will open a PR against the ´hello repository in a few minutes.


Shouldn't the `writel` be *inside* the `for_each_set_bit` loop? Otherwise it looks like only the last interrupt (i.e. last value of `i`) will be cleared? (alternatively I guess you could build up a bitmask inside the loop, if you wanted to keep just one `writel` call per bank)


foreach문에 대해서 공부해보고 적용해보시길 바랍니다.
수업때도 언급한적이 있습니다.


TeamCity Hyrise :: Unit Tests [Build 57](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=65&buildTypeId=Hyrise_UnitTests) is now running


Rename modal service to confirm service. Fix spaces!!!


I don't see why ounit is not a depopt. I just built, installed, and used uri fine without ounit.


Yeah, compat w/ older CMake's was the idea, specifically versions between 2.8.8 (current required minimum) and 2.8.12, if I'm correctly understanding when {CMAKE_}MACOSX_RPATH got introduced.


i have added that code into my kernel but only 1 core stays online and not 2 like set here:
#define DEFAULT_MIN_CORES_ONLINE 2
Is that a problem on my kernel or is there an error in the code ?


DONE in 82944a2cfae23b3cab194117e5c745c50e28f93a


Doe eens encoden.


WebCLDevice has different logic from other WebCLObject.
How do you track it and release?


Always group methods with types - type, method, type method.

If you want to parameterize context not found with the name you need to add a method `IsContextNotFound` - at that point you can make ErrContextNotFound private.


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/3243/console to remedy the problem.


do you want me to merge my changes? ;)


This should be

    var ErrNoContext = errors.New("no context chosen")

otherwise someone can't do an equality test.


This code is in a loop. It should actually be:

```csharp
if (y[i] == null)
    continue;
else
    return false;
```

which simplifies to:

```csharp
if (y[i] != null)
    return false;
```


TeamCity Hyrise :: Unit Tests [Build 57](http://ares.epic.hpi.uni-potsdam.de/hyrise-build/viewLog.html?buildId=65&buildTypeId=Hyrise_UnitTests) outcome was **SUCCESS**
Summary: Running Build time: 00:03:22


이 클래스를 통해 생성된 객체를..한번만 쓰고 버릴게 아니라면
위와같은 멤버변수는 지양해주시길 바랍니다.

가장 좋은건 input / output이 명확하게 떨어지는 메소드입니다.


Fix spaces in confirm template. Tabs are evil.


哥，你现在也写lisp代码了？ emacs好用吗


aaaaaaaaaaaaaaaaaaaaaaaaaaaa


Why don't you Vector<OwnPtr<ImageBuffer>> m_buffers?


Non traduit.


Please, so we can see how it looks like


Osea que llenase los csv que estaban vacíos y borraste los yml @ovnicraft ?


No traduit.


@ferencek are you interested in this commit ?


Should -stoponerror STOP for any !!FAILED!! run ?


You are quite right. Pushed a fix.


This actually makes window scroll to the top if textarea is in absolutely positioned div.


HVACSizingSim_CoincidentPlant (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1094 of 1094 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)


This breaks access for normal users to shared patients (and for admins as well).


Ich bin der wahre Coralover!!!


주석은 나쁘지않습니다.
다만, 메소드명으로 의미전달이 명확하다면 추가적으로 주석을 달 필요가 없습니다.
가장 좋은 주석은 주석이 필요없는 코드가 되도록 이름을 짓는겁니다.


At least with -random, I think it should stop as soon as any kind of problem occurs, so that you have all the information you need (incl. the randomized word list file) to reproduce the problem in the command line.


Why refer them using RefPtr? could you explain?


여기에 있는 2줄을 1줄로 바꿀수 있겠죠?


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=f66490f1308b4565814fc456fb8bdcd2
 plone-5.0-python-2.7 [SUCCESS]


WebCLDeviceList has same member. WebCLDevice's refcount can be zero?


Is this a fix for an undocumented bug here?


@tjvantoll Is the block level element requirement just so that any items that contain a submenu will be valid markup? If so, does that mean I would still be able to use, e.g., spans if I know my menu won't have submenus?


test


C'est pas comme ça !


how about moving it to WebCLObjectImpl? It looks easy to make a mistake


hexa?


각 변수들 이름을 의미있게 지어주시길 바랍니다.


Lol, I guess that explains why it was the mouse sprite which was modified


README.md needs to be adjusted as well if merged


Why don't WeakPtr?


- Added scenarios to verify ARM Type is not visible by default
- Default ARM Type is 5-1
- Changing different ARM Types
- FHA and VA are disabled when Adjustable loan is selected


@NeoRaider I need a bit clarification here: do you mean I should call `fastd_socket_close` inside `fastd_socket_android_protect` (and probably return `false` I guess)? In this case what is the caller (`fastd_peer_reset_socket`) supposed to do next up? should it allocate another socket and try `fastd_socket_android_protect` again? or just call `exit_error`?

A side note, this "protect" action at Android side is unlikely to fail as long as the socket handle is valid.


Awesome !


모든 반복문, 조건문은 무조건 대괄호 {   }를 통해 범위를 정해주시길 바랍니다.
단 한줄이라도 범위를 지정해야 코드를 읽을때 햇갈리지 않습니다.


it should do releaseAll()


develop (Myoldmopar) - 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)


1: M100 needs more reviewers
1: M111 unrecognized component name: bibupload


@miltonlab si ahora los csv esta en el directorio data, y se cargan al instalar el modulo.
Es importante ver que en el campo description se cargo el codigo, esto ayuda para la busqueda y visualizacion desde la factura, ahora en el detalle de factura solo se muestra el codigo (metodo name_get por defecto del sistema) para un mejor uso.