更新時のメール送信処理は
https://github.com/aipocom/aipo/blob/e92dfdb9ab71bec5461da82aa7d87aeb40abb8d6/portlets/schedule/src/main/java/com/aimluck/eip/schedule/ScheduleFormData.java#L1835-L1909
あたりでやっています。

まずはdeleteFormDataメソッドにメール送信処理を移植してみましょう。


I see, so some of the null check can be removed, or the whole thing can even just be replaced with a call to ElementTag's constructor.
Well, it's fine keeping the method around for extensibility, though, there's no real need to remove it.


Warning, this line appears to be duplicated twice, lines 101 & 120.
+ you should'nt try to run parser tests before testing that TokenSource works properly.


test


Is it possible to have a tag at this point. Else there will be lot of deprecation warnings in the builds using this add-on.


So you want to keep `isDisabled` for consistency. Am I right?


what are you testing `patch-ver` for here? `nil`? perhaps you could normalize `nil` to `0` and avoid the branch here and below

(sorry to add so many nitpicks about this version warning, but it was hard for me to verify the correctness of)


Warning, the Antlr visitor will first call EnterAddStatement, then EnterAddStatementFormat1.
Because you already call EnterAddStatementFormat1 from EnterAddStatement, this method will be called TWICE.


If you can test it would be great! I'm back from vacations but facing a bit
of a backlog... dealing with a few clients at once... earliest I can get a
look in would be Friday / Monday. Could you ping me if you don't hear
anything from me by next week? I should be on Gitter too if needed.

On Mon, 6 Jul 2015 at 23:08 Nicolas Brassard <notifications@github.com>
wrote:

> But could you test the whole PR please ? ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DeuxHuitHuit/order_entries/commit/e85ef5d8c28d2fd4ffd43f184bff32e8d0677219#commitcomment-12025726>
> .
>


You can use a [Pixmap](http://libgdx.badlogicgames.com/nightlies/docs/api/com/badlogic/gdx/graphics/Pixmap.html) to create a 1 px texture pretty easily. I have an example of using a pixmap in [this class](https://github.com/BrianErikson/Electricity/blob/master/core/src/com/beariksonstudios/electricity/entities/grid/GridPattern.java)


That would have been a good one to include in a change log somewhere. ;)


I think you mean `(<= patch-ver 50)` ? also ditto on normalizing `patch-ver`


-Reused UberView (get rid of necessity to declare init method).
-Refactored methods in this interface to hide direct GWT Widget manipulation behind the interface's implementation. It makes the separation between controller and view sharper. Also without this step, mocking of view in the unit tests is much harder.


Declared the callback as static inner class below to remove some duplication.


same here waiting for the hot-fix..


Moved the decision logic to presenter (view should be as dumb as possible and logic should be in the presenter for testability purposes).


I believe the logic on this line is wrong.. you are passing a null inputInteractor into thy self's this.inputInteractor..


nice. feel free to make a pull request :)


Made the behavior of UI more consistent - now not only the pager appears when more than 6 comments are added, but it also disappears when the comment count is reduced back to 6 or less.


Yeah, we need to find a way to get a no-op build down to 3 seconds or
less.  Currently it's 13 seconds to literally DO NOTHING.

On Mon, Jul 6, 2015 at 9:49 PM, Zach Loafman <notifications@github.com>
wrote:

> Yeah, that plus make && hack/update-swagger-spec.sh works. Yewch.
>
> That bumps the mark-new-version.sh time up considerably, because it's
> currently a kind of checkout-twiddle-checkin operation that doesn't involve
> the code but *should* involve a clean repo.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thockin/kubernetes/commit/e7189041eed7dc535600210a521afd9fade94ce0#commitcomment-12031315>
> .
>


@maxogden hi, the `socket_address` represents the `hostportpair` the `request` uses to fetch the content from network and should not be empty, before we didnt handle non-rendereable URL's this could tell us if a connection was actually made to fetch the resource or not. Another scenario where the value would be `true`(empty) is when data loaded from `blob` urls.


I haven't figure out how to test the logic in callbacks in some nice way. Any idea? Anyway if not, I will still keep one or two "happy path" selenium test in QE's test suite to make sure stuff like this is tested.


nice


I import this and its inverse in another tool. Mind leaving it here (just so it's where the perms are defined)? Don't need AccountPermissions to/from though.


what is this ?


aa


ファイルの中身はちゃんと書きましょう


中身いれなさい


Like you said, the final version of the base class Expression should have methods like : GetLanguageType(), IsConstant(), GetConstantValue(). We could also distinguish the expression types by their types from the beginning : ArithmeticExpression, ConditionalExpression, CharacterStringExpression, because their syntax seem different.


Here we could create a common class ArithmeticOperation, with an Operator property


Moved the decision logic to presenter (logic should be in the presenter for testability purposes and view should be as 'dumb' as possible).


Made the pager hiding/showing behavior more consistent - now not only the pager appears when more than 6 comments are added, but it also disappears when the comment count is reduced back to 6 or less.


Print is not a good idea here. Some systems don't catch the stdout at all, some goes for only catching the `logging` .

But the whole thing about `print` and then `raise` doesn't feel right here.
If you're going to  `raise` then raise the exception with the message so in that case there would be no need for print.


おｋ


you can also create an overloaded method of saveItem which accepts an actual Item model object..


Maybe we need three different classes to model the three variants of the ADD syntax, as they implement a completely different behavior. It would also allow to give meaningful names to the properties.


this should be implemented in your view and not on your presenter..


See the previous remark, here we lose all the precise informations on the parameters we gained in the grammar by matching separately the 3 variants of the ADD syntax


Streamlined in commit [22ae2c5988b9e0126dc3a6a5403d366c7f60dbf8](https://github.com/ghusse/MagicExpression/commit/22ae2c5988b9e0126dc3a6a5403d366c7f60dbf8)


True, this EnterAddStatement should be deleted


THIS HEAD


misspell "canola"


first line and last line is not  right


Do we really need an Identifier here (which is an complete expression), why can't we directly instanciate a SyntaxNumber here ?


Your saveItem method's role should only save an Item.. move the implementation of getting all items on a separate method..


this itemsList could have been retrieved from your presenter..


Yes. Arrow functions has lexical `this` which means that u should not worry about lost contex


Can Step 6 be Node::pre_insert(node.r(), parent.root().r(), viable_previous_sibling) ?
Or maybe even Node::pre_insert(node.r(), parent, viable_previous_sibling) ?


this operation is repetitve.. this could have been manage well from your presenter


@Yaffle Hi, could you release the version 0.0.6 to npm ? Thanks.


INFÉRENCE STP


update


The compiler most certainly inserts the optimization for you, but you should use an explicit System.Text.StringBuilder here


Hi Lan, Im just wondering how you create the Seller, through scaffolding or manual? Im wondering why the controller and the views are so simple, did you edit that before pushing or it is the default from the latest Rails?


This might end up being more complicated due to gson not handling polymorphism well... But the basic idea is there.


I had to double-check the behavior of `strbuf_git_path()` and `strbuf_git_path_submodule()` if the format string is `""`: does it leave the trailing slash, or not? It's a strange use of the function and it's not documented. Either behavior would be reasonable, and in fact if anybody had consciously considered this use I would have expected "don't leave trailing slash". The true behavior happens to be "leave trailing slash"...

...unless `get_git_dir()` is `.`. In which case `do_git_path()` first stores `"./"` to `buf`, then `adjust_git_path()` strips the `"./"` prefix leaving the empty string. (That's pretty good evidence that the author of `strbuf_git_path()` **didn't** consider the case of an empty format.) So your `gitdir_len` can end up set to zero.

I think everything will work even in that case.

I also worried that there might be issues on Windows due to ignoring the distinction between reference names (which always use `/`) and paths (which could potentially use `\`). But it appears that the path functions always use `/` as path separator, so I guess that is OK too.

But I have to admit that I'm a little uncomfortable that this code relies on undocumented behavior of the path functions.


That member seems over-qualified... isn't `CodeModule` and `vbext_ComponentType` in the same namespace anyway?


Ah that's weird because I read it CrossMediaBar here :
http://en.wikipedia.org/wiki/PSX :smile:


https://github.com/bitcoin/bitcoin/blob/master/src/primitives/transaction.h#L144-145

- "A typical txout is 34 bytes big, and will need a CTxIn of at least 148 bytes to spend"

Notes: If takers are not USING all utxos in their fill request, the fee contribution does NOT change.

This may have the effect of contributing more miner fees than intended per utxo, however it will NOT affect profit.

Transactions with higher miner fees are (generally) confirmed faster on the blockchain, which may be desirable.

The MINIMUM count of utxos in a default coinjoin:
(1) taker CTxIn + (2) maker CTxIn
(1) taker txout (destination) + (1) taker txout (change)
(4) maker txout (makerA dest, makerA change, makerB dest, makerB change)
(3 * 148) + (6 * 34) = 444 CTxIN bytes, 204 txout bytes = 648 bytes total

Bitcoin Core Client uses 1000 satoshis per kb as a default minimum to relay transactions (0.9765625 satoshi per byte)

Assuming miners and relay nodes use this amount or higher (transaction spam prevention) should be safe.

Some real-world blockchain examples:
[3-party coinjoin](https://blockchain.info/tx/665a9d7848cc0d28869ef866ca9a1117f20358e1e372dbbb01f1b75054584e70)
(3) CTxIN, (6) txout, 444+204= 648 bytes (above method) - actual blockchain size = 659 bytes

[another 3-party coinjoin](https://blockchain.info/tx/da1a2259be752dd6b5162221989181b7334bd0acbbc1bca31596e5bc32375770)
(4) CTxIN, (6) txout, 592+204 = 796 bytes (above method) - actual blockchain size = 805 bytes

makerA creates at least (1) CTxIN + (2) txouts, or 216 bytes alone.
makerB creates at least (1) CTxIN + (2) txouts, or 216 bytes alone.
taker  creates at least (1) CTxIN + (2) txouts, or 216 bytes alone.

Each should equally pay at least 210.9375 (211) satoshis per input to meet the default relay minimum for core.

Makers running this bot can, of course, raise or lower this amount as desired.

Some makers may want to ensure their transactions propagate quickly to ensure their coins remain available for new orders while allowing takers to contribute low or zero mining fees

Some makers may not care to ensure quick propagation and may feel that takers should decide to pay no fees, low fees, or higher fees to affect propagation time as desired


Committing to issue 120


resolves #377


yup


see comment on pull request


fixed


yup, fixed


The individual plots now look like this, with some statistical information added to the top:

![image](https://cloud.githubusercontent.com/assets/5267078/8539270/bbe2091c-2436-11e5-9026-323d94436325.png)

The original plots from the .PDFs available at the [**C. elegans behavioural database**](http://wormbehavior.mrc-lmb.cam.ac.uk/) look like this:

![image](https://cloud.githubusercontent.com/assets/5267078/8539307/e4de9236-2436-11e5-8fa2-25492f778f6d.png)

![image](https://cloud.githubusercontent.com/assets/5267078/8539309/efde983e-2436-11e5-94e6-84c483c4e013.png)

So, getting there, but still not there yet.


Aren't the library files needed to compile? Btw can you fix Lua? I can't build because of Lua.h please. :/


In other news we are in fact now running the latest version of pyelasticsearch in production. So @erikrose I'll be happy to start doing pull request reviews, and providing more of them very soon.


Here is the logic now: if a player has `hidePlayer` set to true AND it cannot `playPause` then it is hidden. Otherwise, the tab will always be shown in the popup, but the buttons will be disabled if it cannot `playPause`.

Reasoning: On most sites that are strictly music players you will only have one tab open (this covers the majority of the sites, say 80%+). So even if you are not on the main player page, it makes sense for there to be something in the popup showing that the extension has detected the site & is working. In those cases disabling the play buttons is actually a pretty good user experience IMO - it informs users that the extension is working & they are not on a player page.

For sites that have multiple purposes (all of the sites that have the `hidePlayer` property set) their tab row will be hidden unless a player is detected. This solves the problem of say amazon flooding the popup with tabs & should not really be a maintenance issue (ie: sites will probably not go from primarily non music => music. It's more likely that the reverse might happen, but that will also be rare)

Let me know if I'm missing something :smile_cat:


update   recovery-config


https://pyelasticsearch.readthedocs.org/en/latest/changelog/#v0-7-2014-08-12 ... or I could read them better


Ah, no, there's no need for that. I meant that you don't really need a
separate method if it's only used once, but it's fine to have it too, in
case we want to add to it in the future.
On 7 Jul 2015 12:57, "Truong Ngoc Khanh" <notifications@github.com> wrote:

> So you want to keep isDisabled for consistency. Am I right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TEAMMATES/repo/commit/90b41c154e65920ce005ba6696d862812810ba6a#commitcomment-12031410>
> .
>


Teach me how to internet, Zhangerr!!!


mnvmn


I cannot delete pTracker; because its defined as mem<ITracker> pTracker and not as pointer mem<ITracker> *pTracker;


If I understand the logic correctly here, I think that the type of this Dictionary should be :
IDictionary<SymbolReference, Expression (or int if you want to emphasize the fact that operations are reused, but you could index a result just as easily with a direct reference to the expression, and it would more robust)>


Show the summary if there is one.


Otherwise show the new button if the rights are sufficient.


If you want to generalize it later to any type of expression, it should be recursive and take a StringBuilder as a parameter


I'm reverting this because removing `NODE_ENV` well... removes the `NODE_ENV`. For that issue regarding windows, we'll find another solution


changed to the correct directory.


瞎改一下


Question: What causes you to stop and then start significant location changes when the app enters the background? What's the rationale behind this?


WHO ARE YOU


IMPORTANT : please remind that identifier can in itself be a complex expression, like MYVALUE(10:20) .
If you only take the first token of this expression, MYVALUE, you loose the subscript (= substring) and compute the wrong value.
I think we can not merge this pull request until this error is corrected : you should at least delegate identifier parsing to a method with a TO DO comment.


瞎改的


Graciassss :) :+1:


Thanks for your help @samccone, I appreciate it.

Could you avoid merge commits and rebase instead? Ta!


This is for speed issues. Searching for things that return thousands of results takes upwards of 7-15 seconds to load.


Ok. I will try in future. For this you can take a look in the [Spring Reference Documentation](http://docs.spring.io/spring/docs/current/spring-framework-reference/htmlsingle/#_core_container_improvements). There is a _small_ improvement:
> Beans can now be ordered when they are autowired into lists and arrays. Both the @Order annotation and Ordered interface are supported.

So the self implemented ordering was unnecessary.

Hope this helps.


This should give a SyntaxError because of wrong indentation.


I replaced a cascading if elseif elseif elseif else statement with a switch statement in FtcEventLoop.java


Actually you're right, when destructor runs Aruco will be destroyed.

Before 2.3 release we're going to support arbitrary combination of trackers with selectables axes, you won't have to do this anymore.


Fair enough. My only issue with it is that clients don't know their line got split. Though I suppose there's the `echo-message` capability in IRCv3.2 if clients really care about that.


Why does this need to be explicitly turned into a list? In any case
```python
self.sites = list(NX.connected_components(self.graph))
```
would be easier.


I wonder if all the callers are okay with losing SCRATCH2.  Still crashes with a PUSH/POP, though... so that's not the current issue.

-[Unknown]


I simplified an if statement in FtcEventLoop.java as well


Is this supposed to be interactive in RStudio? On my PC, it isn't:( I updated RStudio, and all packages. When I export it to an svg file, and open this file in a browser, I get the interaction.


I am pretty sure that this should stay outside of the loop to possibly add a *final& newline.


IMPORTANT : here, you can not compute all the affectations and operations at this stage of the parsing.
We need to resolve all symbols and compute data items nesting before.
For example :
-- DATA DIVISON --
10 FATHER1.
   15 CHILDA PIC 9(3).
   15 CHILDB PIC 9(3).
10 FATHER2.
   15 CHILDB PIC 9(3).
   15 CHILDA PIC 9(3).
-- PROCEDURE DIVISION --
ADD CORRESPONDING FATHER2 TO FATHER1.
-- operations & affectations --
CHILDA OF FATHER2 = CHILDA OF FATHER1 + CHILDA OF FATHER2
CHILDB OF FATHER2 = CHILDB OF FATHER1 + CHILDB OF FATHER2


I uncommented the bounds check in CinderBlockTeleOp.java


I uncommented the bounds check and put in values for the servo in MantisTeleOp.java


No need for apologies any more


Where is the rest of the `_rate()` method??


The whole method *code* is missing – something went wrong during merging???


The code looks a little clear to me this way. The idea was to keep related parts close: we use pending_daemon_health_rm in 'for' loop to add statements to transaction, so clear it as soon as it is not needed any more. Having other statements between these two might give impression that pending_daemon_health_rm is still needed and is used for other purposes (e.g. you have to look at remove_from_metadata to be sure it does not use it).


looks like bad indent, make sure that you are *never* using TABs and *always* spaces. Set `vi` to never use TABs when editing python code.


Good point, I was unaware of that.

networkx 1.9.1 returns a generator now whereas hop expects a list.


Sometimes you quote the second macro argument, sometimes not.


Hm? I would expect that any Mono version before 4.2 is affected by this bug


Weird, I have my vimrc set to spaces instead of tabs.

In fact I don't think I have ever modified trajectory.py


I think this func ought to also handle ML_ARMREG_IMM -> ML_IMM, in case it's used elsewhere.  Maybe we should refactor so it's commonly used to make dirty?

-[Unknown]


Will fix


This should be a class.


It seems there is a typo here. Also, please, add a space.


There are lots of weird tabbing errors. I have no recollection of modifying trajectory.py. Is it possible that merging can have this sort of behavior?


I think it'll be better to use lodash to clone. For some reason, this feels wasteful. I admire the effort to NOT use external libs :smile:
Or perhaps we create a utility file, with a clone method. So if we drop lodash, we can use a different implementation without digging ?


Not sure, will search through old code and find it again.


I just added a comment regarding the cloning. I'm not familiar with resolver.js to fully comment.


This is supposed to not happen, still it would be useful to know if it does in real life. Not many people are running their systems with high debug level so increasing the level to 5, I think we have small chances to detect this.

But I don't have a strong opinion here so will change to 5.


Will do


r=stanford-scs


Missing space here.


Thanks James! Very helpful.


This closes #21


modularization :grin:


saw approval from stanford-scs
at https://github.com/graydon/stellar-core/commit/eaecc1f4abe2cd065e2a070cc1762aa05a0dfc67


merging graydon/stellar-core/fix-fuzzer = eaecc1f4 into auto


graydon/stellar-core/fix-fuzzer = eaecc1f4 merged ok, testing candidate = f6ff8332


More info is required to reproduce your problem:
- OS type: Linux/Mac/cross-compile(Android?)
- used command lines
- Andoroid SDK/NDK versions if used
- GCC/Clang versions


@kritikanangia Why remove the other div? The one with ```asset``` class. It's sole purpose is to provide that background image. Copy that back again please!


@gylaz - Hi greg :P Wouldn't it be better to use the context to drive the difference in the underlying examples? eg:

```ruby
  let(:allowed_report_ids) { [1,2] }
  let(:policy) do
    user = instance_double("User", report_ids: allowed_report_ids)
    report = instance_double("Report", id: user_report_id)

    ReportPolicy.new(user, report)
  end

  describe "#allowed?" do
    subject { policy.allowed? }

    context "when user has access to report" do
      let(:user_report_id) { allowed_report_ids.first }

      it "returns true" do
        expect(subject).to eql(true)
      end
    end

    context "when user does not have access to report" do
      let(:user_report_id) { 10 }

      it "returns false" do
        expect(subject).to eql(false)
      end
    end
  end
```

I've only found using helper methods rather than `subject`/`let` to be expressive when it's an action you're testing rather than a property of an object (ie, `do_post` or `create` an object or something along those lines, something that would be inappropriate/misleading to cache and reuse).

Would you mind elaborating on your objection to using `let`? I've found it to be invaluable in "proving" a context does what it says it does... In the above example the context actually behaviorally changes the let, it doesn't just depend on the assumption that the test code happens to be doing what the surrounding context claims. Everything inside that context is proved to be working under different behaviors because the context itself changes it.


Rebased commit with the review comments from the previous commit and some tryouts with the Range() function.


windows8.1+mingw-g++3.7.1
Thank you very much!


This cd is not needed. Changing directory is done by the test harness.
--tag-relative=yes is not needed, too.


ss


When you are done, close #2


Commented out while it doesn't work


I think we also want `mr[i].loc = ML_ARMREG_IMM` here.  One benefit is that `SetRegImm` could reuse an imm in a reg if two have a matching value (not sure it does this now, but maybe a MOV is faster than MOVK/MOVZ.)

Also it would be more correct for later, just in case.

-[Unknown]


Currently doesn't work. Not sure how it should look like in regex


Separated simple and complex range scenario


Wouldn't it be less intrusive to use `update_column` here, so you won't trigger callbacks? Even it's not a good practice, the gem-user may have some intensive callbacks that he doesn't want you trigger.


As for now we only support English language


The code we already have should be tested to work for a good variation of scenarios (include multiple concurrent builds, like yours here).

We discussed the current regexps in detail in this PR: https://github.com/josteink/csharp-mode/pull/29

You can see the tests we execute here: https://github.com/josteink/csharp-mode/blob/master/csharp-mode-tests.el#L55-L94

To me it looks like we need to have a few more test-cases and might need to adjust the regexpes to work with them. Would you be willing to look into that and submitting back to mainline?


a


if GetRecruiter needs `await` why it's not called GetRecruiter*Async*


ddescribe


this method and below one looks very simillar, don't we want to DRY it ?


Do these methods belong only for current controller? what about other controllers?


copy-paste detected :)


is it `*async*`


good, but I'd prefer to not use return with long expression, it will be better to split
to
1. get something fromdb to variable
2. return variable value - > simple debugging


It's a human oriented text, right?

there is no need to use capital in every word.


Question : After using "void slip", does this AddTimerBar will reset timer before next station ? So the timer will be false. Right ?


Some general comments about this patch (and series):

The disadvantage of defining the functions multiple times is not only the wasted memory but also that the one-time initialization cost has to be done multiple times.

It would be possible to define the functions a single time, with something like

```
#ifdef GIT_PATH_DEFINE_FUNCTIONS
#define GIT_PATH_MEMOIZE(func, filename) \
	const char *git_path_##func(void) \
	{ \
		static char *ret; \
		if (!ret) \
			ret = git_pathdup(filename); \
		return ret; \
	}
#else
#define GIT_PATH_MEMOIZE(func, filename) \
	__attribute__((__unused__)) \
	const char *git_path_##func(void);
#endif

GIT_PATH_MEMOIZE(cherry_pick_head, "CHERRY_PICK_HEAD")
...etc...
```

The file containing this would be imported from most places without `GIT_PATH_DEFINE_FUNCTIONS` just to get the declarations, but from one implementation file it would be imported with that variable being defined.

I suggest that you change the first macro parameter to be the full name of the function being created, rather than prepending `git_path_` within the macro. That would make it easier to grep for the function definition if you don't know that it is one of these magic functions.

Wow, I never noticed the awful `update_common_dir()` mechanism before. Wouldn't it be nice if we had specialized functions for the exotic paths so that other paths didn't have to go through this?

I don't suppose you want to take the opportunity of this patch series to move the declarations for these functions from `cache.h` to a new `path.h`?


你能多写点中文博客么，挺有意思的~


@JcPires Check this out:

https://github.com/TylerBre/Project-Cars-DS-LiveApp/commit/de0f4344fa55eb326db79e6aa50e3a51a621c8e5


On 06.07.2015 21:15, Othmar Truniger wrote:
>
> OK, found it in config.h. But nevertheless error when compiling
> src/examples/mpropscanpoll.c for missing fd_set from sys/select.h but
> I don't see why.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/knxd/knxd/commit/c0e1121957c892822dd1c3c35092774ec006a5b7#commitcomment-12023699>.
>
Run the command compiling it with "-E -dD" instead of "-o where/ever.o"
to see what happens. Does it simply fail to include config.h?

--
-- Matthias Urlichs


@addyosmani sorry to bother you. Just a quick ping in case you missed the notification ;)
The number of issues based on the master readme and the outdated npm version increases so i think we should get our release done. Is there anything missing from your point of view?


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/69841727


so good so far, but I'd like to not create new controller for infrastructure stuff.

What if I access to /UserProfile/GetIdFromRequest


Thanks! :heart_eyes:


让我们通过一个简单的进行分支和合并分支的例子来说明我们实际工作中可能会遇到的工作流程。


Thanks @tdonohue, we were missing closing more than that span tag!


This snippet resulted in a string neither being HTML, nor formatted at all when used in Sinatra/ERB. However, this works for me:

```erb
<%= @accounts.first.ai :html => true %>
```


你将做如下步骤：


You can find it here. http://agar.gcommer.com/index.php?title=Formulas


fast-forwarding master to auto = f6ff8332


Void it, nekreipkit dėmesio į šį submissioną.


update the comment "Don't need to update when replace is False and record is exist in dns server"


I would say neither one of those. As we both discussed when reviewing that part it would be best to change at least the IRQ allocator in the Platform class to use a more appropriated one, e.g. Bit_allocator.


MinGW is not supported officially. You may try to use native builds via MSVS Express/Community edition (take a look on steps 5-6 of these builds: http://build.opencv.org/buildbot/builders/master-win64-vc12).

About this problem Google says that there are many reasons for this:
- missed, but required for some reasons (broken MinGW instalation), `-latomic` linker parameter;
- invalid `march`: http://stackoverflow.com/questions/130740/link-error-when-compiling-gcc-atomic-operation-in-32-bit-mode
- similar unresolved question: http://answers.opencv.org/question/63127/how-to-configure-opencv-30-with-codeblock-1312-win7/
- the same: http://stackoverflow.com/questions/30512780/why-does-opencv-code-not-work-in-my-codeblocks-ide


@ankit just FYI, when you use an `each` loop (or similar looping methods like `map`), the _named iterator_ (the thing between the `|` (pipe) characters) should not be named the same as the thing you are looping over.

In this commit, I've renamed the thing we are looping over to make it so we don't overwrite the variable.

I don't think it would have broken anything, but it makes it confusing to read the code.


I followed the instructions provided .But an other  Notice: Error: EHLO not accepted from server! in /hsphere/local/home/myuser/mysite.com/system/library/mail.php ...I don't know why?Is there any solutions about it?Thanks very much.


@m-stein : cool that you've found it so fast, and sorry for introducing the bug.


Thank you!


saw approval from shino
at https://github.com/basho/riak_cs/commit/dae708d53b73295e49d464e7d9690c61eca698eb


merging basho/riak_cs/feature/active-delete = dae708d5 into borshop-integration-1174-feature/active-delete


basho/riak_cs/feature/active-delete = dae708d5 merged ok, testing candidate = 055b8432


V1 had an ad hoc TCP protocol. But then I needed to come up with a message framing mechanism, add headers like content type and length, a way to request different output formats, etc.
All of those things already exist in HTTP with minimal complexity so I thought I might as well (ab)use that rather than reinvent the wheel.

The http client and server are probably not 100% HTTP 1.1 compliant since I hand wrote the absolute minimum code I needed to get things working. But I don't think that really matters for this use case. I did this to avoid pulling in more big external libraries. I thought, if necessary I can always drop in a proper http server and/or client library.


I'm a bit surprised by this opening "fact". What gave you the impression that following blockchains is computationally expensive?

Next to that, this document introduces various extra blockchains that the current documentation doesn't use, so aren't you being a bit ironic?

Last; Babel doesn't blockchains. Babel doesn't use blocks anywhere, if I wrote "blockchain" that should be fixed.


What is the purpose of this?


I have successfully compliled my program on windows8.1+mingw32-make3.99.90+i686-w64-mingw32-g++4.8.1.
Thank you for help me!


You also need to pass the scoped flag in somehow, no?


@name Your codes suck!


Lowercase "l".


That algorithm would need some arguments I think, otherwise we don't get a useful result.


In applications parsing a chain the effective state at any point in time is inserted into the database, why would you make this part of the protocol?  Its an end-applications that can decide to do this, or not.  I'm confused why you invent a blockchain to do this.

Next to that, who would sign this new state in a distributed system? Or can anyone just invent the new state and push it to the network?


W którym miejscu tu jest dynamizm? Przecież to jest zahardcodowane!


Typo here.


this work is in progress... give me a few days and everything will work right.


we need to constantly push progress to avoid conflicts...


I'm not sure this is possible.

Imagine having 2000 nodes in a network, and two people create a post at the same time at the far ends of the network.
Your idea that those posts have to have only one parent and be a real chain would imply this conflict to be resolved. Since those two people just forked your chain.  How do you expect to solve that?

Next to that, why would you need a chain of posts anyway?


Same problem with posts, why would you do this and how do you solve the branching issue?


I think I would not even store utf-8, just the raw bytes. If you want anything changed in https://fetch.spec.whatwg.org/ please file an issue against the GitHub repository.


Nah... this plugin is not at that tipping point yet...


lgtm


Don't have access to the system right now. I know something is not included because it works well when I remove the if-clause in common.h. I'm not very experienced so I don't really know what to look for but at least I could not find where config.h is actually supposed to be included.


You should use `process.env.NODE_ENV` in DefinePlugin instead of custom constants. It is used in other libraries as a convention including React. See #13


I just gave my machine for cleaning so this will be done by day after. I
was trying to go for a colorful print. :smile:
I am not sure how to do that with the  statement in raise. Will do day
after tomorrow.
On Jul 7, 2015 10:42 AM, "Alireza Savand" <notifications@github.com> wrote:

> Print is not a good idea here. Some systems don't catch the stdout at all,
> some goes for only catching the logging .
>
> But the whole thing about print and then raise doesn't feel right here.
> If you're going to raise then raise the exception with the message so in
> that case there would be no need for print.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/theSage21/html2text/commit/a5f8e85ce3ff91abba8f73ac792310e34a0f0cef#commitcomment-12031514>
> .
>


Typo - should read. *when one of the ...*


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1303


In general, Babel currently uses a message-chain only in places where authors are expected to actively avoid forking of the chain.

The user-avoidance of forking in the board ownership-chain is actually an intended effect so as to make sure that nodes can solve the issue at the lower level (reject the lower-prio chain) instead of them having to be able to merge possibly conflicting state.

Introducing lots of chains misunderstands that, for instance, bitcoin requires miners and currency to solve the branching issue and we certainly don't want to do that in a forum software.


Hier sind Änderungen reingekommen, die DFB-spezifisch sind. Im master-Branch sollen aber keine DFB-spezifischen Sachen enthalten sein.


Push replace call into checkForPresenceOf


Hier noch einmal.


Oops, fixing it..

Plugin rename was pre-agreed in https://github.com/pmoravec/sos/commit/a532f608502112519b3e4e6e93278374e0ee0622#commitcomment-11563816

See https://github.com/pmoravec/sos/commit/bd329bbf327ca8267abdcc9f6820665760aba6b1 for new commit.


und noch einmal


Last two nits: `char weekday[10] = {0};`


This check is exactly what isDetachedBuffer() does and why I suggested we use it.


And: `char iso_8601[21] = {0};`


The whole structure should be loaded here, not any partial.


Here is a bit of code that a friend of mine wrote. It seems like it would be applicable here. https://gist.github.com/ChrisTM/5834503


Server issue…
Please use SMTP or Mail you cannot use both, this means only fill the required spaces…
if you decide for SMTP you must start the smtp address with tis:// or ssl:// (some servers will accept this connection only if you have an sll installed on your site)
and i said before the issue is more caused from your server, it may be blocking the smtp outgoing connections. it happen to me once exactly the same mistakes

Best Wishes
Carlos

> On 7 Jul 2015, at 07:33, DaisyOne <notifications@github.com> wrote:
>
> I followed the instructions provided .But an other Notice: Error: EHLO not accepted from server! in /hsphere/local/home/myuser/mysite.com/system/library/mail.php ...I don't know why?Is there any solutions about it?Thanks very much.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/opencart/opencart/commit/b03149e4802cc4ac7e9f9ec59c7be4759840c98f#commitcomment-12032326>.
>


IsDetachedBuffer asserts that [[ArrayBufferData]] exists, rather than checks-and-throws if it does not exist. So I think callers of IsDetachedBuffer need to ensure that the value passed to it is already an object with an [[ArrayBufferData]].


Hi


Details!


sgtm here


done


count($teams) zonder -1 - dit betekent dat alle teams dan zijn overlopen.


setUseBatch(false)


New sentence sounds fine, but can you move the first sentence that was deleted to the end of the paragraph?
Also, maybe we want to add a sentence to express the SESE property of ASTs that don't have any free break or continue labels?


That's good to know, thanks! I only know enough web pack to bundle basic scripts, lol. Will check back on #13 once I'm done


LINES[0] isnt good! Should check the existence.


This function should really be a `filter`.


Use commands.contains here instead


@Ovilia 要碰著有勁道個事體再有物事寫呀


I'm not sure I agree with a "mode" that changes the types of these bytecodes, and the v8-native-prototype just has different bytecodes for heap accesses that take a 64-bit index. We should probably have an issue to discuss that, but until then we might want to avoid committing to one choice.


resolved #29
resolved #24
resolved #21 (pending)
resolved #8


The use of `/` here is not cross-platform. ;)


"the junk" is hard to maintain and frankly half of it can't be executed as it is.
This too can't be copied and fired without prior modifications ...


Oh wait a minutes, this only happens in CLI right?
Sorry for my bad reviews, this is actually good to go, leave it as it is.

As long as print or raise doesn't happen in module itself and its only happening in the CLI, then it's good.


GOD DAMMIT WINDOWS GITHUB GUI


Many thanks for your code. This was very helpful in understanding as i am new to Python. I am trying to combine WIND/WAVES spectrogram with the e-Callisto dynamic spectrum (which is a fits file). Can you help me in anyway.

Many thanks in advance.

Abhishek Johri


This defaults to local time, but given we have mixed local/UTC, it is probably helpful to call it out explicitly to prevent confusion i.e. `moment.local()`.


Sollte "Dieser" heißen.


The _extend modifies the defaults. This shouldn't be a problem for now as the defaults are always re-created and aren't used later in the code.


What if ARRAYSIZE was not defined at all?
Android build broken.


Habe eine Beispiel URL eingefügt.


Again, these are ok, and are by default local datetimes, but should probably be called out explicitly.


Apparently if the commit starts with a slash it will include my git's path!


Wir können da keine Empfehlung aussprechen, da jeder Template Entwickler individuell entscheiden muss, welcher Tracking Mode der richtige für seinen Fall ist.


HELLO


No, it's not something that should absolutely done now, it can be done in the futur


Use `fail` instead of `raise` to signal exceptions.


Remove unnecessary public declarations


If you look at include/linux/sched.h, this is the common way used in a "configurable" header.
There is CONFIG_SMP surrounding some struct task_struct members.

For ACPICA debugger, this is likely to be a configurable Kconfig item in Linux.

Thanks and best regards
-Lv


Is it correct that we're passing in a UTC date, but treating dates internally to the datepicker as local? Is there a proper conversion from the internal (local) representation to the external (UTC) representation? I believe that before we used a scope value that was explicitly created as local time, then converted to UTC only when we built the final poll for sending to the server.


@pbathia although your function is completely ok, i would probably do below :)

caseNumber.function() {
    var validCaseNumber = this;
    while (validCaseNumber.length < 8)
        validCaseNumber = '0' + validCaseNumber;
    return str;
}


In fact, from checking, we pass the UTC date property (which hasn't been set, yet), but then treat the model as local time, meaning the ExpiryDateUTC property isn't actually UTC.


`filename` does no longer work, this should be `system_file.name` instead. We had a test covering that, but it was removed as a part of this pull request. I think it should stay.


@yagobski it fixes problem you described in https://github.com/sl4mmer/phpcent/issues/6


Arg, a comment in github is not enough ? In fact I already discussed that with Fabrice, and he told me that maybe in the futur it will be possible to set different quality profiles on module.
As I want to keep the same behaviour as before to not break anything, I didn't think it was necessary to launch a discussion.


Yes, the first step is to porting debugger to the kernel without porting disassembler.
The single-step can be done without disassembler, the only problem is without disassmebler, the source code can not be dumped by the debugger.
IMO, during first stage, this is not a problem.


这么说确实可以，但是，感觉并没有`好的多`。再听听其他人的意见吧！这个地方关于 `branching`的翻译似乎有点不妥，容我再考虑一下。谢谢！


OK, I'll add the disassembler init function.


:) In fact I copy this one from the batch -> The goal is to aggregate which quality profile has been used on each leaf modules (las one on the module true)


s/git/sed/


Indeed !


It shouldn't  try to find links when the content is not textable, so it is necessary to check content-type from the header of the response. And, for other purposes is better to store the response like the chance to retrieve the text as a attribute as well.


Again, I accidentally pushed onto the main by doing 'git push' instead of 'git push upstream'. I guess this is a consequence of doing this too late at night. This commit is just the pictures being changed over from png to jpg, so I left it. The actual code changes are in the pull request I sent.


For this subject, I think you should send an email as there were indeed many discussions...


I think the name `ngModel` isn't really the most appropriate here. Firstly, it's one character different from the angular `ng-model`, and secondly it's not imparting any real information about what the model is, or should be. There's nothing wrong with calling it model, I don't think, but something along the lines of `locaDateTimeModel` would be more appropriate.


Indeed, forgot about it.


Sure no problem. :smile:
On Jul 7, 2015 12:29 PM, "Alireza Savand" <notifications@github.com> wrote:

> Oh wait a minutes, this only happens in CLI right?
> Sorry for my bad reviews, this is actually good to go, leave it as it is.
>
> As long as print or raise doesn't happen in module itself and its only
> happening in the CLI, then it's good.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/theSage21/html2text/commit/a5f8e85ce3ff91abba8f73ac792310e34a0f0cef#commitcomment-12032632>
> .
>


@dietzc Do you have an idea here? Should I resolve all ops manually until I'm able to set my mean as a parameter for Moment2AboutMean?


delete


It doesn't seem very DRY. Also I'm unsure what is going on here


If this is OK, the same should be done to gppublish's usage string


Thanks!


Porting only debugger related functionalities to the kernel can help to reduce the project risk.
The biggest risk is not the scripts to release the debugger code.
The biggest risk lies in the kernel side to hook ACPICA IO functions to implement an IO driver for the userspace.
After prototyping the IO driver, we can consider to move on to port disassembler to the kernel. And we may face more issues at that stage.

Thanks and best regards
-Lv


You might want `reorder` on line 6 too


actually, this line will be very long... it better folder it...


Composer install order is alphabetic and this has to install after phpunit.


doesnt seem like platform/dalvik has r6? complained to me


` <RegEx find="([uU])bj?edljiv" replaceWith="$1vjerljiv" />`


Отлично, дружище! Спасибо за начало :)


两行合起来？


Please check the other files where printf statements have been replaced with echos, whether their output is not used for for-loops, because if a for-loop is used on the output, really NO newline must be passed.


Could this be moved into a filter? It's display logic, so doesn't necessarily want to be in the directive, but needs a copy/paste for reuse.


add:
`     <RegEx find="Suštin(e|i|om)" replaceWith="Biti" />`


The identifier for the comments embed needs to stay as the short url otherwise their will be multiple embeds for the same blog post.

For example: One person may link the entirety of the url (with the filtered title), while another might only link the short version without the title. As well as if the title changes, the fullurl value changes too.
Those situations each create different embed identifiers that the comments embed would use to find relevant comments.

Using the fullurl as the identifier for comment embedding (not social links or SEO related stuff) makes the comments system inconsistent.


Also, actually somewhere, I read that printf is actually more robust than echo...


That's a good point. There is indeed no reason to send the queue if it's empty. I made an issue and we'll roll out a fix: https://github.com/appsignal/appsignal/issues/42

Thanks for reporting!


I was happy with the two order clauses being chained for that one as there are not mutually exclusive.
Edit: Updated in 14d2ee8.


That would be awesome! For me, the ```htmlwidgets``` was one of the highlights of #useR2015, tnx @ramnathv!

Why is the self-containment not necassary?


Previously the callback wasn't called for *un-fixing* so it was not as useful for my particular purpose (perhaps for others' too?)
Now it is called on *fix* and *unfix* events, and users can use `this.fixed` to determine the *fix status*.


then, if I get it right, the current code is correct given the current and known set of supported feature in SQ
there is no TODO unless to handle an hypothetical new feature which has not been validated
I'm confident we will manage to find what should be changed if we decide to go with this new feature
this comment should be removed, it only brings confusion when going through the code
WDYT?


.setServiceAccountUser("user@example.com")


Is there any particular reason why the registry mirror was removed from the setup?


这个是不是也得改成 `/[\\\/]/`


Might be worth putting this in a function that can be appropriately named, as otherwise this might not make sense if you're not familiar with (or have forgotten!) the behavior of javascript's `%`.


then "aggregates quality profile on lower-level module nodes on their parent modules and project"


:+1: I don't like it too, but:
1. We use `konf` gem here which has some drawbacks - if the key doesn't exist - it throws an error. You can check the existance of a key by appending `?` - but then it just checks if the key exists (therefore if the value of that key is false, it still returns true :( ). So I have to first check if it exist, and then ask what's the value of that key.
Another limitation of `konf` is that it breaks `try` method -> it always returns `nil` :o!
2. I'm not sure how I can write it better (I've extracted this to a method.. but still it's ugly)
3. I thought about using other library that has less "wtfs" but I haven't found any at the moment that wouldn't have other deal breakers (nested keys for example).


@tonilopezmr Thx for your PR, is awesome but I can see a mistake, What happen if you have a getter of a Boolean value with the pattern isXXXX?


Finally maybe found it.  This should be `mr[i + 1]`.  I think that's the cause.  Otherwise, the imm is discarded.

-[Unknown]


Interesting emplemntation using even numbers!


This won't help much as the dir == null is guaranteed to be done once where we try the mkdir() already, so the directory is usually created correctly. You only capture cases where the directory is removed *while* the application is running, something that is very dangerous anyway and will lead to all sorts of trouble...

Also isDirectoy() also returns false if there is a file named this way, the correct check would be exists() and failing completely on exists() && !isDirectory(), however as said I don't think these additions make too much sense here...


fast-forwarding develop to borshop-integration-1174-feature/active-delete = 055b8432


This unit test does not actually cover what the code changes above, it just verifies that the temporary directory is created in the constructor, something that we test in other places already...


Can you provide explanation if code aftet form_for? Why do you use two variables?
[@question,@question.messages.build]


You can drop parenthesis and simplify condition:
```python
if fo.currentTurn() == 1 or total_rp  > research_queue.totalSpent:
```


Global helper?
```ruby
def use_feature?(method_sym)
  AppConfig.features.send("#{method_sym}?") && AppConfig.features.send(method_sym)
end
```


Ah ok, my concern was that the order priority might not happen as intended (title first priority). `order by title ASC, created_at DESC` (I've not tried it out to confirm).


`  <RegEx find="\b([sS])avet" replaceWith="$1avjet" />`


`<RegEx find="([sS])arađ" replaceWith="$1urađ" />`


Is the `ng` prefix not normally used for core directives and the like?


> **Core/Spells: Fix Prayer of Mending visual bounce**
> Close #12855 , thanks **@P-Kito**
> **jackpoz** authored on 24 Aug 2014
 sql/updates/world/2014_08_24_02_world_spell_linked_spell.sql

Sure, you are mentioned there. I don't know, maybe it was because your name was not placed on its own line saying **by P-Kito** or because you were not listed as author or not as the owner of a PR? Perhaps it was looked at in the same way as when new users create a PR and say "thanks" to the main devs?

I would say that your proof is this: https://github.com/TrinityCore/TrinityCore/issues/12855 (solution posted as issue).


`<RegEx find="\b([pP])rimj?eni(m|š|imo|ite|ti|li|la|le|lo)" replaceWith="$1rimijeni$2" />`


утилита возможно не корректно работает после изменений - мне кажется проблема в задании переменной   npart - 257 -строку убрали и на этом этапе теперь переменная всегда нуль.
также не парсится номер раздела в параметре -w хотя в хелпе это указано..
наверное необходимо провести некоторые правки и  тестирование утилиты записи разделов.


Without this commit, hw_x86_64 currently has warnings when compiling core. But for me it's also ok to fix the allocator. I'll open an corresponding issue.


Oh no you're right!
With `order` it produces `SELECT "posts".* FROM "posts"  ORDER BY created_at DESC, title ASC`.
With `reorder` it produces `SELECT "posts".* FROM "posts"  ORDER BY title ASC`


delete completely


While we're at it, we can also document in the man page that a second parameter is required to be there : cloud.

And now that I'm trying to read the script instead of the man page, I'm wondering if "imagefile" is really what I think it means. Can we give it a "/path/to/my/new/image/file.qcow" or is it something else ?


`  <RegEx find="([pP])o[dt]stič" replaceWith="$1otič" />`


initial commit


Maybe we can find a tricky way to get syntax background color ?
We currently use some js to get the font family https://github.com/webBoxio/atom-term2/blob/master/lib/TermView.coffee#L96


`     <RegEx find="([oO])rganizov" replaceWith="$1rganizir" />`


Just technical comment - you can put all these methods to an implementation of enum which can expose all distributions. But depends on your intention - if you need to hide different distributions behind a shared interface (+ performance cost).

```java
interface F {
  public double fce();
}
enum X implements F {
  A {
    @Override
    public double fce() {
      return 0;
    }
  },
  B {
    @Override
    public double fce() {
      return 0;
    }
  };
}
```


There's no storage setup **above**.


I was searching for exactly this feature, started trying to implement this myself. This needs to be documented.


` <RegEx find="([nN])aslj?eđ(a|e|i|u|em|ima)" replaceWith="$1aslijeđ$2" />`


You're welcome! :)


The `[]` parameter in the function call seems strange to me. Can we not have
````collect_all_results(index_page, results=None):
    if results is None:
        results = []
    ...```
Or is there a reason for the function being like this?


TeamCity NEventStore :: Raven :: Build [Build 6.0.0](http://build.prxm.it:8811/viewLog.html?buildId=3356&buildTypeId=NEventStore_Raven_Build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:01:57


I had intentionally left it that way so that gem-users could always hook
into the gem's action, for example to have the locale uppercased or update
a counter.

But you're right. The god-model User usually has all kinds of nasty
validations, observers and callbacks. I'll use `update_column`.

Thanks! :)

Le mar. 7 juil. 2015 08:13, Thomas Lecavelier <notifications@github.com> a
écrit :

> Wouldn't it be less intrusive to use update_column here, so you won't
> trigger callbacks? Even it's not a good practice, the gem-user may have
> some intensive callbacks that he doesn't want you trigger.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sunny/devise-track_locale/commit/d5273ccffc47442d41eca66e51d1ecdab61810f1#commitcomment-12032133>
> .
>


Instead of try/except, why not check the Python version and only load this for Python 3.4+?


Many thanks for your help!
![smtp](https://cloud.githubusercontent.com/assets/13131511/8541117/4608ccb6-24be-11e5-83e5-2d55ac32df3f.jpg)
I  use SMTP,and this is my settings.But I don't know where is the  problem.Could you find it?


` <RegEx find="mj?eša(lic[aeiu]|licama)\b" replaceWith="miješa$1" />`


`<RegEx find="([kK])učk" replaceWith="$1uj" />`


` <RegEx find="([kK])ontrolisan" replaceWith="$1ontroliran" />`


this part is wrong. zend_arena_checkpoint() must be called before zend_arena_alloc()


Duplicating `git clone` doesn't help.


`  <RegEx find="([kK])onkuris" replaceWith="$1onkurir" />`


`<RegEx find="([kK])ompjuter" replaceWith="$1ompjutor" />`


it is better to use io.Pf(...) because it gets deactivated for non-verbose runs


Fixed in 587704d. I'll try to write some similar tests for the other drivers later.


Yup, I saw that.
Last version of  **CodeElementBuilder.cs** has an empty EnterAddStatement method: c79e1fc979.


`<RegEx find="([kK])elner" replaceWith="$1onobar" />`


the DOF is actuall a T1var since the equation has only the first derivative w.r.t. time


Я бы слово обязательные убрал бы только.


nginx-tfs这个最新代码在alibaba/tengine主干中，请使用那个，谢谢！


typo: found pattern


ack :)


Other *NOTE*s in the document look like **Note:** `**Note:**`. Let's stick to one style.


`Update at ...`, `... something was ...` :-)


committ message isn't sane


Yes, I commited too fast ! That's fixed now.
I put the *Check_StatementCodeElements* method at the beginning of *CheckParser* because some tests of *CheckParser* (namely: *Check_NOTCodeElements*, *Check_ONCodeElements*, *Check_EntryCodeElements* and *Check_ParserIntegration*) are currently in error and I didn't have time to fix them yet.


Looks good to me. Cross checked other areas of the std lib that throw exception's and they use NULL instead of a specific exception too.


Думаю что лучше будет `Для понимания того, как работает Symfony`


Hi, Bob

I trend not to merge this commit right now.
We can only merge the configurability stuff for now - one of them changes a whole file and thus easily faces conflicts with other commits, so it is better to merge it earlier.
And after I having done the kernel IO driver, we can merge this commit to make debugger files appearing in the kernel source tree.

Thanks


See also https://github.com/simoncozens/cassowary.lua/pull/2 about the unpack issue.


Why, you're welcome ! o(^▽^)o


Great catch!

I probably won't have time to work much on it tonight, we'll see. Otherwise I'll fix it tomorrow, unless you beat me to it with a pull request to the branch ;)


Indeed, you're right ! let's remove this comment


Ok for me !


That's right in line with the style of Asciidoctor core; the minimum of what's needed to function in the name of speed and keeping dependencies lean. Nice!

I absolutely see the value of using HTTP for this purpose. In fact, I think it opens doors in the future for other libraries to join into the pipeline...something a low-level, “proprietary” protocol makes difficult.


Doesn't time.Sleep take duration? This should be ```time.Sleep(30 * time.Second)``` or something similar.


**Коммент**


I'd prefer to handle this case on our own then, e.g. `return false if File.empty?(system_file.name)`.


:bomb: :wink:


Coding convention. It should be Path. Capitalize.


It should be children.


parent should be initialized with null


good idea :)


GLOB_BRACE isn't really portable (although it seems to be almost universally available, I've checked recent BSDs, glibc etc) see http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/glob.h.html

However some diet/light libc's are without it, e.g. http://www.musl-libc.org/

For this and similar things, would proposing a HAVE_GLOB_BRACE or similar be the right idea or "let the libc's deal with it" ?


What is it? Why add to the menu?


Right, missed, that


fixed


test 코드가 쌓이면 나중에 힘들 수 있으니 다음부터는 지우고 커밋해주세요.


:pushpin:


I'm ok if you don't want to make a specific release tag. But perhaps update the NEWS file to add the commit id alongside each release so people playing along at home don't need to trawl the git history for the appropriate IDs.

>I don't think people will necessarily understand that if you have a “release” with e.g. 1.1.2, that is a different tarball with a different checksum from the true 1.1.2 release. Perhaps if you're very into the details of GitHub, but most people are not.

Did you find someone who was confused by this? I'd be surprised if it was less confusing that `git log --grep=1.1.2 | grep commit | cut -f2 -d'


@hax 用call()来将migi注册到全局变量上有啥隐患吗


It feels like the only thing advanceMinute is really doing is this semi complicated maths conversion, pulling it out to another function might be a bit of a waste. I will put a comment on both lines explaining what is going on.


Well this list should be only for people who really contributed and not for someone who just wants to be here. You requested your name here after 3 sql lines you did, well I think with that attitude you won't be added. That's like you request state awards after nothing. I apologize if I said something impolite, but that's my opinion.


Thanks for the changes @sindresorhus! We were using these fields as a source of metadata for the header banner in our final build files (homepage link, license.type). I'll update to reflect the revised version you've got here. It's way trimmer.


Can you try solve it recursively.


:round_pushpin: Is not the same, and not interest to change it in this commit


I have seen various other libraries use this. I think it's a good idea as it is very obvious what the binding is doing and is certainly the first thing I would try.


Awesome, yeah I think that's a really good idea.


"trvMenu" is name treeview.
initialize should be add to it trvMenu


Great to hear that the tool is of use for you! So if the BOM is causing issues I see no problem in removing it, just done so in this [commit](https://github.com/sebastian-lenz/typedoc/commit/5011b37bcbec1a2dc4bbdd36ab012a6a9c56fabd).


Hi,

Thanks for the nice idea and realization. Please note, that your solution is not threadsafe and it does not support multithreaded build.

Regards,

Gábor Lipták


so the name is reasonable?


Thanks


You have just manually implemented a sparse matrix/vector multiplication. I assume you are doing this to improve performance? In the long run we should consider to use one of the existing sparse matrices (from eigen or istl). They do support creation from a dense matrix (such as the `FieldMatrix`) and they do support to drop all entries from the pattern that are close to zero (via the `pruned` method).


测试提交本地代码


Merge from origin code


Also why do you add it to Items? Why don't set it to DataContext or DataSource?


Falta eliminar el error_log


Just a minor nitpick for Python's style compliance: keyword arguments in Python should not have spaces around the equals sign.

I can fix that, though.


Looks good.


please always start class/ids with lower case


Can you give more details on how to reproduce the segfault here: https://github.com/christopho/solarus/issues/706
Thanks!


OK with me.


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


sivarajk and staticvoidmainpub are  the same


Does something like this helps you ?

```css
@import "syntax-variables";
.term2 {
  .terminal {
    background: @syntax-background-color;
  }
}
```


Why is it 10? default value is 1000


ok a. it's a root. the name should be "rootTree"


Small typo: youre -> your


There's no need to send the stream to sys.stderr, as it will be automatically used if no tty is being used, according to the documentation.


这名罪犯在被独自关押的时候不会感到被镇压。他丝毫不在意。


Does the preceding sentence read better without the 'and' at the start of line 46?


No. It's not root. It's a tree. It should be name treeView.


wow, nice


You would be surprised by how confused non-dev people get when they see messages like these at level 1, even if they don't happen often. AFAICT, this should be a transient state and there's no point in alarming the user.


Stray newline.


@zghbssjzzhx 感谢，不过这个已经被修正了，[commit](https://github.com/azige/Prison-Architect-Chinese-Language-Pack/commit/1584aea9da9ca1362193131811bc6f75d6ee1249)


Another stray newline.


'props' instead of 'properties'?


Two more stray newlines.


@zyx0225 it should be already released in the play store. Just please mention that the minimal version is v3.7.7


Die Änderung war aber nicht wirklich sinnvoll. Da soll doch stehen das noch 2 euro fehlen wenn man zB. 8 eingibt. Das Problem ist das der Buton Geld zurück anabelt wird.


자갈 해안이나 암벽 해안, 석 해안되 되는데 직접 보기전에 말하기 힘들듯


To shorten this, try to replace this to `src.replace(/^[\\\/]{2}/, '')`


Oh man, I didn't know this existed!  This **so** cool!


Yes, this looks very much reasonable. The impact on Python 2 is small enough for this to be merge immediately.


essaie de loader le VersionedContent avec sha_public puis fais un print du traverse qu'on y voit mieux.


should this be $sub_product->getTypeId() ?


It does. Thanks!


Is there `e.code` property available? If yes, please use it (compare with constant).


aad


"Also why do you add it to Items? Why don't set it to DataContext or DataSource?".
add it to Items. It' s not true?


cute.


Also note that Beakers `install_puppet` and `install_pe` functions are deprecated, in favour of `install_puppet_on` and `install_pe_on`, respectively.


Oh good, I was worried about this and had this on my list of things to do! but you bet me to it!


delete FullMaterialPackage.tpl  as well


should be in management, not display


we only have a material package on the event level, not for contributions etc - so you it shouldn't be inside that loop


done


that can be a single rule (accepting both GET and POST) pointing to a single RH


No particular reason. I chose a slightly simpler function at the expense of a slightly more complicated call site. Other variants of this collect multiple index queries into a single result set, so the mandatory parameter is less of an issue there.

I'll make the change, though -- you're right about this particular case.


no need to specify that argument if you don't have any defaults


i'd use singular in those method names


:smile:


Ok


to use one RH for both GET and POST:
```
if form.validate_on_submit():
    # do stuff
    return ...
return WPAttachmentsPackageDisplay.render_template('package.html', self._target, form=form)
```


also, please don't use `self._target` - `self._conf` is much clearer and should also point to the event.


wrong base RH; it's management, not display


hm. naja diese Zeile und das Beispiel darunter sind eine de facto Empfehlung. Wie das so mit Doku ist wird der Großteil dem Beispiel folgen ohne lang über die Folgen nachzudenken.


FAKE_SERVER_NODE 상수도 CacheManager 쪽으로 가져오면 어떨까요 ?


I have so many build systems. I forgot to update all of them :-(


:+1: @vorcil I was going to check with you if we could turn vote.html it into a server side page so no client browser can bypass the check.


`parse_kwargs={'dayfirst': True}` - otherwise it'll break for ambiguous dates with the date format we use in indico


indentation is off here


We already have `IndicoSelectMultipleCheckboxField` which probably does exactly what you need.


Introducing global?


Yeah that's a good idea
Want me to look into that?


what does "WDS" mean?


`Media.onStatus(id, Media.MEDIA_ERROR, reason)` will call `Media.errorCallback` internally, `lose` will do exactly the same. Looks like `errorCallbacks` for Media instance will be called twice here


this comment could be clearer


Is your current system just redirecting to the index?


`return false if File.zero?(path)` is a bit shorter.


this doesnt seem to do anything


it's not immediately obvious why this change is needed


That's not confusing, it's just very cumbersome. (And I accept that it's cumbersome.) In the choice between giving out too little information (lack of tags) and false information (“release tarballs” that are not release tarballs at all), I'm picking the former. I'm not saying it's good in any way, but this is how Github works.

@Optiligence: I agree that the _tag_ tarballs are not confusing. The problem is that you also get a “release” tarball which you can't get rid of. Github is simply conflating the two concepts.


Flask-SQLAlchemy provides `AttachmentFolder.query` as a shortcut for `db.session.query(AttachmentFolder)`.

Also, we have some extra functions in IndicoModel, e.g. `AttachmentFolder.find_all(event_id=int(event_id), is_deleted=False)


`win` can be (and will be) `null` here, no? See https://github.com/apache/cordova-plugin-media/blob/master/www/Media.js#L131


the default folder has no title. So you might want to use `"No folder" if folder.is_default else folder.title`


those imports are not neeeded


Is using `ESQuerySet` to build the query and pass base_query not better (also for the `groups_es_call`)?


See https://github.com/MSOpenTech/cordova-plugin-media/commit/41d6149b7b55a6a8ce067d0a1a30ace3ca636231#diff-4f63e7842d9af4df877344c399338332R263


Currently view.html makes a client side request to check if they are allowed to see view.html then if not it redirects i think for sanity and security we should get server side to do this check not the client. I can add this in tonight


Bad indentation


See https://github.com/MSOpenTech/cordova-plugin-media/commit/41d6149b7b55a6a8ce067d0a1a30ace3ca636231#diff-4f63e7842d9af4df877344c399338332R268


I guess for desktop version is ok but it should be moved somewhere for mobile version


Either `get_groups_size ` or `group_es_call ` sounds like a misnomer.


Ha! Finally :smile:


Indentation misaligned here for NAZESRC


kenapa di turunin fan ?


have you tried detecting mobile devices? so we could enable this footer for desktop only, and dialog for mobile


FAKE_SERVER_NODE 를 여러 곳에서 수정하는 줄 알고 수정하지 않았던 것인데.
실제로는 두 군데 밖에 없네요.

수정 했습니다.
commit : e2b43ae074ba3e5168e6d2661b8b64240604ef9a


This is all very good!


@jpuri I think this is broken now. We usually use `properties` internally, but `props` for the API.

```
    this.state = {
      value: value,
      focusedValue: undefined,
      generalProperties: sanitizeWrapperProperties(properties),
      characterProperties: sanitizeCharacterProperties(properties.characterProps),
      isFocus: false,
      isActive: false
    };
```

What about we switch to `props` everywhere to avoid confusion?


Probably have some simple comments here to state these map to channel freq


Will push once the weather becomes more acceptable.
Currently I feel like spending all day long in the swimming pool...


Yeah, this improved performance a lot, as the matrix I use has only two entrys per row and can be quite big. I thought about using the EigenRowMajorSparseMatrix or IstlRowMajorSparseMatrix, but I did not see a possibility to create one of these from the FieldMatrix (there is only a constructor from BackendType, which is not FieldMatrix (?)), so I decided to do a quick and dirty implementation on my own. If you point me to the constructors from a FieldMatrix, I could just use one of these internally (or templatize Affine by the MatrixType?).


I feel like this is redundant for mobile version, so we need detect a desktop


test commit comment


Bad way of overriding a hashCode()


needs further discussion


Can you put the throw statement within brackets?


yrydrtyt


Doing this is reasonably expensive, but i assume this is only done when setting. Do we need to make sure that you cannot change VTX while armed?


I would rename the unit test to something like nonExistingResourceShouldThrowObjectNotFoundException


also the location of the sql-update files were changed so they did move to


If the loops is skipped at 164 line, `total` will be more but options will not count the data-source because `start > count`. May be I am reading something wrong. Or you could add another test to skip loop and make sure.


Should this go after line 164?


for desktop: I'd put info about ruling in dialog and use footer (like you did).
for mobile: hide footer (put info in author dialog maybe) and put ruling in somewhere (dialog?) also.
I believe we should not overload the UI


对于数组的初始化，`memset(m, 0, sizeof(m));`会比较好。


insertAdjacentHTML 사용해서 li 안 만드는 방식으로 바꿔보기는 했는데, 그럼 querySelector로 다시 이벤트를 등록할 dom element를 찾아와야해서 좋은 방식인지 잘 모르겠어요.


@sergeylunev насколько помню, они уже сами отказались от цифры 2 в названии


Sorry when this is confusing, but this is not necessary. `parseInnerQueryBuilder()` should never return null anymore but EmptyQueryBuilder instead. Same for `parseInnerFilter...`. The null-checks will have to be applied to what gets returned when calling `toQuery()` on any q.b. later, since those can return `null` (not only the EmptyQueryBuilder one, but also some types of filter etc.. when they internally have subclauses returning `null`).


Great!


No it is just that the condition is not in the right place. Will fix that soon


The package name is `iproute2`, but not `ip-route` ☺


Hmm. Well, it works for me here, and my code should be identical to the
head of 15_contact_preferences, since the only commit to this branch since
I left was Cecilia using the files in my email (admittedly, an error prone
way of doing a commit).

I'm still using Couch DB of course. I don't know why that should matter,
but what else is different?

N

On Mon, Jul 6, 2015 at 10:06 PM, Karl Fogel <notifications@github.com>
wrote:

> I don't know whether it was this commit that did it, or something else on
> the 15_contact_preference branch, but right now this is the head commit
> on that branch, and if I check out the branch and do npm start, it fails
> with errors:
>
> $ git checkout 15_contact_preference
> Switched to branch '15_contact_preference'
> Your branch is up-to-date with 'origin/15_contact_preference'.
> $ npm start
>
> > smoke-alarm-portal@0.0.0 start /home/kfogel/private/work/ots/client/red-cross/r/smoke-alarm-portal
> > node ./bin/www
>
> module.js:340
>     throw err;
>           ^
> Error: Cannot find module 'generate-object-property'
>     at Function.Module._resolveFilename (module.js:338:15)
>     at Function.Module._load (module.js:280:25)
>     at Module.require (module.js:364:17)
>     at require (module.js:380:17)
>     at Object.<anonymous> (/home/kfogel/private/work/ots/client/red-cross/r/smoke-alarm-portal/node_modules/nano/node_modules/request/node_modules/har-validator/node_modules/is-my-json-valid/index.js:1:76)
>     at Module._compile (module.js:456:26)
>     at Object.Module._extensions..js (module.js:474:10)
>     at Module.load (module.js:356:32)
>     at Function.Module._load (module.js:312:12)
>     at Module.require (module.js:364:17)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenTechStrategies/smoke-alarm-portal/commit/d3eb4e09c511583b08728aed5ce6829ecb477398#commitcomment-12030518>
> .
>


Yes this looks very good to me. I like this part:
```javascript
//By defualt we first check for the old front and back selectors for backward compatibility
//if they arent there we fall back to auto selecting the first and second div
if (settings.front == "auto"){
    settings.front = ($dom.find('.front').length > 0)? '.front' : 'div:first-child';
}
```
Clever stuff!


hmm


Maybe package private since it should only be used in the parser as well?


review 완료.


Maybe we want to simply remove it. And find the values of the children in every case


Correct me if I'm wrong but this could be used to provide custom authentication right?


Since parsing to null should be prevented now with EmptyQueryBuilder, I'd rather say this `generated` a null (lucene) query or something the like. Doesn' have to come through JSON, but yes, that's most likely.


@arizzi  @veelken
Any particular reason why you removed this line?


Hello


Wait, aren't you using Angular-Bootstrap? I thought it doesn't require jQuery/Bootstrap.js.


more love


Oh, sorry, didn't realize...


You can remove the `<pre>` tag


这题似乎没有必要做如此大的改动，因为你的方法要简洁一些（无需排序），我之所以排了序只是惯性思维而已。


Yeah, Java has StringBuilder too -but in general I'm not a big user of it, because I find it less readable (I prefer to read my expression with words direcly inside it instead of doing the gym necessary to mentally put parameters on the right inside the target formatting on the left).
But that's just me, so I've done the refactor in 9275f28. ;)


I think using template strings like this definitely makes things more readable


Sorry mate, but this is ugly. :-)

Some of these aren't really needed nowadays (I'd stick to webkit+normal). Preferably include mixins from Bootstrap and use .transition-delay(100ms) etc. Thoughts?


Tagged v0.1.3, published on npmjs.com!


I guess you removed this because it was not a case we use ?


TeamCity Jarvis Framework :: CI-GitFlow [Build 15](http://build.prxm.it:8811/viewLog.html?buildId=3366&buildTypeId=CqrsFramework_CiGitFlow) is now running


:+1: :+1: :+1: :+1: :+1: :+1: :+1:


Что тестирует этот тест? Мне кажется, нужно проверять, что encrypted_password изменился, а не то, что не равен оригинальному.


Check for pre tags


Test


Ne also...


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.0.1-beta.1+3](http://build.prxm.it:8811/viewLog.html?buildId=3366&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 107 Build time: 00:02:28
### Failed tests
```
Jarvis.Framework.Tests.dll: Jarvis.Framework.Tests.EngineTests.SagaTests.SagaTests.listener_tests: <no details avaliable>

```


Note that this change means if you ever run this program in a different network namespace it will output a confusing message.


What if you generate UUID & beaker secret token by running python in the virtualenv & adding the output to `development.ini` file using [lineinfile](http://docs.ansible.com/lineinfile_module.html) module ?
Here is an example of how I did it: https://github.com/oguya/ansible-ckan/commit/5fc052d0ccecff441416877481d81283fd6f8ce8

- generate UUID & secret token

        /path/to/virtualenv/bin/python -c "from paste.script.util import uuid; print uuid.uuid4()"
        /path/to/virtualenv/bin/python -c "from paste.script.util import secret; print secret.secret_string()"


`if(data && ...) {`
`null` является валидным JSON'ом и выдаёт как ты уже понял `null`, поэтому лишняя проверка не помешает. И надо бы создать функцию `earlyInit` которая будет выполняться до загрузки DOM'а если возможно, куда и поместить этот код. Тогда можно будет избавиться от обновления страницы. Да и инициализация скрипта ускорится.


I think you should add a comment here that these were added later


If you use a modern distribution, you can install libnl3-devel.x86_64


Please look into using Netty's CookieDecoder


Doch?


Wirklich?


My suggestion was to remove the next_tweak right after the beq branch, and keep the one right after the label.
Your version is not equivalent, and I think it is incorrect: the branch to .Lxtsencout from the 2x/4x loops will skip the final next_tweak, so the tweak will be incorrect for the next invocation. (Same below)


I remember these methods when browsed code months ago but it seems to me 6secs is not used by default? I know FFmpeg is slower then MEncoder but what I remember, response from renderer is definitely shorther than 6secs. (1sec for MEncoder and 3secs for FFmpeg or something similar)
Anyway it seems they should be useful for FF/RW response it it is not already hardcoded to shorter value (UMS don't know it user is playing video or making FF/RW so it seems it can't be hardcoded)


and a comment here that this is for backward compatibility


Try replacing `self.item_etag` with `etag` (see line 584). Test should pass after the change.


Yea you could make your own custom authentication,its like writing a middleware for django.


Refactor begun in 65f32a4.


CookieDecoder


fix user can not get login logs bug


Mmm, I removed that because IMO Compose should not know about recipients and we don't have "direct" dependency to it anymore. Do you think we need similar test somewhere else?


Please move your db to Postgresql.


TeamCity Jarvis Framework :: CI-GitFlow [Build 16](http://build.prxm.it:8811/viewLog.html?buildId=3367&buildTypeId=CqrsFramework_CiGitFlow) is now running


this commit breaks compatibility with the 1.0.1 :(


Nice!


Fixes https://github.com/nnattawat/flip/issues/27


why are you using float fminf for comparing long?


오오 대단한데?


Add proper validations.


이게 모니


Refactor begun in 4a56316.
Yes for SymbolReference ; I've scheduled to use all the nice Symbols you created soon (that's what I implied by saying "my Expression > Identifier is crap and should be replaced"), but I've got to do some additional code analysis first.


희은이 코드 잘짯네


너 나가


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.0.1-beta.1+4](http://build.prxm.it:8811/viewLog.html?buildId=3367&buildTypeId=CqrsFramework_CiGitFlow) outcome was **FAILURE**
Summary: Tests passed: 108; exit code 1 Build time: 00:02:09


@tinwelint Any chance for this fix to get included in 2.2.4?


오오오오


무플방지


This is nice and all but your branch is behind now.


ah yeah ok, got it


sorry, I misread that one. Ignore me.


Related PR: #424.


Awesome! Hope it gets accepted!

I think it would be useful if you added documentation of this feature too.


mmm wondering if merely calling setupLock should not refresh? What do you think?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.432.5e9883c1afec4bc05a48cf8f237af4ffbe27530b](http://build.palaso.org/viewLog.html?buildId=59357&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3306, ignored: 124 Build time: 00:09:13


Also [DbDocs] Update to creature linking using dbdocssubtables by @Olion17

Based on the following pr: https://github.com/mangoszero/database/pull/11


it was used to keep the whole T_liveliness array length
opcache won't work without it.


Please add comments in the future. This is hard to decipher.


74f1258 Done


Fixes #3


you are right the key wrapping is only used in those two places. I am adding the integration tests now, however, I am not sure how to actually run them properly? where do I put my HSM credentials etc?


Jihawww! :+1:


addresses #14 and #105


@CurlyMoo

existing installs can be switched to Jessie just by changing "wheezy" to "jessie" in sources.list. APT will keep  needed packages even if from previous generation of system (if another package requires it).
(for RPI1 raspbian needs to be used again).

RPI2 on the other hand should go to Debian/Jessie. even existing Raspbian system can be updated directly to Debian/Jessie by changing not only wheezy->jessie, but http://address to (to debians). then dist-upgrade will do the job in about 20-30m + restart. dist-upgrade command should be run with rpi2-jessie repo already set (instead of rpi-wheezy). it solves some minor conflicts and contains rpi2 4.1 kernel, libcec3, and with NEON/arm7a architecture compiled KODI.
part of the added value by switching to Debian directly is that all RPI2 stuff is compiled in the same environment as IMX6.


better to user self.env.uid


you can omit 'string' parameter here.


Да и лучше оставить `try/catch` как и было раньше, либо весь оставшийся код перенести в finally-блок. А то если будет ошибка, то код после finally-блока не выполнится.


added.


Thanks for review, addressed in https://github.com/MSOpenTech/cordova-plugin-media/commit/f503d1675658c047c01adc257111041858d063d0


Why don't we enforce this process from `xbian-update`?


Can add comment, each define has "<band><channel>" values (e.g. SET_A0 is band A channel 0). I need to change channel to range from 1-8 rather than 0-7 (as it's not required for a define, only an array!).

Only reason against reversing bit order is that the documentation of chip assumes LSB and if we are going to do the math to convert MHz to Hex command, then the output will always need reversing


I've now pulled out the flake8 tests as a separate tox (and Travis) environment - 147267c2df1af0ba606eb34905223ac4421d5eca . Having it run in `before_script` meant that flake8 errors stopped the tests from running at all, which is far from ideal, because you can't see the more serious problems until you've fixed the minor ones. Thanks for your help on this.


pls add comments..


@rogatty - have you added translations to crowdin?


I think the problem with this is that we still can't read back structs from other languages, as they might encode the key/value pairs in a different order than we are trying to read them. Actually, IIRC, the msgpack spec does not say anything about structs in general only about maps. That is, directly mapping a (Rust) struct to a hashmap is a difficult thing inter-language-wise, or at least doing so without buffering the input.


print j.slug?


Is this more readable than

````
var getLabel = function(field) { return vm.getLabel(vm.workPackage, field); }
````

:question:

At least this is the meaning that I parsed from that `function(x) { return function(y) { }}(z)`. I might also have misunderstood what is happening here...


can both items either be decoded or encoded on the MW side?


print j.get_absolute_url_online()


I can't remember exactly what was the issue, but I did this initially and then removed since updating Compose properties in beforeEnter had some undesired effects. I can try once again and see if something has changed...


this can also be done on MW side.


Can you specify what the problem?


Oops, `<=`


I am working on it.There will few more changes to the way middlewares work before it will be merged


Also: You probably want to use your newly created `getLabel` method for both sides of the `localeCompare`


I thought that it would be common practice to do it like this.
But I can change it..


Les trois commande à éxecute quand tu as télécharger les mises à jours

php composer.phar require friendsofsymfony/rest-bundle (Bundle api rest )
php composer.phar require jms/serializer-bundle
php composer.phar require nelmio/api-doc-bundle


see above


I just noticed this one when I made the getXxxResponse methods typesafe as well. I believe that, because neiter TxStatus or ZBTxStatus has any fields of its own (they just access the frame data directly), the wrong method did actually work properly :-)


Speed of deployment and the fact that the mirror only works for docker hub.
On Jul 7, 2015 08:22, "wingedkiwi" <notifications@github.com> wrote:

> Is there any particular reason why the registry mirror was removed from
> the setup?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GoogleCloudPlatform/kubernetes/commit/db00532b471585b4afa255d3879e9b306efe8416#commitcomment-12032971>
> .
>


@memphys возможно. Вот только я думаю, все всё равно ищут по Symfony2


Oh, indeed I overlooked the exit of the 2x/4x loops :( And our test suite did not catch that.

Anyway, I still don't get it. Are you recommending this?
```
.Lxtsencloop:
	ld1		{v1.16b}, [x1], #16
	eor		v0.16b, v1.16b, v4.16b
	encrypt_block	v0, w3, x2, x6, w7
	eor		v0.16b, v0.16b, v4.16b
	st1		{v0.16b}, [x0], #16
	subs		w4, w4, #1
	bne		.Lxtsencloop
.Lxtsencout:
	next_tweak	v4, v4, v7, v8
	st1		{v4.16b}, [x6], #16
	FRAME_POP
	ret
```

But then, there is no more `next_tweak` in the `.Lxtsencloop` loop?


@amorgner Yes, the PR that introduced the commit was merged into 2.2: https://github.com/neo4j/neo4j/pull/4903


nevermind my comments above...

I just saw: https://github.com/floriank/openproject/commit/a3a4459d130162cc5765124268f38a0a426ae302


@sergeylunev не факт) Я уже давно не ищу с цифрой 2. Прошло уже почти 5 лет с релиза же


ok merci, je dois te dire que je n'ai le temps de rien, pas que je ne sais pas ou que je ne veux pas, mais juste peux pas, je ne te fais pas la liste des trucs qui m'en empeche pour ne pas m'épencher sauf si tu le desires, c'est juste pour te tenir informé au plus juste.


@memphys я постоянно ищу. Потому что зачастую на просто Symfony2 выдает не совсем релевантные результаты.


On 7 July 2015 at 11:34, Jérôme Forissier <notifications@github.com> wrote:

> Oh, indeed I overlooked the exit of the 2x/4x loops :( And our test suite
> did not catch that.
>
> Anyway, I still don't get it. Are you recommending this?
>
> .Lxtsencloop:
>     ld1     {v1.16b}, [x1], #16
>     eor     v0.16b, v1.16b, v4.16b
>     encrypt_block   v0, w3, x2, x6, w7
>     eor     v0.16b, v0.16b, v4.16b
>     st1     {v0.16b}, [x0], #16
>     subs        w4, w4, #1
>     bne     .Lxtsencloop
> .Lxtsencout:
>     next_tweak  v4, v4, v7, v8
>     st1     {v4.16b}, [x6], #16
>     FRAME_POP
>     ret
>
> But then, there is no more next_tweak in the .Lxtsencloop loop?
>

Ah yes, you are quite right. I don't remember exactly what I was thinking
at the time, but it was obviously wrong :-)

So I suppose you got it right the first time, sorry for the noise.


IBM Enterprise Cobol 5.1, pp299-300 :

>For all formats:
>
>*identifier-1*, *identifier-2*
>In format 1, must name an elementary numeric item.
>In format 2, must name an elementary numeric item except when
>following the word GIVING. Each identifier that follows the word GIVING
>must name an elementary numeric or numeric-edited item.
>In format 3, must name an alphanumeric group item or national group
>item.
>
>*literal* Must be a numeric literal.
>Floating-point data items and literals can be used anywhere that a numeric
>data item or literal can be specified.*

So ... yes : numbers only.


@sergeylunev ты даже тут автоматом написал Symfony2 =)


To run


=Ъ


Please advise


in the browser console jspm error is logged:
Uncaught ReferenceError: System is not defined(anonymous function) @ jspm_config.js:1


Concerning SyntaxNumber, it's on my TODO list. As I said, my Expression > Identifier is only temporary.


в общем, пусть будет с циферкой, убедил)


Is this needed?


@metaloid I think the whole info window is missing a little bit in style.. do you agree?

let's try to give it a deeper LCARS feel... maybe let's use some reference from the net...

cheers.

like what we doing so far.


@vangheem We all appreciate your work, but I cant find a matching PR for this massive and the following smaller  commits. Our policy is: No changes on master. PR is mandatory. Please follow this. We talked about such sneaked-in-features at last FWT meeting and decided all together its not ok. Even if its more work to make a PR: it increases overall quality of Plone and reduces possible mistakes in code and architecture.  It gives us a chance to decide as community if, how and when we want to change things. Thanks for your understanding


@portante great, thanks!


Thank you for this optimization.


Thanks for that :+1:. Just pushed a commit that does just that :)


Okay, apparently I am unable to read JS code. Starting with the good parts:

````JS
return left.localeCompare(right);
`````

This reads nice and understandable :+1:

Now on to the stuff I don't understand, snipping out stuff:

````JS
var getLabel = function(a) { return function(b) { ... } }('a');
````

I read this the following way:

1. You define an outer function, that takes one parameter and returns a function that takes one parameter itself
2. You immediately execute the outer function with a given parameter, assigning the resulting (1 parameter taking) function to `getLabel`
3. `getLabel` now is a function that takes one parameter (it is an instance of the inner function)

However, this contradicts the fact that you then call `getLabel` with two parameters.

And I still don't see the advantage of writing those nested functions. That leaves me with the conclusion, that I simply don't understand what is really going on here.


Looking good Chris.

When working with other developers on projects, or any website where you may be including library's or widgets or other peoples code, you don't want your code to get overridden...
Its worth having a read about "JavaScript Encapsulation and Name-spacing".  Its a practice that is good to get into whatever project or demo you are writing, and I found helped me early on to understand Javascript as a language, objects, methods, variables etc.

http://addyosmani.com/blog/essential-js-namespacing/#beginners


Done on c731c68ec5c719bf5a94b9fe136d5c5c315a3b3e

Thanks!


建不了。。。repository


LGTM :+1:


Potentially stores invalid ref to temporary


Ha! Was testing - will fix.


``+ ""`` is not a nice way to convert a number into a string. This will also not allow you to define a format, which you should.

Try to to condense too much, it makes it hard to read. A better (more readable) solution that is not slower (because the VM will optimize it) would be roughly as follows:

```java
// define one DecimalFormat for each phenomenon and put them in a map
[...]
String number = formatters.get(CO2).format(statistc.getMean() * ...);
StringBuilder sb = new StringBuilder();
sb.appen(< all the appends you have);
hm.put(CO2, sb.toString);
```

The decimal format will even allow you to use different locales in the future, e.g. using ``,`` or ``.`` as decimal sign etc.


Hello  sorry if i command in here but i need please help about pbot i write to you in irc pm please help to me thanks


No problem. Bad commit reverted and pushed.


Checks if a number is binary or not


What's the benefit in using `stderr`?


git log arch/mips/boot will show the subject lines used to editing files in that area is slightly different.

should be MIPS: CI20: Add DT node for led triggers

Also, adding the dt node for led4. Can that gpio still be exported from sysfs? For a user-led perhaps. What about the others. Does exporting from sysfs disable their functions. Or are they not usable by sysfs when used as triggers?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.728.d7b206ed68daae1cf072818605659721e7fd2553](http://build.palaso.org/viewLog.html?buildId=59365&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 363, ignored: 3 Build time: 00:05:30


ab9c62d Done


fast-forwarding end-to-end/timeseries to borshop-integration-1149-gg/tidyup/temp = b12a7dc7


When I was in school, we had a book (an old commentary on an even older short book on Islamic law) where most of the pages were one or two lines of body text and the rest is footnotes, some pages didn’t even have any body text at all and they would just put a dotted line as a place holder.


The idea was that it would be more noticable.


Thanks!


`<script src="//code.jquery.com/jquery-2.1.4.min.js"></script>`
or
`<script src="https://code.jquery.com/jquery-2.1.4.min.js"></script>`


Makes sense! Thanks for the reply.


Comment line needs updating if we keep this change.


Thanks Will! I'll have a read over the article and make sure to namespace everything afterwards !


I'm sorry, but items not working correctly again. Bot cant find any items with ' symbol, for example Kyr'Ozch or o'nine-tails. Interesting, but 2 days ago it was fixed and worked, but now not working again.


See also #106. :-)


TeamCity Jarvis Framework :: CI-GitFlow [Build 17](http://build.prxm.it:8811/viewLog.html?buildId=3370&buildTypeId=CqrsFramework_CiGitFlow) is now running


The first two work. Why doesn't it work to call on the user.avatar_url? Or is it a slim-language thing?


Remove all cms service. From line 13 to 58. It added on PLF, so, we don't need add it.


It does not store a ref but an object. This should work, shouldn't it? Maybe it is misleading to use references in the constructor, though.


you made a typo spelling canola


Not sure where someone's going to see `stderr` over the rails log. It doesn't output in `rake` tasks for example. If you're running in passenger I guess it going to go to `/var/log/httpd/error_log` and nginx… not sure where thin's error log would go. Did you have any example cases?


:warning: spacing between operator and numbers


I'd prefer it if this could be made generic: something like a `add_copy_spec_limit()` call that accepts a limit, path, and a pattern and collects all files up to limit from path that match pattern (with predictable ordering).

This would be useful and fits in with our general intent to overhaul/rewrite the file size limits interface (see #281).


Why?


Hi Victor @vic,

I'm really sorry to have to ask you this after so long, but to be in compliance with Google's open-source policy would it be possible for you to sign the Google CLA? https://cla.developers.google.com/

If you would rather not I understand. In that case I would have to remove the code from the repo.

Let me know, thanks!
Josh


97648758-VentSlabCoolCoilOutput-4978 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1683 of 1683 tests passed)

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


I started with this but Rubocop was complaining about the `has_` prefix. It doesn't seem to be complaining about it anymore. I kept the methods as they are but added an alias for each of them with the `has_` prefix
errors? => has_errors?
api_errors? => has_api_errors?
shipwire_errors? => has_shipwire_errors?
warnings? => has_warnings?

Or would it be better the other way around?


Fixed in commit cf358b5


What did it treat it as?  That looks like a rename to me (which is what it is)?


Why did you remove this?


Use a struct.


Two element loop? Maybe just have a function and call it twice.


what was the reasoning behind inline style and inline border?


as the version of your schema is “2015-04-01", please put your schema file into folder "2015-04-01".


Ah, duh, yeah. Done.


Long on


I noticed you use a lot of class names called "[element]style".

If you're not setting these specifically (ie you're setting all paragraph tags with this class) you can do generic selectors `p` instead of `.mainbodytext`.


good catch!


and 연산자는 && 연산자로 변경하는게 좋을 것 같네요


Yeah, duly noted. 1f2166f3cf53eec584f3f6844e671550ffc64540


97648758-VentSlabCoolCoilOutput-4978 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1683 of 1683 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)


lose this space


looks good!


love it


This is a comment.


LGTM.


def markdown_title
  render_as_markdown(title)
end

def markdown_body
   render_as_markdown(body)
end


Look at you and your fancy commit message


Hmmm, as this script runs in context of systemd unit, all its stdout should go to journal. So why not just `print` everything? If you want to set error level, you can start line with `<3>` (which is the syslog level for errors).


add to this file:

def markdown_to_html(markdown)
  renderer = Redcarpet::Render::HTML.new
  extensions = {fenced_code_blocks: true}
  redcarpet = Redcarpet::Markdown.new(renderer, extensions)
  (redcarpet.render markdown).html_safe
end


Function name should probably not be capitalized since it is not a constructor. Could perhaps use a more descriptive name as well (should be a verb).


thanks, fixed that


add to posts helper:

```
module PostsHelper

  def markdown_title
    markdown_to_html(@post.title)
  end

  def markdown_body
    markdown_to_html(@post.body)
  end

  def markdown_to_html(markdown)
    renderer = Redcarpet::Render::HTML.new
    extensions = {fenced_code_blocks: true}
    redcarpet = Redcarpet::Markdown.new(renderer, extensions)
    (redcarpet.render markdown).html_safe
  end

end
```


@Alfwich :+1: :smiley: Thanks for all the commits! I'll be working on the engine...but you can too, if you want to. It's just that I've been busy.

Your work ethic is awesome :).


44-46: UserProfile.objects.create(user=new_user, nickname=nickname) ?


Conditional body should be on next line (and enclosed in braces).


Shut the fuck up
On Jul 7, 2015 8:16 PM, "CrackerTeg" <notifications@github.com> wrote:

> Look at you and your fancy commit message
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BrokenROM/device_lge_g3-common/commit/bc6efe5456006ffdb74a6cc97b720c8f3f675f9d#commitcomment-12051271>
> .
>


if requiest.method != "POST":
    return render(request, 'session/register.html')
username = request ...

으로 하면 flow를 더 보기 쉽게만들고 indentation을 줄일 수 있음


Thanks z


UserProfile과 User가 OneToOne이기 때문에 request.user.userprofile로 접근가능함


膜拜onz


The spec says
> Picks a random value x (at least 320 bits)

Is there any reason to make it bigger than 320-bits? Does it make sense to make it configurable?


NP Joe.


한 학기가 지난 코드지만 ... https://www.python.org/dev/peps/pep-0008/#programming-recommendations
if userprofile.calendar_id is not None:


What is the intent here?

#1219 refers to LibSass **not** throwing an error when it should. A caveat of #1219 is that the issue only happened on `.sass` files. I think this commit can be backed out.


credit to scott
wasn't sure how to commit from his account


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


migrations도 관리돼야하지 않나요?


Yes I do. Whoops!


I really suspect this whole `self.file:` stuff is what is keeping the file locked.  I will verify this tonight with the new build 106.