You are not using your callback at all. I think you tried but you spelled it wrong :) and even if you did spell it correctly you didnt pass it anything. also you don't you $names at all either.

Lets link up on this one since you look lost here.

So you will have something to look at, I think Jason was wanting something like this here:

```
function myfunction($string, callable $callback){
     return $callback($string);
}

echo myfunction(clean_then_call(' jAsOn hUnTeR', 'md5'), function($string){
	// we will store this in the db for later lookup
	$saltHash = md5(rand(1,100));
	// add saltHash to my md5 name and md5 them together
	return md5($saltHash . $string);
});

/*
This portion is the closure:
function($string){
	// we will store this in the db for later lookup
	$saltHash = md5(rand(1,100));
	// add saltHash to my md5 name and md5 them together
	return md5($saltHash . $string);
}
*/
```


NB: output in CSV format for further use in ML algorithm


I researched it and most of the conflicts centered around the Kashmir region, not the other regions of Pakistan or India.


Se debe corregir el uso de "respuesta" a "respuesta.get"


This commit has added '+' signs to lines 41,42 &43, and completely broken my site :(


A better solution would be to have a single default account - with overrides per repo.
Similar to how Jenkins does it's SCM configuration.

Requiring everyone who wants to create a dashboard knowing a system account credentials isn't great.
Also having the ability to reset password's in mongo with an Upsert is definitely something I can see people liking.

Ideally I would have also dockerized this and have it connect directly to mongo, but I just needed solution today to get it working and document it.


Why is root (Root | None)? Why not just Root? With this type, the Main actor never knows if it has a Root, so it can't really do anything with it.

I tried changing it to just Root and it messed up the httpget test, but I don't understand why.


i think so. If one put happens in the current txn, it will increase the `currentRev.sub`, so `rrev` will be increased in the below 3 lines(if statement).


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (764 of 764 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-10.43%25-green.svg)


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (764 of 764 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-18.2%25-green.svg)


protected fieldを使っているのカプセル化が不十分なようで気になるので，onLoadNextPageを親に移して，その中でmPage++をするようにしてはどうでしょうか．Observableを作るメソッドはページ番号を引数に取るようにすればいいと思います


ok


because it removes check and `delete` should always succeed, there is no need to return whether `delete` succeeds and count the success number.


This module would benefit from a comment here


@Kiboom
띄어쓰기 두 칸 말고, padding으로 해결할 수도 있지 않을까요?
밑에 게시의 경우에도, 띄어쓰기 대신에 letter-spacing 속성으로 해결할 수 있을 거 같아요.


delete for test


test


class명에 대해 고민해봐야겠다.
[참고](https://css-tricks.com/semantic-class-names/).


I don't think this is right.  How does OpenCV know the dimensions of the image?


I think if there is 0 post for a month, the month is not even shown.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 54](http://teamcity.otenv.com/viewLog.html?buildId=1494403&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 54](http://teamcity.otenv.com/viewLog.html?buildId=1494403&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 36; exit code 1 Build time: 0:0:0


Probably not a good idea to read the same image over and over again.


oh. ok. you are probably right. is this case covered by a test?


Can you send some examples, not sure what you mean.
On Sep 4, 2015 00:04, "Sreekanth S" <notifications@github.com> wrote:

> Any reason for removing this line in the example here?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/websanova/js-url/commit/559e98b9f4c5462b9b5a21fd66867b1c42a90d4b#commitcomment-13052608>
> .
>


@walsh2000 - the thing is, that causes a regression with the toolbar. see https://github.com/jessesquires/JSQMessagesViewController/pull/1175#issuecomment-137346318


A better name might be something like `FileCaptureSource`.  Similarly, we would have `WebcamCaptureSource`, `NetworkStreamCaptureSource`, etc.  `FileInputOperation` doesn't say what type of file we're talking about, and "operation" implies that it's doing some kind of transformation.


Also maybe make a `edu.wpi.grip.capturesources` package.


`UnsupportedOperationException` probably is a better fit... which extends `RuntimeException`.


ok. sounds good. is this covered by a test case?


Another random idea I had that we might want to think about is making `CapturesSource` a separate interface from `Operation` instead of hacking this concept into input and output sockets.  Then, we make `Step` abstract or an interface, and have both an `OperationStep` class that performs an operation and a `CaptureSourceStep` class that sends a new `SocketChangedEvent` whenever there's a new frame available or an image changes or whatever.


Thanks @mmottl for taking care of this.


Hey Pat, these styles were in the starter code from the assignment. There was no instruction remove anything that was not applicable and as the instructions stated it would be worked on over the next few weeks, I didn't want to delete things that might be applicable (such as meta tags etc as well).


This code doesn't work as written.  Will be walking this change back for now....


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Once again this application of background image was taken from the starter code.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 55](http://teamcity.otenv.com/viewLog.html?buildId=1494416&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 55](http://teamcity.otenv.com/viewLog.html?buildId=1494416&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 37; exit code 1 Build time: 0:0:0


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 507](https://teamcity.labkey.org/viewLog.html?buildId=270071&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 30, passed: 9 Build time: 00:22:15
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

##### there are 20 more failed tests, see build details
```


great! please delete this branch after u have learned how to use GitHub!


So this is all organized by sub deps of other deps which is actually nicer visually so you can see what imports what


ah yeah weird that I didn't pick that up! thanks pat.


remove `use Config;`. and use `$^O` instead.


iteration (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2295 of 2343 tests passed)

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


Thanks for letting me know, the starter code comes from GA in New York which I assumed would have been following Best Practice.  My bad though should have given it the once over.  It's good that you have followed best practice in your portion of the code.  Don't be afraid to flag spots where the code smells a bit and/or to refactor based on Best Practice.  The web is a wash with bad code and quite often cleaning up is necessary.

Based on this I'll bring you'd score an extra point in the rubric bringing it up to 83.33%.


as one of an expirenced dev. come back and say that when you can do more then rip off cm tree's and kernel by picking in a metric shit ton of caf commits over it to make it look all new and such. until then politely fuck off @h2o64


Yeah no doubt- the web is a wash with a tonne of incorrect information in any field! Cool thanks.


이렇게 cid를 노출시켜도 되는걸까?


This was moved out to `gyp.js`


Added this so `node-gyp` builds in correct mode


They aren't the same advice. "minimum you need" implies that there is a static minimum. "start with what you need now" implies that needs change over time. Also, the use of "minimum" implies that a better decision uses less of these options - where in fact that is almost never the case.

The most likely readers of this article are people who are new at Redis. They may even be new at setting up new operational systems (a lot of posters on the redis-db mailing list certainly are new to all of this), and may not understand what really is a reasonable minimum.


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


For supporting `npm i --build-from-source --debug`


* MainServlet에서 `request.setAttribute("cid", cid); ` 해서 넘겨주는 것과,
main.jsp에서 `getParameter`(`${param.cid}`) 하는 것 중 어느 게 더 효율적일까?
* 지금은 `forward`를 통해 main.jsp에서도 url에서 cid를 그대로 노출시키니까 `getParameter`가 가능하지만,
facebook.com/hello.heesu처럼 id를 직접 노출시키지 않으려면 `setAttribute`가 필요하겠다.


`--debug` should yield `build/Debug` output folder


iteration (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Close [test](http://a1c9a437.ngrok.io/w/SnKPAov9ePNu9XRTeqnt2UCE): test close


@Exitare Hello！ Can you tell me your email address or Skype account number? I need your help, thank you.


Eldrazi spawn tokens split into 3 broods, referencing Psithief/Cockatrice-tokens/issues/7.


restaurantの間違ってました。
keyはindexと同義。
indexについてはこっち（ http://www.dbonline.jp/sqlite/index/index1.html ）
ただindex張りすぎなきがするからちょい変えるかも


You don't know what you're doing, are you? :P


- ReplyDAO should be corrected, too.
- redirection CreateReplyServlet should be corrected, too.


I changed it so that all the `ps_capt`, `ps_sent`, and `ps_netdrop` are defined if and only if both `_WIN32` and `HAVE_REMOTE` are defined, so it should work the same way WinPcap does.


Don't use JSON.toJsonNode. Instead, use JsonNodeFactory.instance.numberNode() or textNode()/


Don't use toJsonNode.


1579840-GroundTemps (mmatts) - x86_64-MacOS-10.9-clang: OK (1811 of 1827 tests passed)

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


this code has no effect


HTML is invalid (missing `<title>` element).


Wait hold up, are you calling yourself an experienced dev..!? 😮 Also, you're holding quite the double standard by calling his work kang, but also kanging yourself.


It's not just a wording difference, it is the tone and implication of the words used. Yours says that there was a purpose that Redis was designed for (that you don't discuss at all), implying either that they should already know these reasons or you will explain them shortly, and that keeping daily backups for 1-2 weeks is the right answer if you're using Redis as intended.

The thing is, Redis has gone well beyond what Salvatore originally intended, and *most* people aren't using Redis for what it was originally designed.

What I wrote doesn't make any implications about what Redis was designed for, but still gives the same "back it up daily for a couple weeks" as good starting advice.

I'm explaining this because if you are not clear in what you say and how you say it, then readers will be confused. Confused readers don't come back and read more. That may seem like hyperbole, but I've been writing for a long time, and this is what my editors have told me.


More expirenced then any of you. and I'm in no way holding a double standard. I'm not kanging. so kindly go fuck your self @YoshiShaPow


Most of the methods inherited from LightblueResponse is meaningless for this class. Maybe we can make this not extend LightblueResponse, and overload the client APIs instead of handling bulk with the existing implementation. Something like

BulkResponse data(BulklRequest)

Construct BulkResponse with a BulkRequest, not List<>. Clone the list from BulkRequest, i.e. requests is a final uninitialized var, and you need to clone, because you take a snapshot of the request list once request is submitted. this.requests=reqs is broken.


Added a new block to the MinecraftClient.ini file.
\[ChatBotMessages\]
herochatmessagesenabled=true
unknownpluginchatmessagesone=true

Edited the ini creation to reflect this.  Will comment INI file to include the "regex" of the chat message formating in question.


why the NOTHING here at the end?  It seems like a bad trend to write NOTHING instead of just leaving it out entirely.


Nice!


Yes. TestStoreDelete one.


migrate gradle to mvn? Why?


so cool


涨姿势了……


Should have specified that this applies only to `mavsummarize.py` in the commit message.


Why? BTW, how do you know we exist???


Oh cute. you think I copy and pasted. any wanna be kernel dev with half a brain knows how to do simple shit like this. @h2o64


So this makes the answer to any question which is otherwise  unanswerable due to instructor error sqrt(-1)?


Насчет SearchScope, по-моему, ты что-то не то написал в примере Не так должно быть совсем, а просто "SearchScope": "SearchScopeIncoming", либо, на худой конец, "SearchScope": 0. А как на самом деле - это еще проверить нужно


locking the mutex is unnecessary here


Keep believing that pal. Maybe your "experince" will be recognized one day and people will actually start using your work.


我也是第一次用.... 现学的...


Hmm. Not sure how I missed that. hope that didn't cause many headaches.


from my point of view your usage of m_ut in the Watcher class is correct - you have to have a Test context to make an assertion


sdhidfhv


pls add http:// in front of the url


Also. I have been doing this shit much longer then any of you have. I don't really think you get the right to call my work out as kang when you have all been part of it at one point or another. So if you wanna be an internet tough guy and try to call me out for my supposed kang. have fun. ask anybody though. my commits always have proper authorship and this one is no exception. if you'd like to talk about this in a message. then by all means hit me up on hangouts. I'd prefer my commit comments not be riddled with comments from the low life wanna be's of the android world.


Is jumbo.[ch] the right place for this function?


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


The root of this change is that kvindex should always return existing keys, which is tested at TestIndexGet and TestKeyIndexGet. For functional test, TestKVDeleteMultipleTimes covers it.


That's the intention, yes.  It also prints something to the console, so hopefully the instructor notices...


LGTM, +1.


102687702-Schedule-Unit-Tests (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (2309 of 2346 tests passed)

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


* DEPENDENCE => DEPENDENCIES


If conn is None here, something went seriously wrong. I'd rather not 'if' around it, but root cause it.  Are you seeing a genuine failure here, or just trying to be careful because the assignment is inside a try: block and could potentially fail, leaving it unset?


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1294 of 1294 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-70.0%25-green.svg)


woops


As we discussed in our meeting, when a user has lots of wikis then this would get slow and you may want to consider using `user.wikis.update_all(private: nil)` to do it more efficiently.


@iflamed Can you take a look at this test you wrote a while back? It's `xit`ed out because I couldn't find a way to make it work. Thanks!


If we're sticking with the simple Django pages, then we probably shouldn't worry about AJAX. The management interface is mainly only seen by us, so it doesn't have to look too pretty.

The last time I had to touch AJAX it looked cool but it was very manual and tedious.

The `scoreboard` and `teamcentral` might be the only places we need anything fancy and flashy, and I understand JQuery sits on top of AJAX to make it easier to use, though nowadays I believe there are even fancier technologies that are easier to use than JQuery. Is this true? (I haven't touched JQuery at all yet.)


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1294 of 1294 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-70.19%25-green.svg)


just testing it out


* Phần này là anh đang làm search, anh nên đặt tên tham số là q. Ví dụ anh muốn searh từ khòa là "my pham" thì url sẽ là /api/v1/shops?q=my+pham


@josyoun You are welcome!


* skip với limit đặt ở cuối này có hiệu quả ko anh?


This repo is a great idea.


I have no idea how these two lines would prevent the error `Unable to load class Drush\Sql\Sql`.  I'm seeing that error also, but am lost as to how to fix it.


ok, I will check it later.

> On 2015年9月4日, at 下午12:20, Mohsen Azimi <notifications@github.com> wrote:
>
> @iflamed Can you take a look at this test you wrote a while back? It's xited out because I couldn't find a way to make it work. Thanks!
>
> —
> Reply to this email directly or view it on GitHub.
>


He can look at our profiles...


Here Lies Bukkit (as we've known it for 4 years).
December 21st, 2010 to August 17th, 2014
Rest In Peace


Oops.  My comment was too short (and ambiguous).

What I meant was just asking if you wanted to simplify the query from "select * from (VALUES 1)" to just "VALUES 1" (since the subquery isn't necessary to use VALUES).

(Yes, it's fine that it tests with only one data type since this test is just testing the shading.)


Hi, my two cents about the utils.py, get_rdataset method:
- it won't allow user to plugin proxy auth on the fly when working in an proxy'ed environment, might be nice to extend the method to allow adding proxy hander (workaround: urlib2 is able to read http_proxy from  the environment variables list, which can help to get around the http connection error)
- https is somehow not supported well, as the default SSL version will not get an "connection refused" error, after changed that to TSL1, the error changed to  certificate verification failed(monkey patch: override the method to create unverified content by default, suggestion (to get it more user-friendly) would be to give it a few more iterations by changing the SSL version, if still fails, throw in unverified http content to let it continue)


- The code should be put into variable as String type, not Number type.
- Code had better to be written as static value such as 'INPUT_FAILED_CODE'.
  - The static value should be written in shared file because other class can use same static value.
- Description should be also written in shared file as static value.
- It's better to create new class for getting message, maybe.


why not just ```IOTJS_ASSERT(0)``` ? Is there any particular reason for this?


@SirFaizdat please don't comment on this commit, it notifies everyone who follows the Bukkit project.


Why does he care???? :P BTW, IK what I am doing.


I'm curious why you added HealthBarPal: in main.asm when nothing appears to call it. It seems to be nearly identical to the changes you made in engine/battle/14.asm


- The code should be put into variable as String type, not Number type.
- Code had better to be written as static value such as 'CODE_INPUT_FAILED'.
  - The static value should be written in shared file because other class can use same static value.
- Description should be also written in shared file as static value.
- It's better to create new class for getting description, maybe.


* Tạo shop mới thành công thì anh trả về kiểu json là cái shop mà vừa được tạo


Agreed that only `scoreboard` and `teamcentral` might need AJAX.  JQuery simplifies it somewhat.  I don't know about more newfangled things.  Would vote to stick to more well-known, widely used add-ons, only as needed.


Two bugs I have noticed.
The tilda key is not working. It's replaced by >.

Secondly, after every reboot, the Function key priority changes. Meaning if one one bootup, the Function keys correspond to F1/F2/F3, etc, after next reboot it will get swapped and become the special keys.

I request you to fix them.


Thanks sir, I will fix it.


* Tạo shop mới thành công thì anh trả về kiểu json là cái shop mà vừa được tạo, ko nên trả về next vì khi đó client sẽ ko biết kq trả về ntn


It is, I derped and forgot I left that in main.


* Ngắn gọn hơn: var newShop = new Shop(shop);


I thought that was probably the case, but just making sure I wasn't overlooking something.


* Trả về shop mới được tạo


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-MacOS-10.9-clang: OK (1815 of 1816 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)


* {_id: id} => { _id: id }. Nên để space trước và sau dấu {}


* DEPENDENCE => DEPENDENCIES


Did we lose any error detection worth having in dropping the JSON serialization-deserialization cycle here?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 508](https://teamcity.labkey.org/viewLog.html?buildId=270078&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 28, passed: 9 Build time: 00:21:55
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFilters: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBasic: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisBasic: <no details avaliable>

##### there are 18 more failed tests, see build details
```


* Remove thẻ tab băng 2 space


This revision is clean.


It's used in the Gen-II hack but I decided after I put it in that it might be more appropriate directly in InitBattleVariables.


* FIND_NOT_SHOP => SHOP_NOT_FOUND
* SHOP_NAME_IS_EMPTY => SHOPNAME_IS_REQUIRED


atv open


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


This should hold over until i rewrite the IO system + remove unsafe code dependencies.


- Variable names such as $k, $v, $e are not good. Other one can't understand these variables being used for, when they see these at first sight.
- It's better to create new shared utility class for getting and putting messages into array. If you created the class, it can be used in other classe and you don't have to write same long code in other class.


I meant "Yes." here.

We don't use Github for code review. I'm probably the only one who sees these.


yeah I'm assuming the Gen 2 hack did it that way to keep from shifting data around. It seems to work fine this way though. I also changed ld a, $00 with xor a, since it does the same thing but only uses 1 byte instead of 2 when compiled. Not a big deal but hey, every little bit helps and whatnot. Thanks man.


- Read my above comment. It's a same.


File a bug or comment on the original code review (not here, where basically nobody sees it).


- The response is included many users, right? If it's so, the variable name should be plural like '$users'.


1579840-GroundTemps (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1817 of 1833 tests passed)

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


수고하셨습니다.


TeamCity ReOpenLDAP :: devel-2.5 [Build 109](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=331&buildTypeId=ReOpenLDAP_Devel25&guest=1) is now running


@nsarno I am using auth0 with this and for some reason because you pass the flag **verify=true** the token doesn't validate properly. Can you help me out with that.


This function above is just registered one user , so i think $user is better.


The shifting data thing is something I still can't fully grasp. I get it, just don't know when its important and when its safe.

-----Original Message-----
From: "TheFakeMateo" <notifications@github.com>
Sent: 9/3/2015 11:39 PM
To: "Teksid/RedPlusPlus" <RedPlusPlus@noreply.github.com>
Cc: "Teksid" <teksid304@gmail.com>
Subject: Re: [RedPlusPlus] Palette changes for EXP bar and party indicators(472a51d)

yeah I'm assuming the Gen 2 hack did it that way to keep from shifting data around. It seems to work fine this way though. I also changed ld a, $00 with xor a, since it does the same thing but only uses 1 byte instead of 2 when compiled. Not a big deal but hey, every little bit helps and whatnot. Thanks man.
—
Reply to this email directly or view it on GitHub.


Grate Job dude


@Denis090712 Еще не скоро, однако))


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1818 of 1818 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)


Ah, I see. Thanks for your response.


Oh, sorry I was busy last week...
Is this a paid channel?


FourPipeBeamAirTerminal (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1825 of 1826 tests passed)

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


```
*Main> tell [1]
"The list has one element: 1"
*Main> tell [True, False]
"The list has two elements: True and False"
*Main> tell [True, False, False]
"The list is long. The first two elements are: True and False"
*Main> tell "hoge"
"The list is long. The first two elements are: 'h' and 'o'"
```


Well with Danny's patch, he tried to avoid shifting thigns like map data and event data and pokemon data and stuff that old editors loaded, so the graphics patch would still be compatible with old editors for people who didn't want to use the disassembly.

With us, it isn't super important. It is in a few places though, like if we shift collision data around, the player will be "stuck" and unable to walk unless they fly or use teleport or something, so that isn't good. And if it shifts event data or level script data around, the player can also get stuck when they continue as well. But so long as we aren't in a bank with that sort of stuff, if probably doesn't matter too much if we shift data around, so I usually don't worry about it.

Another way it can mess things up is when working with wram. Adding new ones at the end for the EXP Bar is fine, but like if we added more Pokedex flags in the middle, it would mess things up when they tried to load their existing save, since the data was different now.

I probably still didn't explain it super great but that's at least a general idea.


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.29](http://pixelated.design/build/viewLog.html?buildId=110&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:08


Thanks for spotting that. :)


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.33](http://pixelated.design/build/viewLog.html?buildId=111&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 324 Build time: 00:00:11


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1821 of 1822 tests passed)

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


`_` で３つ以上の場合のパターンを全網羅している。


I guess Something is failing here for Travis build.


for future - code formatting, cleanups, refactorings should be in seperate commits to new functionality / fixes


Octree (DeadParrot) - 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)


martynbm explain what you mean? there is no repo for this either in Cyanopgenmod or AOSP so what do you mean
#Git clone /vendor/pittpatt/models/recognition/face.face.y0-y0-22-b-N/full_model.bin


This is terrible that melech aka moloch did this.


You needed to create a class to calculate interest. Please read the question carefully.


Perhaps check if the string has changed and then do another read if it has?


I have no idea. I'll have to look into this.


Since we say it's "not a resource", the PHP name seems the more sensible one to use, so I've changed this over.


FourPipeBeamAirTerminal (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)


It's not really a class, though, is it? it's an instance. I also fail to see how this provides any additional expressive value.


Is this ready for a PR?:santa:


This makes sense.


typos on lines 17 and 18


```, mask ``` is missing here, did you chop that off?


this should have been fixed #2


No, I don't even remember what I was doing here :|

On Fri, Sep 4, 2015, 10:29 AM Christoph Hermann <notifications@github.com>
wrote:

> Is this ready for a PR?[image: :santa:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hemanth/gh-contributors-table/commit/a22fb171b608f502c8a114252a987d74ca434cc5#commitcomment-13062260>
> .
>


:laughing: haha. okay then. I was just curious.


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


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


phan nay anh test rui. Ok ma em


Suppresses deprecation warnings (missing config.log_level for production environments).


Ahem, "functions"!


@DSPhat If the comments bother you, please consider pressing the Unsubscribe button at the bottom of the page. Thank you for your time.


yes, i knew. i fix it


WHO THE FLYING FUCK WAS USING A RAW `TBu.log.push`?  The fuck?


I think I agree looking at this... it's not really a use of this feature we want to support. :)


This actually broke the test:
https://wrench.mono-project.com/Wrench/ViewWorkTable.aspx?lane_id=33&host_id=8&command_id=816


@legoboy0215 No, you don't know what you're doing with `/** @var string blah blah blah */`. You don't know how they work. You are using it for the wrong purpose. You looked at another plugin, didn't you?


I'm also getting the old hash ATM.


Yeah, maybe you right, but Mono is also required to use Nuget, so I am not sure that it is optional.


Okay, one last comment (irrespective of any opaque identifier).

In the constructor all the properties should be set to the default values, with the jsdoc comments above them, in at the 'top level' of the function. Then *after* that, branch out with the `if` to either sow or set/restore the state.


:heart: nice!


So it sounds more or less like making sure things don't move from where the save file expects them to be. So backwards from what I'm used to, but it makes sense. A lot of these files I don't even know what bank they're in unless I play "follow the includes" down to main, is there something to simplify that?


OK, it seems like a good idea.


Right


A constant is not a variable! -_-#


not n-dimensional... 1-dimensional, length n
not call to range, it is a functional construct, see the code:
`````` Scala
  /** Returns an array containing values of a given function over a range of integer
   *  values starting from 0.
   *
   *  @param  n   The number of elements in the array
   *  @param  f   The function computing element values
   *  @return A traversable consisting of elements `f(0),f(1), ..., f(n - 1)`
   */
  def tabulate[T: ClassTag](n: Int)(f: Int => T): Array[T] = {
    val b = newBuilder[T]
    b.sizeHint(n)
    var i = 0
    while (i < n) {
      b += f(i)
      i += 1
    }
    b.result()
  }

``````


Yeah, many people still scared of passing functions.
As an old Mathematica/ML hand, it is nothing, but.


If possible, do not use spell lookup (find_spell) during runtime.
Instead add a spell_data_t pointer, or directly a timespan_t variable for the duration to the class, and use that in execute().

Oh and that sim -> log entry isn't necessary as well, as long as you call execute() of the base class in there, action_t::execute() will add enough log/debug messages..


The urlencode/urldecode was introduced by 3ee20ca1b62bff67fcf71adfcbef1affa8007dc8 to fix #188.
I agree that it shouldn't to that. The URI in the backend should not be urlencoded. For that purpose it's a simple string. Only when it's transferred it may need encoding which depends on the protocol thus needs to be done elsewhere...
If there is a problem with Google sync that needs to be fixed at a different place...
So I vote for removing the both line changes introduced by 3ee20ca1b62bff67fcf71adfcbef1affa8007dc8.


为啥改我的代码？


Maven libraries is easier to reference. If your launcher is using maven or
gradle, you just need to add a dependency. Also, I will push this library
to the maven central repository.

2015-09-04 12:06 GMT+08:00 Derek.CHAN <notifications@github.com>:

> migrate gradle to mvn? Why?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/to2mbn/JMCCC/commit/929f8db6e080e63fb03cd007a9816eac62af8380#commitcomment-13061897>
> .
>


It isn't so much that the save file needs to be able to find things. It's that when it loads things from the save file they need to still be in the same format. So if you make the save/load routine look for more pokedex flags than the person's save file has, you'll end up with problems. Or if it loads a collision pointer for the map you're saved on, and it doesn't point to the right data anymore, it's gonna have problems. It is kinda weird though, and it can be annoying, you're right.

As for finding which bank something is in, there are a couple of ways you could try it. Most of the vanilla routines have a comment beside them like ; 15:4000 or whatever. The first number is the bank and the second part is the address in that bank (for the vanilla rom, of course) so that's one way to usually tell if something is in the same bank. But finding the INCLUDE line is probably safest. For example, the Mart data may still have comments claiming it is in Bank 0 in RedPlusPlus, but they have been moved into the same banks as the other scripts on their respective maps, because I wanted/needed to free up space in Bank 0.


Виноват, надо было отдельный коммит на это сделать.


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (762 of 762 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-10.41%25-green.svg)


FourPipeBeamAirTerminal (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (765 of 766 tests passed)

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


Man! Where did you ideologic roots went?
Real bastards, should block they target on social media and then make a whole community angry, saying shit about them (#commnunismfuckyeah). I was wrong apparently, you aren't a kanger... You're just the oldest and most experienced (as you said) troublemaker (aka shit) in the whole world since 1622!


1579840-GroundTemps (mmatts) - i386-Windows-7-VisualStudio-12: OK (1817 of 1833 tests passed)

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


HelloKevin


Have you checked with hooks if powersuspend is really working? I don't see what sets/unsets powersuspend for lettuce.


asdfasdf


I would say it's often good style, it forces you to think through the branches more carefully.

Remember that __all__ functions/statements have a return value:

``````Scala
def sideEffectingFunction(): Unit = {
    distance = 10000
}
``````
...just some return "Unit" (i.e. don't return kind of)

``````Scala
for( i <- collection ) {do something here without return; }
   => Unit
``````


Is there such a warning? IDE didn't show anything. I'll check.


I dont know if i like it!


I'm talking to myself :+1:


:P That is php docs...
BTW, it is true IDK what it means too much. I forgot to add a experimental tag then :)


Check out @EvolSoft's code?


This happens on_rez always, no need to do it twice


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.573](http://teamcity.monogame.net/viewLog.html?buildId=10049&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 539, ignored: 6 Build time: 00:33:26


With this damage will need to add a check for rend damage and subtract spell shield amounts.


Ací s'està imprimint una línia en blanc. Es pot eliminar.


OK


Wozu?


WHOOPS forgot to revert this


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


You need to disable it in boardconfig too, otherwise it would throw no rule to make error ;)


Check out @PEMapModder, @shoghicp, and @Falkirks' code?


Teach me....


todo: figure out if this just checks IDs or if your name is this guy

if its the latter restrict to lings


Круто.

> пофиксить окна настроек

И менюхи "Обновить" и "Звук"


their "special" objectives dont work and cause runtimes, even on ninjas

top notch code


causes runtimes when randomgenned because the code is heavily snowflake and has to be handled in traitor gamemode code for some reason


Люблю, егда код упрощается.


Perfect use of the integration system. Well done! :smile_cat:


> И менюхи "Обновить" и "Звук"

Эти уже в процессе. Сейчас в скрипте аж две функции для создания меню (одна для постов, другая для всего остального) я их почти объединил в одну.


Double space detected.


?
/**
		* @param Player $player
		* @param int $seconds
		* @return bool
		*/


just look at all that copypasta dead


BTW I wouldn't recommend learning from @Evolsoft's code.


Trailing whitespace.


Why?


Looks like a smooth integration :) :+1:


I already documented the builder access hack. Any other ideas for what it
could have been used? In my last project we did some ugly hacks to get
access to GetAddressForMessageType. But this was never on the public API so
I wouldn't consider this worth documenting

>

--
*#NSBcon 2015 • The "All About NServiceBus" conference is back *
*• Learn more <http://go.particular.net/l/27442/2015-08-21/5d4wzp> →*


honestly I'm tempted to just remove force_wizard_objectives and have it just call this


Intentaria buscar noms per a aquestes seccions que, més que per com es veuen, indicaren què representen (`qui-som-home`, per exemple).


oh yeah also did some laws cleanup because aran wanted me to get rid of some copypasta in my emag stuff


I will fix all the things identified by you. n thanks for identifying the problematic parts in early stage.


just curious, do we still need the servlet-api in provided scope here?


Just use autocomplete-plus, it supports Sass now
On Sep 4, 2015 04:07, "mike3run" <notifications@github.com> wrote:

> I'm sorry but I can't get this to work like on the gif from the readme:(
> do you have any tips for a noob like me?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/saschagehlich/autocomplete-sass/commit/81f4493099fd087854360642b8b39a2dc8381f00#commitcomment-13061120>
> .
>


wahah


final! at your will :)


ps. we should put the Ninja fluff on fanfiction.net and see how long it takes someone to give it a review


Hauriem de ser consistents en la nomenclatura dels camps personalitats, i utilitzar sempre `_` o sempre `-`. Jo sóc partidari de `_`, ja que això permet convertir-los en variables (i així eliminar la necessitat de trucar a la funció `attribute`.


final :dancer:


L'atribut `alt` l'haurie d'obtenir també del contingut (probablement, el mateix que `slide.title`)


@NathanYukai Good catch, both fixes look good. Can you please rebase on master and add `mpmath>=0.19` to requirements.txt before making a pull request for this?


We should consider USS Version 2 of the API


Hi @ywang19, I noticed a couple more inaccuracies (not typos) in the User Guide. What's the preferred way to report them? Should I open a new issue?


Why do you think that Java is interpreted language? It's wrong, some details you can find here http://stackoverflow.com/questions/1326071/is-java-a-compiled-or-an-interpreted-programming-language


`<a href="/noticies" title="Més notícies" class="bt-basic">Més notícies</a>` es veu igual


I guess thats spring java config? have we used it yet at other places or should we, perspectively, refactor all our tests to use it? I'm just looking for consistency


@cannondalev2000 has been bricking people's devices since many years. @h2o64 and @YoshiShaPow You guys need to respect that!


This doesn't seem to be taking into account the preference?


Эм... без него действительно работает :) Я передаю в `liquibase` dataSource базы H2. И в этом источнике данных, как я полагаю:
`EmbeddedDatabaseBuilder` говорит что я создаю базу в памяти (т.е.  = `url:jdbc:h2:mem:db1;DB_CLOSE_DELAY=-1`), а `setType(EmbeddedDatabaseType.H2)` определяет тип базы (т.е. = `driver:org.h2.Driver`)


Why was the release dropped? Looks leaky.


Do we need to Unpower-Mock Tracee.class?


You don't have to do this. It's done in `super.activate();`.


По второму пункту не все так однозначно, в целом, нет прямой зависимости между тем сколько памяти потребляет процесс и тем как быстро она работает. Например, твоя реализация алгоритма сортировки может потреблять много памяти, но благодаря этому работать очень быстро. Или другой пример, за счет того, что ты держишь в кэше много объектов (занимаешь память) ты можешь очень быстро получать к ним доступ (чем если бы ты их вынимал каждый раз из БД) и скорость работы твоей программы повышается. Твой второй пункт я бы уточнил следующим образом _потому что задействуется дополнительный объем ресурсов (память, время) на работу самой JVM_


Good idea unsettling the variable.
I was however wondering if WorldGuard does offer a singleton instance getter.
Does it? In such case we should use that one instead.


testing again


Здесь я немного в замешательстве, мне нужно немного времени на изучение дополнительных деталей... :)


Looks good


In our coding style we use space after `for` and `if`. Please add spaces :smile:.


You're right! After all, switching from the old mercurial to git might messed up this old man... Bricking is just a little side effect at his scale..
We should redirect him to a git tutorial instead


I think that if you remove that, the p12 file will be generated every run because it's `der` representation contains a timestamp. Did you checked that if you converge twice, the file is not updated?


Sorry -.-'


Hello. I have rcd300 where i found serial rx and tx pins???


confused by the name - maybe "ensure" ... HasHighPriorityOrdering ?


Because it's early morning, and I'm stupid ... fixed by last commit :)


Could you re-add this section? Test fails because it expects at-least two tsd-s in config file, and first one having the 5431 port and second one having no port defined and defaulting to 4242.

yaml supports comments, so maybe add the "port: 4242" line as comment and write that it's the default.


Avoid indentation where possible. Do this instead:
`if (regions == null || regions.isEmpty()) continue;`


Code review approved!


something like `verify(backend, times(0)).putAll(Mockito.anyMap(String.class,String.class))` to actually verify that it is skipped?


i think the right thing wa to install this plugin using bower :8ball:


Use continue to avoid indentation here as well.


That did the trick.

And the documentation builds now. Thanks!


why hard coded employee details ?


да, этот атрибут определяет когда применять изменения. в данном случае действительно лишний :)


also `Ordered.LOWEST_PRECEDENCE`?


I can see that you are not implementing custom region flags. This implementation is better than nothing but I really liked your feature suggestion with custom flags a lot.

Could you not implement the custom flags instead?

If you decide to go for this simpler implementation there needs to be a configuration option to toggle the feature on and off completely. Maybe using an ID "exception set" as well to enable the feature for a few regions only or disable it for a few regions only.


Created methods for each function so that the code becomes more mantainable


hello, sorry for this I forgot to change it also here, radio has no RX/TX its for serial from computer!

t


這邊好像寫錯了
好像是算到description的字數了


as this is still a draft and i do not have the time to review it now i will postpone the integration of this single commit to next week. simply proceed on this branch if you want but let's use this branch only for this activity from now on.


@cannondalev2000 you are a dumbfuck, just so you know.
You call people out for kanging and when they call You out for doing the same you get mad and call them keyboard warriors.
Please m8 get a life and stop using got.


probably for the best.


This'll have to be -INF.


Lel :)) Sorry. Awayin mo ako pag may mali ako. Para masanay ako. haha. Nitpick mo yung PR ko. Di nako pupush master :)


Can it happen that the ReflectiveMethodInvocation, for some reason, already has an TPIC-HEADER userAttribute? If so, do we care?


Oh! Oh yes! This IS very working! I never checked normal press, that why I thought it were not working, long press was wrong!
Great thing, good work!
So I can tell that it is working on SGS4 with CM12
But: Swype button is needed to mark a word fast.
This feature at least I use often and is missing now.
Not possible to use the Dragon button?


Event OFPT_ERROR message received.
version=0x4, msg_type=0x1, msg_len=0x4c, xid=0x58ea90ef
 `-- msg_type: OFPT_ERROR(1)
OFPErrorMsg(type=0x4, code=0x9, data=0x4 ... 0x0)
 |-- type: OFPET_BAD_MATCH(4)
 |-- code: OFPBMC_BAD_PREREQ(9)
 `-- data: version=0x4, msg_type=0xe, msg_len=0x50, xid=0x58ea90ef
     `-- msg_type: OFPT_FLOW_MOD(14)

構造的には、メッセージの前半がOFPHeader,後半がOFPBodyのパースになるんですね。
ふと思ったのは前半の

Event OFPT_ERROR message received.
version=0x4, msg_type=0x1, msg_len=0x4c, xid=0x58ea90ef
 `-- msg_type: OFPT_ERROR(1)

ですが、「Event OFPT_ERROR message received.」が文頭にあるので
意味が重複する「 `-- msg_type: OFPT_ERROR(1)」は無くても成立する＆見栄えとして
スッキリしそうなきがしました。


not that css are so well organized but in the initial per page separation all this classes should be put in tip.css and not in submission.css. i will change this by integrating.


properties-файл решили удалить, я думаю, теперь нет необходимости указывать имя базы?


very nice :)


The pref is checked at the top of this method, and returns early.


To be honest, your "dev resume" does show a lot of experience in being butthurt, downright impertinent, and hypocritical. Oh, and a permanent ban from XDA is the icing on the cake. This "expirence" and "been doing this much longer than you guys" doesn't really reflect in your work. Be a dick, and others will rek you.


Теперь по-умолчанию меню открываются вправо вниз, но если не хватает места могут открыться влево и/или вверх.
![screenshot_20150904091454](https://cloud.githubusercontent.com/assets/1163958/9677668/82844c7e-52e5-11e5-9fab-5b58d3cca060.png)
![screenshot_20150904091623](https://cloud.githubusercontent.com/assets/1163958/9677674/a15adbe0-52e5-11e5-9a6e-0e3910230b9b.png)


could also use https://github.com/Charuru/lionbars


a scary message with also a typ0 is not a good thing to put i think.
le'ts eventually put and uniform this options with a label [Development Option] and if we want to avoid it to be used in production at all we should allow using it only if also -z is used.


Ммм, хотя нет. Для панели надо вернуть `fixed`, иначе меню уползает при скролле.


Alterations made for User service :


TeamCity ReOpenLDAP :: devel-2.5 [Build 109](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=331&buildTypeId=ReOpenLDAP_Devel25&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:34:20


TeamCity ReOpenLDAP :: master [Build 19](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=332&buildTypeId=ReOpenLDAP_Master&guest=1) is now running


even ii this is a developers only handler, at least a python unit test should be written


I found this hack yesterday

http://docs.particular.net/nservicebus/operations/throughput#reading-throughput-at-run-time

But I've already updated the upgrade guide (still a pull) to mention that
we no longer support this

https://github.com/Particular/docs.particular.net/pull/827

On Fri, Sep 4, 2015 at 7:49 AM, Daniel Marbach <notifications@github.com>
wrote:

> I already documented the builder access hack. Any other ideas for what it
> could have been used? In my last project we did some ugly hacks to get
> access to GetAddressForMessageType. But this was never on the public API so
> I wouldn't consider this worth documenting
>
> >
>
> --
> *#NSBcon 2015 • The "All About NServiceBus" conference is back *
> *• Learn more <http://go.particular.net/l/27442/2015-08-21/5d4wzp> →*
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/NServiceBus/commit/cf9c27d56756be73f81773472aa70a1a6766a9fc#commitcomment-13062582>
> .
>


I might be wrong but if we use bootstrap we have fonts in px so it's pointless to use em here.
http://stackoverflow.com/questions/12999267/why-does-twitter-bootstrap-use-pixels-for-font-size


else if or just if?


There were some unintentional regressions and jshint 2.9.0 was unpublished just after you published this. The next version 2.9.1 will have those regressions fixed.


what the hell is Viz?

are you taking vulcanian lessons for aliens?


Added a missing `;`. I think this matches your style.


I'll go over all the samples and snippets then.

--
*#NSBcon 2015 • The "All About NServiceBus" conference is back *
*• Learn more <http://go.particular.net/l/27442/2015-08-21/5d4wzp> →*


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


less is more :-)


why is this necessary here and we got no issue at all in other handlers?


decided to use bower, later ll change that plugin to another one with lesser issues on github

ab9f1b21c1e1ffc59eebbc08fc59a2b7eb789ce0


wtf is with this commit message


@charleso @markhibberd I believe with this code Sanction should no longer fail in the situations when old Ordinance happen to create queue with `VisibilityTimeout` set to default value before Sanction has a chance to create it with 8400. This should work for opposite situation as well (when Sanction creates queue with VisibilityTimeout = 8400) since old Ordinance doesn't seem to set `VisibilityTimeout` at all.
NB: I am not sure why Ordinance doesn't set it though (I am 95% sure it doesn't). What is the simplest way to check the parameters of the queue created by old Ordinance?


Because the same instance of ``NuProcessBuilder`` can be used multiple times, with parameters altered between ``start()`` invocations, the user should be allowed to "null out" the ``cwd`` parameter.


ok, i'm doing it.


這邊應該只是抓一個欄位的長度來進行判斷，這邊名字要稍微改一下即可。

最後還是會存入 product.name 裡面。


Which button is Dragon button ?
This one ?
![a](https://cloud.githubusercontent.com/assets/13740643/9677857/60430304-5311-11e5-8aef-62345640a8f0.PNG)


С этим JSTL весело получилось - я же его еще в первом релизе в мавен добавил не зная зачем он. Потом на очередном релизе почистил `pom.xml` и удалил JSTL за ненадобностью. А в этот раз долго не мог понять, почему `<c:forEach` у меня не работает :)
Ссылку классную ты дал, спасибо!


Octree (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1291 of 1291 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-70.0%25-green.svg)


Aula 12 Exercicio


Seria millor posar el CSS al directori dels CSS (master/static/css)

Jordi


PS


Unfortunately World Guard 6 doesn't have direct support.

There is a complicated way but this could be something we integrate further upstream (MassiveCore?) as it requires quite a bit of fiddling around.


this commits is magic pot with totally fucking unrlated staff.
one simply looking at the name is totally fooled on you of what it contains.

nice way to continue working.


I was just asking - why did you remove the following from here?

'url('domain', 'test.www.example.com/path/here');            // example.com
url('sub', 'test.www.example.com/path/here');               // test.www


I'll fix the throttling one today

On Fri, Sep 4, 2015 at 8:23 AM, Daniel Marbach <notifications@github.com>
wrote:

> I'll go over all the samples and snippets then.
>
>
> --
> *#NSBcon 2015 • The "All About NServiceBus" conference is back *
> *• Learn more <http://go.particular.net/l/27442/2015-08-21/5d4wzp> →*
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/NServiceBus/commit/cf9c27d56756be73f81773472aa70a1a6766a9fc#commitcomment-13062899>
> .
>


Maybe this issue requires public discussion?


TODO:
1. getSemVersion  as standalone function to simplify testing
2. Add two tests for getSemVersion (for three and four digits components versions)
3. Add more tests for _getMatchingProjects


Exactly what we needed! Thanks!


This seems an unrelated change. What's it doing and why wasn't it needed before? (Or was it?)


s/ENABLE_FIREFOX_SYNC/FIREFOX_SYNC/g

The "ENABLE" part is implicit by the value of the flag (1/0) ;)


we cannot do this this way @vecna; we will have to handle it properly with the right implementation

the problem is that for every initiative with more thant 100 leaks (i know at least one) you will be going for adding a bug where they wont see some of the leaks they have, with the impossibility to continue postponing/managing it and so possibly even loosing it.

i'm going to rebase this single commit, leaving only the export stats staff and renaming the commit in relation to the export stats only.


I don't think we need to make any differentiation between html and js files here. We can infer that from the file extension and the only thing that changes in any case is the type of comment.


Could we delegate the `getFile` part to the preprocessor?


Aren't disable and remove the same? :)


I've double checked and it works correct for empty string case, but I think I need to add an inline note here so that it does not confuse.


I'd leave this like:

    var fileList = {
       process: ['index.html', 'elements/root.html'],
       remove: ['/js/fxsync.js', '/elements/fxsync.html']
    }

You can then pass `stagePath` to the preprocessor and let it get the files


FourPipeBeamAirTerminal (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1296 of 1297 tests passed)

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


Could we read this value within `preprocessor.execute` as well?


\o/


Well, I admit I can't remember the code precisely enough... If this behaviour is ok, we can keep it, but I think a behaviour similar to the bolshift would be good: negative kern + zero-width hbox + positive kern at each syllable... I admit I don't really understand this code, and I'm affraid it doesn't correspond to the first intention...


Check more about babylon grocery store http://babylonsupermarket.com


nit: one global per line and alphabetically ordered, please.


These methods in the provider are only to check if the pkcs12 generation is enabled by the resource call. So checking if the parameter value is set is enough.

Then we check the PKCS12 file content in the [`SslCertificate::PKCS12#verify_pkcs12`](https://github.com/zuazo/ssl_certificate-cookbook/blob/a634a0cf0169c14e1cc8b0ebdd4ccbb5063b3af8/libraries/resource_ssl_certificate_pkcs12.rb#L50-L55) method. So it should work as expected.

Thanks for the review by the way!


TODO:
1. Add note (see above)
2. Add tests covering both cases (with and without `target-dir` attr)


Problem here is that passing through a dict makes the order arbitrary, so we lose the sort stability of the original order, which was nicely dependency based.


`dump` is not used


babylon supermarket hull


I understand that we don't need a tarball, but the comment suggests that we build the gem in the task you are removing here, so something doesn't really fit...


`dump` is not used here either.


@hflynn I think that doesn't read right. WSGI is a tech not a server. All the listed servers are capable. Let's go through that text after standup.


nit: no space after name function.


the logging moved to the user of `Send` because different messages are sent which need different logging. You should see it on the `Heartbeats` class.


`man` is part of the git tree, no? So in which cases would that directory not exist?


The same here


Thanks for the PR.

Two comments:

1. Can you move the make file to jni/build/android ?
2. Can you move the definition of LOGE and LOGD to jniP11private.h and make it platform dependent ? That way we can replace all fprintf( statements with LOGE/LOGD and get rid of the #ifdef ANDROID statements cluttering the code

Andreas


I suppose this file was added to cover some special cases in the tests. How are they covered without that file?


What's `false`? :)

Could you add a comment explaining that flag like in `file.remove(false /* whatever */), please?

If this is the same `remove` as in https://mxr.mozilla.org/gaia/source/build/utils-node.js#93 I think there's no need for that flag anyway.


Remove this comments, please.


local variable seems enough


As I said before, I think we can abstract this to html and js files. The only difference should be the type of comment:

    html: <!--IFDEF
    js: //IFDEF


This is a demo


Йоба, пажжи чутка. Я щас коммит вкачу, надо пофиксить 404 страницу на ычане.


This commit fixes warning about defined but not used function show_modifytable_info(). This is happened when we compile a coordinator part.


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


Now I understand what you meant in the first place.

Thanks

Noam

________________________________
From: Vineet Gupta <notifications@github.com>
Sent: Thursday, September 3, 2015 7:21 PM
To: EZchip/linux
Cc: Noam Camus
Subject: Re: [linux] ARC: Set vmalloc size from configuration (c3eb0e3)

Not sure what u mean here. You can define a range 1 to 512 and in the code we shift is by 20 to get full value ?

-
Reply to this email directly or view it on GitHub<https://github.com/EZchip/linux/commit/c3eb0e353970e5e225c0469da81b1ea194ce4c1f#commitcomment-13051788>.


nit: no space after `function`


Можешь проверить на Некроопере? У меня вроде работает, только лагает, пиздец.


Thanks for the feedback ! :) Do not hesitate to make some other feedback via the google groups : https://groups.google.com/forum/#!forum/sonarqube


you owner hivewallet? please fix my problem!!!!!!!!!!!!!!!!!
https://github.com/hivewallet/hive-js/issues/245


please use


Дома разве что. Тут ставить надо.


the backticks in here might need to be escaped or something

this was very badly needed though, thanks for writing it


Это не срочно.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.574](http://teamcity.monogame.net/viewLog.html?buildId=10063&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 539, ignored: 6 Build time: 00:27:08


They are all used and can be found in/from theme settings page.


public function addProduct(\TestBundle\Entity\Product $product)


$this->products[] = $product;


public function removeProduct(\TestBundle\Entity\Product $product)


FYI: if you specify it here, you don't need to specify it in the `spring-boot-maven-plugin`. The version is also provided by dependency management so you can remove it. Finally, I'd add `<optional>true</optional>` to the declaration as this is a dev only dependency. Let me know if you have any question.


$this->products->removeElement($product);

(add or remove is always referencing **one** element)


Looks like `aws` (which is used by the current Ordinance) still has this bug: [it should be VisibilityTimeout, not DefautVisibilityTimeout](https://github.com/aristidb/aws/blob/4417273d7c193381588d600609727fce1ebe1868/Aws/Sqs/Commands/Queue.hs#L39)


Very nice. Also the indexOf(), I also thought about that (partial matching). Want to make a PR?


:fireworks: :fireworks: :mortar_board:


public function setCategory(\TestBundle\Entity\Category $category = null)


The commit message should read:

In the TYPO3 CMS backend, newly added content elements
were not displayed in the container they have been added to.
This was caused due to the colPos being set to 0 instead of
-1.


This is sorting the utility codes in dependency order, then making sure all dependencies reference the same object rather than just being equal. It's needed to share declarations among modules (specifically so the enum types can extend __Pyx_EnumBase).

I'm admittedly not super excited about this part; can you think of another way to reference a class from a dependency utility code?


А некроопера на больших тредах всегда лагала, сколько помню.


If I remove the dependency from the plugin, spring loaded is not used anymore it seems


if-else in this recursive function does not act to force returns integer(s), but checks for recursive base case.

I recommend you to read "Recursive Function" further


Появление/пропадание превью постов и менюх происходило с заметной задержкой.


It is mandatory that file name Product class will be src/TestBundle/Entity/Product.php


Ага, вот и у меня так же в куклотреде. Из-за этого и поймал эту ошибку. А на доске работает норм.


Here forgot to remove console.log


Why do you have the A.class  binary file commited with this ?


Такое было еще в ту пору, егда сосачем рулил педалик, а опера 12 была в онгоинге. Причем у меня было ощущение, что это тупо напрямую связано с размером страницы - браузер не справляется с рендерингом.


Yeah well my bad. I mixed up with something else.

You need to remove it from your main dependencies, it's useless. Right now, it will only work with `mvn spring-boot:run` and it would be nice if you could update your `README` to instruct how you can use it right from the IDE.

In short, download spring loaded and add this to the JVM arguments of your Spring Boot app (fixing the path):

```
-javaagent:/Users/foo/bar/springloaded.jar -noverify
```

For the record, dependency management for plugins is totally separate in Maven so unfortunately you have to specify the version there. But you can use `${spring-loaded.version}` as long as you inherit from the starter parent.


Cool :)

Mistake above, those config keys go in `config.components.www.web.routes.*` as per usual


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


DU borde ha en annan färg för den skär sig med bakgrunden


It also has limitations in integer (min,max value that integer type can stored)


Actually, this is not true. It needs to be `(total_sum_left * weighted_n_right - total_sum_right * weighted_n_left ) ** 2.0`, since the previous expression can be negative but the improvement cannot.


When using a plugin like indexmenu there is no # in $src, which results in an "Undefined offset: 1" error on $hash when using list()


Points / Questions that I have:

* all Gfbio enitties have the prefix gfbio_
* need we a function in the sql-script? Basically we stored only the data in the database. The real functions run over the ServiceBilder?
* for the entity like Persistent Identifier, Data Provider, I will test a version on HeadColumnContent
* understand I this right? In the Submision Registry archive is the dataprovider label and archive_pid is the dataproviderId?
* the primarykey of the Submission Registry can be a problem, because the dataprovider ist a part of HeadColumnContent. I have a solution, but I have to test this.


Почему у этого пункта меню стиль `push` ? Что он означает ? Ме казалось, что это для выбора режима моделирования, к примеру.


thank you


I was testing out prawn, this code is no longer used. Presently the template for the PDF is taken from the database, so that one can edit it from the command line.


edit on 04/09/2015


I guess i should remove this to avoid more confusion.


Also added a STARTUP_TIMEOUT in waitForNetwork.


How can you see, I wrote to you that I agree. You did not answer, so I'll write here: Yes, I will fulfill all the conditions of the transaction on time.


input must be acceptable integer only, such that we can't input "asdf" into size variable.


As long as it works! :shipit:


*"...to serve from"*?


Any comments?


I don't like this sort of cosmetics, but I'm not sure if we use the style with trailing {


It doesn't look like features at all.
it's more a datamodel, don't you think ?


Done


moved to `adapter_for_test.rb`


Всё таки нашёл баг с флексбоксами в панели. Если включить функцию автоматического сворачивания панели при отведении мыши, то панель сильно уползает вниз после скрытия. Т.е. неправильно просчитывается `bottom: 0px`.


It's needed as a main dependency as I have a ReloadEventProcessorPlugin, which pushes the changes to the browser. Maybe there is a better way to handle that, but it really is a runtime dependency right now


convenience :)


was missing


The NugetPackager is using the "PACK" command and not push.
The correct link for the documentation therefore would be
https://docs.nuget.org/consume/command-line-reference#user-content-pack-command


Hmm no there was a problem with my commit, it should have included the converter as well...
Le 3 sept. 2015 11:52 PM, Tako Schotanus <notifications@github.com> a écrit :I think the visitor-listener change was done in the previous commit so it's probably just a mislabelled commit.

—Reply to this email directly or view it on GitHub.


Hey Roman, looks like you've done some work to implement the jump lines on the manhattan style lines.  Very cool!  Were you able to update the actual jump line functionality to handle multiple papers at the same time without running into issues of tracking the jumplines currently implemented...namely with the issues sitting on line 29 of the main implementation file...that array...if you tear down a paper, and build a new one, that array doesn't get flushed out, and then all kinds of crazy stuff happens.


Это в некроопере.


@jordimas: és un requeriment de les plantilles de WordPress: necessiten un fitxer style.css a l'arrel del directori. Però si et fixes, no hi ha res de codi CSS en el fitxer.

@PauGNU: potser val la pena fer un import aquí i així ens estalviem la línia 75 del `functions.php`


thanx for the comment - it would be a pain to review and find out no changes :)


I think these two could use the ``$PWD`` variable instead of running the ``pwd`` utility.


Why was this removed? Seem's like a wrong merge (update from upstream master) ?


Same here and in the last one too.


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)


Photovoltaics (evanweaver) - i386-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-20.76%25-red.svg)


There are two tasks here: NuGetPackager and NuGetPublisher. The link in the help for the Packager links to [the pack command](https://docs.nuget.org/consume/command-line-reference#user-content-pack-command) while the link in the help for the Publisher (which uses `push` as you can see [here](https://github.com/Microsoft/vso-agent-tasks/blob/master/Tasks/NugetPublisher/NuGetPublisher.ps1#L90)) links to [the push command](https://docs.nuget.org/consume/command-line-reference#user-content-push-command)


增加匯出訂單按鈕
![2015-09-04 3 22 43](https://cloud.githubusercontent.com/assets/4230968/9678544/f423c5e8-5318-11e5-8e2f-c55d605e4b3f.png)


#Accepted


This is broken on the latest version of FreeBSD, 10.2, whose `tar` [lacks the `--force-local` option](https://www.freebsd.org/cgi/man.cgi?query=tar&apropos=0&sektion=0&manpath=FreeBSD+10.2-RELEASE&arch=default&format=html).


Thanks, fixed


you're right, i was not looking carefully. sorry, my bad.


I already asked you to use constant for every names


Good show, lovely optimization.


corrected it


my hello world Demo!


Либо сочинять костыли, либо дропать. Не думаю, что некрой пользуются больше 2,5 человек. Я тоже вчера промудохался с растянутым окном. Но это можно списать на спортивный интерес.


Можно просто запретить сворачивать панельку. Тогда всё норм.


I know I asked for this "name everything" task but I'm not sure to understand why it's needed in the first place. Can't we remove existing @Named instead ?


useless extra line


Bad import ordering


Ну это уже колхоз.


![the body within](https://cloud.githubusercontent.com/assets/8573644/9678247/efd77d44-52b0-11e5-842e-da6fde1f16a2.jpg)

![img_0001](https://cloud.githubusercontent.com/assets/8573644/9678260/105afd20-52b1-11e5-97df-6c746b96290b.jpg)

![copulating alex gray](https://cloud.githubusercontent.com/assets/8573644/9678180/2b413ad8-52b0-11e5-98be-8401cd923525.jpg)
![alex2](https://cloud.githubusercontent.com/assets/8573644/9678203/5d60f97c-52b0-11e5-8cc3-1f7b4ee43999.jpg)

![alex_grey_sacred_relationships](https://cloud.githubusercontent.com/assets/8573644/9678155/f0beb084-52af-11e5-8bac-7183b5055083.jpg)


xãz


Only comment I'd make on this so far is you should probably base it on the [included-workflows branch](https://github.com/grahamgilbert/imagr/tree/included-workflows) as some things have been moved around.


dsaxas


It makes way more sense to store these columns in the clinical_rules column(where pid = 0). Also, when collect this in the clinical_rules column remember to collect where pid = 0.


Ops, sorry!


Если ты собираешься все переводить на флекс, то придется все запрещать.


Don't know what happened, but some files were not correctly pushed. Also, my commit from yesterday is labeled "22 days ago" :D. Anyway, it's fixed.


Also do not do the html escaping here. Do it when you echo the variables below.


"lyfeCicle" should be lifeCycle in commit message


Incorrent indentation..


Проблема только в связке `display: flex` + `position: fixed`.


Yes we can remove them all


eclipse will do the opposite, could you configure your IDE to not factorize imports please ?


You are right, dgid isn't initialized in this function, Removed.

Relating to dgid/sgid checks, correct me if i'm wrong, I think that there is no way to extract from incoming packet the dgid/sgid because there is no GRH as part of RoCEv2 packet, we can extract just the address from IPv4/IPv6 header.


Do not html escape here. Instead html escape when you echo the variable below.


The name of the event doesn't make sense to me. POST sounds to me as an event that is fired after a job is processed. Should it be something like: SPIDER_CRAWL_JOB_REJECTED and with a reason as property?


Thanks :)


А, всё. Нашёл исправление. И даже не костыль. Просто надо поставить `min-height: 25px;` на панели, тогда всё норм.


Thanks. Fixed at
https://github.com/w3c/webcomponents/commit/f7435a90c3e6ce7a16b061c76eab17cf31f0a0a9


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


you now have to init ... which is the right one to call ?


why this @PostConstruct


I don't think events should be used to process the actual job. Events should be fired while processing the job.


Is the $ruleType parameter currently used anywhere? If so, then this will break it.


I think this should be done threw a multibinding.


I'm wondering if name of protobuf messages should be prefixed by "Ws" (for instance WsSearchTemplatesResponse) in order to improve readability. It's a bit confusing now.


Why not just hard-code the query here and not define the SQL_RULE_USER variable???


That's a possibility, but as I said, this is security concern, we should not require them to do so. if you put `#` there it will be clear for anyone.


all this part of code affect Despotism [look for bad government influence] its one lines above (eexpand code view).

so only under Despotism you have limited resources (3 food) and only under despotism you have +1 food.
I play with this change so i know its working as I intended.

"break saves" - so you want to have portable and compatible saves between vanilla ? Its normal thing that when you "update" game you have to expect that older saves stop working.

single cevo game take few hours, I end every one of my game in one session or two. And definetly I never want to shuffle my saves between different game versions in any game (even rougelikes rpg, like angband when I play one game for few days).


dependencies have to be in the right profile


CONTEXT_BUTTON_REFRESH doesn't appear to be used so I rather removed that


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - 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-81.75%25-red.svg)


Я тоже часто так играюсь. Неочевидный shit в хроме или опере случается часто, сидишь и тупо перебираешь варианты в панели css, надеясь на чудо.


Обычная кнопка меню, для выбора скорее подходит `radio`. (Такие у нас и используются: https://github.com/aurusov/rdo-xtext/blob/release/current/ru.bmstu.rk9.rao.ui/plugin.xml#L561)
Ориентировалась на эту статью: http://help.eclipse.org/mars/index.jsp?topic=%2Forg.eclipse.platform.doc.isv%2Freference%2Fextension-points%2Forg_eclipse_ui_menus.html


This should probably tell you why the native is disabled, or at least where to look. SMC_ParseEnd prints the relevant info into another file which is why this might be confusing.


Is this supposed to be here???


OK - found the issue with upstream (almost destroying my CM in the process of attaching wires to those tiny pullups for the eMMC - now I can not use 4 wire mode any longer...)!

It is related to clk_mmc being configured as 100MHz and not 250MHz resulting in an effective clock of 62.5MHz going to the eMMC...
This also means that upstream is overclocking the "normal" SD-cards as well by a factor of 2.5...

Thanks for your help!


I usually don't ask for documentation, but `String component` don't tell me what should come in


It could be replaced by:
```
response.getPagingBuilder()
      .setPageIndex(paging.pageIndex())
      .setPageSize(paging.pageSize())
      .setTotal(paging.total())
      .build();
```
The advantage is that it removes ambiguity of Common.Paging. The drawback is that the result of ```build()```is not used.


@jgillis @KrisThielemans @dcb210 Apparently this commit did little to resolve the "modules" issue. And since it could potentially lead to more nasty failures (segfaults), should we revert it?


It makes way more sense to store these columns in the clinical_rules column(where pid = 0). Also, when collect this in the clinical_rules column remember to collect where pid = 0.


If users set arg1=1 then script will parse having database into structures (.rdf) and descriptions (.csv).


Also will need to modify sql/database.sql file


what if there's several "." ? I see no tests about that below.


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - 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)


ok, so a string like ".toto" would return "" as ResourceName, that's it ?


I don't think this is correct, you'll end up showing two days not one.


I should have checked this more thoroughly (or just stop committing stuff while sitting in meetings).. If you're up for taking care of this, please go ahead and do so. thanks


erreg


you could map this delimiter into an Optional, it would make the code much more readable


Which data are paginated ? ```templates``` or counters ?


Is the builder pattern really useful here ?


TeamCity ReOpenLDAP :: master [Build 19](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=332&buildTypeId=ReOpenLDAP_Master&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:32:29


TeamCity ReOpenLDAP :: ps-2.4-stable [Build 86](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=335&buildTypeId=ReOpenLDAP_ps24stable&guest=1) is now running


should not display in APP


Please don't remove this, people need to know by transaction what was paid in case they need to check for a mistake with the cash at the end of the day.

I don't see the point of removing this from a daily takings.


I'd make it ACCEPT_ALL :)


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 586](http://build.prxm.it:8811/viewLog.html?buildId=3931&buildTypeId=Jarvis_DocumentStore_CI) is now running


Apparently this is for numerical stability reason as explained in [gbrt paper](https://statweb.stanford.edu/~jhf/ftp/trebst.pdf) around formula (35).

I have taken your remark into account.


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 586](http://build.prxm.it:8811/viewLog.html?buildId=3931&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:08


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 587](http://build.prxm.it:8811/viewLog.html?buildId=3932&buildTypeId=Jarvis_DocumentStore_CI) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 587](http://build.prxm.it:8811/viewLog.html?buildId=3932&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:03


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 588](http://build.prxm.it:8811/viewLog.html?buildId=3933&buildTypeId=Jarvis_DocumentStore_CI) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 588](http://build.prxm.it:8811/viewLog.html?buildId=3933&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:04


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 589](http://build.prxm.it:8811/viewLog.html?buildId=3934&buildTypeId=Jarvis_DocumentStore_CI) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 590](http://build.prxm.it:8811/viewLog.html?buildId=3935&buildTypeId=Jarvis_DocumentStore_CI) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 589](http://build.prxm.it:8811/viewLog.html?buildId=3934&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:02


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 590](http://build.prxm.it:8811/viewLog.html?buildId=3935&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:03


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 591](http://build.prxm.it:8811/viewLog.html?buildId=3936&buildTypeId=Jarvis_DocumentStore_CI) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 591](http://build.prxm.it:8811/viewLog.html?buildId=3936&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:03


two different test for the same condition (> -1 here, >= 0 below), pick one (or drop both with an optional)


We usually don't use the package-protected modifier.
But perhaps it is what is needed here.


If user set arg2=1 then script will standardize parsed structures in .rdf files: aromatize, transforming, isotopes cleaning and hydrogen adding.


Don't worry, they make sense, but these plugins are very strict. I will try to fix all the errors today.


you could just split the string because it's what you do with computeConfigPart and computeResourceName


Is it right?
```diff
- l.isCollinear(h2) && l.isCollinear(h2);
+ l.isCollinear(h1) && l.isCollinear(h2);
```


May i ask why you stopped working on this? This is a feature i highly want to see implemented into php.
I'm currently commenting every method without a return like this:

> public abstract function run() /* : void */;

It would be very very awesome to see a void type soon. (same as a "mixed" return type, which indicates the opposite -> something has to be returned, but it does not matter what type it is. So maybe even merge those two into one rfc?)


747 and 1189  (also 1443/1476) are same I didnt change anything (gitHUB went crazy or I put space dont know- doesnt matter). so yes  CR/LF (I google it, didnt know what it is before).

I copy this code from my cevo Laundry "mod" and Manual works (however something strange thing appear sometimes) so it would be healthy to check by yourself.


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


Should this use a keyref for version number, so that in 2.3 it will not refer to an older version?


@Inject doesn't make sense here


Empty line at end pls


linux/scripts/dtc/dtc-lexer.l:
```
LABEL		[a-zA-Z_][a-zA-Z0-9_]*
```

The regex is a bit different from what defined in dtc-lexer.l.
Is this o.k.?


Well if you look towards the top, there is a "tld", "domain" and "sub"
section now with a full set of tests. If you find a case that doesn't work
by all means let me know.

Thx, Rob :-)

On Fri, Sep 4, 2015 at 1:35 PM, Sreekanth S <notifications@github.com>
wrote:

> I was just asking - why did you remove the following from here?
>
> 'url('domain', 'test.www.example.com/path/here'); // example.com
> url('sub', 'test.www.example.com/path/here'); // test.www
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/websanova/js-url/commit/559e98b9f4c5462b9b5a21fd66867b1c42a90d4b#commitcomment-13063000>
> .
>


This change was just done to ease the testing a bit more. Once the date filtering boxes are added then those variables will have their values coming from what the user selects.

I was just thinking that a standard jquery ui datepicker might be a fast solution here, if I find the time then I'll add it to the filterig options


It looks like a provider


I think there are implementation details that should not be in end-user documentation. How we implement X internally could be in developer docs, but not in user guide.


let's keep it there in that case


Пропуск тут не потрібний!


OK, got your point :-)


Any ETA on when this ends up on pypi? I think this is why pinax-teams won't play well with `django-user-accounts==1.2.0`


For me, this bugs the Stock transfer option between warehouses. It works halfway - it removes stock from the source warehouse, but is not adding the quantity to the destination warehouse. The issue is that the `$price_te` is a string and `is_int()` and `is_float()` report `false`. Maybe `is_numeric()` check is better here?
We've made an override this way and it works.


Warum so viele Leerzeilen hier? ;)


I don't see why, there is no library, only includes. In the root `CMakeLists.txt` there is [this line](https://github.com/poparteu/openMVG/blob/voctree/src/CMakeLists.txt#L305) :
```cmake
${CMAKE_CURRENT_SOURCE_DIR}/dependencies/cereal/include
```
Cereal include directory is set as an include directory for the whole project.


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


I think you problem comes from this : you want to define modules in the server class.
It should be passed in the constructor, then everything becomes simpler and you can use Modules.override


@Gamecrafter I hadn't heard of any PHP doc tags about constants, but if you need one, you should be using \@var.


It seems clear that per compiler behaviour/building is required for open co-arrays

Use of negatives is confusing - you'll eventually end up with checking for double negatives or check for multiple things in negative
+#ifndef PORTLAND_GROUP_COMPILER
+#ifndef PATHSCALE_COMPILER

etc

It's a nit, but I'd recommend

+#ifdef PORTLAND_GROUP_COMPILER

and save the #else block for an Error or warning that an unsupported compiler it being used.


no @Inject


Ага, спасибо.


Wrong value


xsh_offset is signed here, so xsh_size + xsh_offset become long long on FreeBSD/i386...


I'd really just pass a parameter "const char *rvalue" here, rather than "const char **rvalue". I mean, that's simpler and more obvious unless we actually want to continue parsing after this, which I think we don't...


@JKE-be I tried, but there is no "inverse" field in forum model to get corresponding posts (no "posts_ids" field). I could place those functions in a computed field in forum, but in all cases I will need to use "search_count()" no ? (done in my last commit)


Hmm.. there is still something fishy. Can't have two name keys in the same tag.


Tu as une méthode qui s'appelle onFinishInflate. Je te conseille de faire l'injection de butterknife ici. Ca te permettra de l'utiliser dans ta vue custom. Et du coup d'enlever @Bind dans ton adapteur. Encore une fois, simplement afin de bien isoler les responsabilités.


@JKE-be Indeed. It's in my last commit.


lock1 ?


We try to avoid writing things like this these days, i.e. we never mix assignments and if checks in one line anymore. We used to do that a long time ago, and in some parts of the sources this is still existent, but we should really not add new or reworked code like this. Instead do:

```
c = cpu_set_malloc(&n);
if (!c)
...
```


Well, they could only really do `assert(type(return_value) == "number")` and... yeah. The hard part works and I got the easy part wrong. :-(


So I delete it for the moment ? Add a flag to activate progress bar ?


I'd add an immediate check `if (r < 0) return r` here... Makes things easier I think.


you need to also lock here if you want to be thread-safe


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


It probably never made sense to return -EBADMSG here, it should always have been EINVAL i guess...


you should drop this, we'll open a ticket for packaging


Please don't clobber return values if we can still fail. Only overwrite `*ncpus` when we know we cannot fail anymore.


it should not use new


no @Inject


BTW, PHP doc isn't a standard from PHP itself. PHP only recognizes documentation as in the format of `/** ---- */`, but it doesn't have any specification of doc comment tags from its core.


Don't forget to compile CombineHarvester with make after updating to the new tag.


private should work


gh


Hey, I had this pull request which wasn't reviewed yet [1] - it was this line [2]
I'm wondering where to look to fix it again?

[1] - https://github.com/apache/cordova-lib/pull/274/files
[2] - https://github.com/apache/cordova-lib/commit/07271a5c6162c0b2bee55e1ea23f91ebdfbbb34c#diff-e8aa9f6660459c2f0a695066c76a038cL138


log_xyz() actually all internall negate the error numbers, if they are negative, hence I'd never write "-r" here, but just "r", even though both will work.


Why not use the `preRemove` event (see `src/Pumukit/SchemaBundle/Listener/RemoveListener.php`)


you should not start anything during module binding


Hi Kevin, the `jumpover` connector should work no matter what `router` you are using. There is only issue right now and that is that the `jumpover` connector doesn't find the correct crossings when jumping over a link with `smooth` connector.
I can see there can be a problem you described with multiple papers. We're looking into it. The fix for that should be part of the release next week. Thank you for reporting this.


We generally try not to fail on parse failures, but instead just say "Failed to parse, yadda yadda, ignoring", and then do "return 0" and continue. This is important so that newer unit files on older systemd instances don't result in complete failure if they use newer constructs...


A simple @Provides method is enough


If you have no logic, just use a simple bind() in configure()


PHPStorm is not part of this project. This kind of "ignore" should be putted inside your global .gitignore file


После mouseout'а из меню кнопки Обновить (или Звук), панель сворачивается даже, если закреплена.


This is not a good practice, but you are right we will not change that.


> Is this "extends" really needed?

Probably not, I was copying from somewhere.

> it doesn't extend AuthMessage

Yes, I will have a look at the `AuthMessage` stuff that came while I was working on this.

> I think you should not try to be clever here and just have a single message AddJdbcDriverRequest.schema.json. When the time comes and we want to support a separate AddAsModuleRequest command, we can create a second one. Yes it might have a subset of properties as AddJdbcDriverRequest but at least it will be straightforward and won't have these extra "authentication" portions.

OK, let me merge `AddAsModuleRequest` to `AddJdbcDriverRequest` for now as long as we do need a separate module handling functionality.


Great idea! Could you add such a feature to MassiveCore?


@elijh, this seems to be the issue you are referring to in #311, right?


Или открыто одно из окон, тоже сворачивается, хотя не должна.


Hmm, I think we can also get rid of the *ncpus parameter. Instead, on success let's just return the number as return value. That way a negative return value means error, and a positive one is the *ncpus value used... I think this would make a simpler and nicer API, no?


Looks good otherwise. I like where this is going!


Code is like a joke, if you need to explain it, ... is not good code. Also, ... we are inside .gitignore file. Why you are telling the reader that putting "vendor" inside a file that ignore things that the "vendor" is ignored?


sterge asta!


bun


Could you just ignore public folder?


can be done without a class


This is a bit tricky for dev docs in between releases. Since `<keyword keyref="release"/>` currently resolves to the last release (2.1), I hard-coded certain 2.2 references as plain text to avoid confusion. This WIP will need to be revised for the 2.2 release, when it will be safe to use the present tense to say that `<keyword keyref="release"/>` _(and future releases)_ use the 1.3 DTDs as defaults.


I'm open for suggestions; Do we have some other existing one that is more proper? Is there a reason to create yet another pair of something.[ch]?


із 159 по 166 непотрібно тобі так як його ти використовуєш в тесті "Створити тендер"


I think you already says that are libraries in DIR_LIB constant. I suggest comment removals. You could also rename DIR_LIB in PATH_TO_LIBRARIES or something like that.


@xavivars quin avantatge té posar-ho al css en lloc de al functions? No veig molta informació al respecte, només aquest comentari: http://wordpress.stackexchange.com/a/163366/78569. Personalment, hem pareix més adient fer-ho al functions.php, perquè en un altre framework ho feia així. Això permetia al framework saber tots els fitxers css involucrats i fer un merge de tots si calia. Sé que per a WordPress no és el cas, però supose que si existeix aquesta funció d'afegir css és per alguna cosa?


Is this intended? So there are marlins with different baudrate out there?


TeamCity ReOpenLDAP :: ps-2.4-stable [Build 86](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=335&buildTypeId=ReOpenLDAP_ps24stable&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 00:24:13


no @Inject


no need for a class


For consistency, I suggest you to put curly braces in the same way inside all your classes. For example, in LoginController you put open curly braces in new line. Here in the same line. ...


contains ?


Это был чутка кривоватый фикс для другой ошибки, когда панель закрывалась при наведении мыши на меню. Исправлено: f1f5aee526738e033c44b676ace61b4b48c0541b


You may want to consider just having it as a normal class that doesn't extend anything, and just instantiate it in the associated scenes and keep it as a member variable


Цей рядок не потрібний!


copy/paste detected


Why instantiate a never used variable?


added 292cb6c4d337325df026e47956b384a2026dee2b


testFunc is a test? What about TDD and test driven development?


@jossmac not that I can see!


whoops meant to type fixes #283


Do you prefer this to https://github.com/mathquill/mathquill/pull/453? Desmos is actually about to switch to that

(Commenting because I got emailed because the @laughinghan notifies me)


FYI you can write this as: `autoLogin.enable -> autoLogin.user != null`


Mmm have you ever seen twig project?


author, description are missing ...


typo: comments


WTF :)


You are not using phpdocumentor in this project. So, ... why here we can find DockBlock comments?


Why not a good pratice?
You want include it per project?


Why not just use `Path.Combine(rootPath, "TestFiles", "SourceFile.txt")`


Hhhmh. May need to rethink this. My intention was to have a single topic/section with info on 1.3 support, rather than one with "this is what we do" and another with "this is how we do it", but the point that implementation details may not be of interest to end users is certainly valid. Will re-evaluate along with other info on changes for 1.3.


Neat! Them's a lot of savings for 7 characters.


Typo: create a


I don't understand this one, I thought @PostConstruct was made for that


This should be rewritten in:

    return HTTP_SERVER . $route;


just put .gitignore in pylib directory


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (764 of 764 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-10.43%25-green.svg)


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (762 of 763 tests passed)

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


lifeCycle


Duplication of code: session starts here and inside admin/index.php


Hi,

https://github.com/PrestaShop/PrestaShop/pull/3898/files

Regards


Public attribute? Isnt really oop.